ロバストネス原則でRubyコードを簡素化する

Rails2.1のActiveRecord::Errors＃onの実装を見てください ：

# File activerecord/lib/active_record/validations.rb, line 212
def on(attribute)
  attribute = attribute.to_s
  return nil unless @errors.has_key?(attribute)
  errors = @errors[attribute].map(&:to_s)
  errors.size == 1 ? errors.first : errors
end

呼び出されると、これは Arrayのいずれかを返す可能性があります Stringの s、 String 、または nil 。処理しているオブジェクトのタイプを把握するのは、呼び出し元の責任です。 これは悪い考えです：

• 呼び出し元は、目障りな型チェックを使用して独自のコードを混乱させる必要があります。

• 発信者は多くのを知っている必要があります 呼び出しているメソッドについて。少なくとも、メソッドが返す可能性のあるすべてのタイプのオブジェクトと、各タイプがいつ返される可能性があるかを知る必要があります。

• テストするエッジケースがさらにあります。コードが想定どおりに機能していることを確信したい場合は、3つのシナリオすべてを試してみる必要があります。

メソッドは、返されるものについて一貫している必要があります。 通常、 Arrayを返す場合 、常にするために必要なことを行います Arrayを返します 。通常、 Userを返す場合 、ただしnilを返す場合もありますが、Null Userオブジェクトを作成して、それを返すことができます。 nilの代わりに。

厳しくない場合もあります。「Taggableを含むものを返します。 モジュール」。より一般的にすることができます：「 idで何かを返すつもりです およびname 属性。」重要なのは一貫性と、発信者が何を期待できるかを確実に理解することです。

jQueryは興味深い例です。ほとんどのjQueryメソッドは、同じ種類の Arrayを返します -オブジェクトのように。このため、jQueryのメソッドは非常に構成可能であり、1行のコードで膨大な量の作業を行うことができます。

ご参考までに、Railsはそれ以降のバージョンでそのメソッドを修正しました：

# File activemodel/lib/active_model/errors.rb, line 133
def [](attribute)
  get(attribute.to_sym) || set(attribute.to_sym, [])
end

これで、常に Arrayが返されます 。彼らにとってはよりシンプルで、私たちにとってはよりシンプルです。