Question

Is there a pattern to refactor such a construct into a readable single line?

def show_section
  @news = News.all_active
  @news = @news.where(:section => params[:section]) unless params[:section] == "all"
  @news = @news.all
end

I use Rails 3 and Ruby 1.9.2

Answer 1

@news = News.all_active.where(params[:section] == "all" ? nil : {:section => params[:section]})

You can get rid of @news.all – in Rails 3 query will be executed when you use the resulting ActiveRecord::Relation object (for example when you call each or first on it). Passing nil to where method will do nothing.

If all_active is a method, you can refactor it into a scope and then call it in a chain.

Great resources on Rails 3 queries:

• Active Record Queries in Rails 3
• Advanced Queries in Rails 3
• Arel README