Comment SECHER cet extrait de code Ruby?

Question

Cela me dérange. Il ne semble pas trop sec. Quelle serait une meilleure mise en œuvre? En aparté, comment se fait-il que ce finder d'ActiveRecord ne lève pas une exception quand l'enregistrement n'est pas trouvé, mais .find fait?

def current_account 
    return @account if @account 
    unless current_subdomain.blank? 
     @account = Account.find_by_host(current_subdomain) 
    else 
     @account = nil 
    end 
    @account 
    end 

Answer 1

Je coderez cela comme

def current_account 
    @account ||= current_subdomain.blank? ? nil : Account.find_by_host(current_subdomain) 
end 

En ce qui concerne les exceptions, les méthodes find_by dynamiques retour nil au lieu de lancer une exception. Si vous voulez une exception, utilisez le find avec :conditions:

def current_account 
    @account ||= current_subdomain.blank? ? nil : Account.find(:first, :conditions => {:host => current_subdomain}) 
end 

Answer 2

Que diriez-vous:

def current_account 
    @account ||= Account.find_by_host(current_subdomain) unless current_subdomain.blank? 
end 

Answer 3

def current_account 
    @account ||= current_subdomain && Account.find_by_host(current_subdomain) 
end 

Si un enregistrement est introuvable, les méthodes find_by dynamiques retour à zéro, find_by_all retourne un tableau vide.

Answer 4

def current_account 
    @account ||= current_subdomain.present? && Account.find_by_host(current_subdomain) 
end 

#present? se chargera nil et des chaînes vides