Comment puis-je faire ce code, plus Ruby-esque?

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

2010-05-07 ktec

À l'avenir, s'il vous plaît indenter le code par quatre espaces pour qu'il ressemble vraiment 'montré sur quand recueil HSC Stack Overflow. –

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

2010-05-07 18:52:08 ryeguy

c'est le plus soigné certainement, et comme mentionné à la fin c'est à la préférence bien sûr. – ktec

Vous pouvez faire:

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

2010-05-07 14:37:08 mipadi

Ou:

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

2010-05-07 14:56:04 Dimitri

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

2010-05-07 15:11:03 Alex

C'est un peu sèche:

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

2010-05-07 15:36:35

Répondre