Rails 3 - 重构 ruby​​ 条件

Question

我想知道是否有更简单的方法可以在 ruby​​ 中执行这两个条件：

if params[:action] == 'index' || params[:action] == 'show'

并

if !(comment = (session[:my_params].include?(:comment) rescue nil)).nil?

提前致谢

Answer 1

对于第一行，您可以这样做：

if %w(index show).include?(params[:action])

第二行实际上应该重构为两行：条件检查中的赋值是代码味道；从来没有理由。

如果您使用 Rails / ActiveSupport，则可以利用 Object#try

comment = session[:my_params].try(:include?, :comment)
if comment
  # ... comment is in scope
end

否则，您会遇到一些稍微笨拙的东西：

comment = session[:my_params].include?(:comment) rescue nil
if comment
  # etc

Answer 2

1:

if ['index','show'].include? params[:action]

2:

if (comment = (session[:my_params].include?(:comment) rescue nil))

第二个条件中的 ! 和 .nil? 是多余的

但是，实际上，您不应该尝试使所有内容尽可能短，这是首先要关心的是你的代码对其他人来说有多清晰。第二个条件应该是这样的：

if ( comment = (session[:my_params] && session[:my_params].include?(:comment) )

或者甚至

comment = session[:my_params] && session[:my_params].include?(:comment)
if comment 

Answer 3

第一个可以像这样折射：

if ['index', 'show'].include? params[:action]

或

if %w(index show).include? params[:action]

Answer 4

这应该比使用数组和 include? 更快：

case params[:action]; when 'index', 'show'
   ...
end

第二个：

if comment = (session.fetch(:my_params, {}).include?(:comment))