Comment puis-je faire ce code, plus 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

Vous pouvez faire:

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

Answer 3

Ou:

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

Answer 4

J'écririez le bloc en question comme suit. Cela dépend vraiment de la préférence, mais je trouve que c'est la façon la plus lisible.

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

Answer 5

C'est un peu sèche:

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