Как высушить этот фрагмент кода Ruby?

Question

Это беспокоит меня. Он не выглядит слишком сухим. Что было бы лучше? Как в стороне, почему этот ActiveRecord finder не генерирует исключение, когда запись не найдена, но .find делает?

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

Я бы закодировать это как

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

Что касается исключений, find_by динамические методы возвращают nil вместо того, чтобы бросать исключение. Если вы хотите исключение, используйте find с :conditions:

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

Answer 2

Как насчет:

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 

Если запись не найдена, динамические find_by методы возвращают ноль, find_by_all возвращает пустой массив.

Answer 4

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

#present? будет обрабатывать nil и пустые строки