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

Question

Это беспокоит меня. Это не выглядит слишком сухо. Что будет лучшей реализацией? Кроме того, почему этот искатель ActiveRecord не выдает исключение, когда запись не найдена, а .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 && Account.find_by_host(current_subdomain)
end

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

Answer 2

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

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 3

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

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

Answer 4

Как насчет:

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