ruby-on-rails - 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