- Ruby / Rails関連
READ MORE
原著者の許諾を得て翻訳・公開いたします。
以下のようなコードは多くのプロジェクトで目にします。
class Something
before_validation :strip_title
def strip_title
self.title = title.strip
end
end
しかし、この要件を満たす書き方は他にもあるのです。
class Something
def title=(val)
@title = val.strip
end
end
あるいは以下のようにも書けます。
class Something
def title=(val)
self['title'] = val.strip
end
end
.あるいは以下のようにも。
class Something
def title=(val)
super(val.strip)
end
end
クラス内でのデータ保持方法に応じたものを選べばよいのです。さまざまなgemでさまざまな方法が使われています。
私がこの書き方を好む理由は以下のとおりです。
val
がnil
の場合にコケる。そう、そして私はこれがよいことだと思っています。自分たちのフロントエンド側からnil
がタイトルとして送信されることなどめったにないので、たまたまnil
になったときに何かがコケて例外を発するのは問題ありません。いずれにしろこんなことはまず起こりません。私のプログラマーとしてのトカゲ脳はひととおりのエッジケースを察知していますし、私のトカゲ脳のこういうところが大好きです。しかしこのトカゲ脳がごくたまに私を裏切って、まず起こりえないようなエッジケースに焦点を合わせてしまうことがあります。something.code = " 123 "
something.code
# => 123
something.save(validate: false)
私は、Command ObjectやForm Objectといった境界を超える用途に使われるオブジェクトに対して、バリデーションを消し去るルールを挿入する手法が特に気に入っています。
以上で、ささやかなレッスンはすべて終了です。もっとお知りになりたい方は、元記事末尾のメーリングリストに加入いただくか、私どものFearless Refactoringの購入をぜひお願いします。