我该如何制作这段代码，更多的Ruby风格？

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

你可以这样做：

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

Answer 3

或者：

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

Answer 4

我会写像以下问题的块。它真的归结为偏好，但我觉得这种方式是最可读的。

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

Answer 5

这是一个位机：

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