Rails 3 - 重构红宝石条件

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