if/then/else条件でRubyメソッドをコード化する最も明白な方法は何ですか？

Question

私はこのクラスレベルのメソッドをActiveRecordモデルクラスに持っています。

def self.is_animal_color_correct?(animal, color) 
    if AnimalColor.find_by_animal_and_color(animal.downcase, color.downcase) 
    true 
    else 
    false 
    end 
end 

私は、方法をフォーマットする最良の方法が何であるか疑問に思っています。これは冗長ではあるようですが、非常に明確です。

Answer 1

、私はこれがあなたの代わりに欲しいものだと思う：

一般に

AnimalColor.exists?(:animal => animal.downcase, :color => color.downcase) 

、あなたは限り、あなたは何もnullまたは虚偽ではないことを、truthyに対してチェックされているとして気にしてはなりません。

Answer 2

def self.is_animal_color_correct?(animal, color) 
    AnimalColor.find_by_animal_and_color(animal.downcase, color.downcase) 
end 

またRubyでtruthyもののための非truthy値と実際の値（使用可能な場合）としてnilとfalseを使用することが慣用的です。 if/elseを使用すると、他の場所で役立つ可能性がある情報を積極的に破棄し、そのプロセスでより多くの作業を行うことができます。

あなたが本当に本当に本当にあなたは可能性が、あなたの方法だけtrueまたはfalseを返すようにしたい場合：

def self.is_animal_color_correct?(animal, color) 
    !!AnimalColor.find_by_animal_and_color(animal.downcase, color.downcase) 
end 

しかし、再び、これは慣用ルビーではない、と私はそれをお勧めしません。この特定の例では