Как я могу сделать этот код более Ruby-esque?

Question

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

Answer 1

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

Answer 2

Это немного СУШКИ:

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

Answer 3

Я бы написал этот блок следующим образом.Это действительно сводится к предпочтениям, но я считаю этот способ наиболее читабельным.

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

Answer 4

Или:

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

Answer 5

Вы можете сделать:

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