混乱せずにモンキーパッチを適用する3つの方法

Enumerableの理由がわかりません sumがありません 方法。よくあることですが、[1, 2, 3].sumと書けたらいいのにと思います。 、または["a", "b", "c"].sum 、または[Article.new, Article.new, Article.new].sum …ああ。

クラスにモンキーパッチを適用するときは、通常、簡単にしたいことを1つ考えています。 数値の合計を計算したいが、配列が他のものを保持できることを忘れてください。

今のところ、それは理にかなっています。 あなたは ハッシュの束の平均を計算しようとしないでください。ただし、メソッドを呼び出すときに失敗するだけのオブジェクトにメソッドをアタッチすると、後で混乱することになります。

これにはいくつかの方法で対処できます。最高から最低へ：

• 予期しない入力を合理的に処理します。

  これは、パッチが文字列で機能する場合にうまく機能します。 to_sを呼び出すと、ほとんどすべてのものから妥当なものを得ることができます 最初にそれに。そして、Confident Rubyは、さまざまな種類の入力を処理するためのさまざまな方法を教えてくれます。

• エラーをより明確に処理します。

  これは、ArgumentErrorをスローするのと同じくらい簡単です。 予期しない入力が表示されたときに、良いメッセージが表示されます。ランダムなNoMethodErrorを理解している他の誰かに依存しないでください s。

• コメントに期待する入力の種類を文書化します。

  他の2つのオプションは、使用できる場合はより適切です。ただし、パッチ内のエッジケースを確認できない場合は、少なくともそれらを文書化してください。そうすれば、発信者が問題の原因となっているのがパッチであることがわかると、発信者はあなたが何をしようとしていたかを知ることができます。

module CoreExtensions
  module Hash
    module Merging
      def string_merge(other_hash, separator = " ")
        merge(other_hash) {|key, old, new| old.to_s + separator + new.to_s}
      end
    end
  end
end

{}.string_merge({:class => "btn"}) # => {:class=>"btn"}

h = {:class => "btn"} # => {:class=>"btn"}
h.string_merge({:class => "btn-primary"}) # => {:class=>"btn btn-primary"}