ruby-on-rails - この Ruby コードのスニペットをどのように DRY しますか?

Question

これは私を悩ませています。それはあまりにもDRYに見えません。より良い実装は何でしょうか? 余談ですが、この 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 && 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は例外をスローする代わりに戻ります。例外が必要な場合は、with を使用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