How can i make this code, more Ruby-esque?

Question

unless scope.nil?
   @page ||= Page.find(id, :scope => Page.find(scope) )
else
   @page ||= Page.find(id)
end

Answer 1

You could do:

@page ||= unless scope.nil?
  Page.find(id, :scope => Page.find(scope))
else
  Page.find(id)
end

Answer 2

Or:

@page ||= scope.nil? ? Page.find(id) : Page.find(id, :scope => Page.find(scope))

Answer 3

I would write the block in question like the following. It really comes down to preference, but I find this way to be the most readable.

@page ||=
  if scope
    Page.find id, :scope => Page.find(scope)
  else
    Page.find id
  end

Answer 4

This is a bit DRYer:

find_opts = scope.nil? ? {} : {:scope => Page.find(scope)}
@page ||= Page.find(id, find_opts)

Answer 5

@page ||=  Page.find id, :scope => (Page.find scope if scope)