RubyのリファクタリングでNull Objectを使ってコードをスッキリさせる方法 - ベルリンのITスタートアップで働くジャバ・ザ・ハットリの日記

前回、Rubyのリファクタリングでイケてないコードを美しいオブジェクト指向設計のコードへ改良するための方法という記事を書いて、いい反響をいただいたので第２弾を書いた。 Ben Orenstein氏の講演で話されていた前回のとはまた別のリファクタリング方法。元ネタはこちら。 github.com 【リファクタリング前のコード】 class JobSite attr_reader :contact def initialize(location, contact) @location = location @contact = contact end def contact_name if contact contact.name else 'no name' end end def contact_phone if contact contact.phone else 'no phone'

[hrtkmztn]

お、読みやすい！

[yumura_s]

nullオブジェクトであるという情報を使わない場合、nullオブジェクト用クラスを作らなくても、デフォルト値を設定したインスタンスを返すファクトリメソッドみたいのがあれば十分なのでは？

[bamch0h]

rubyだけに限らず、使える話だと思う

[yohshiy]

なるほどとは思うけど、 Contact と NullContact との整合性の管理で問題がでやすそう。継承も使うともうちょい良くなる気がする。

[iTume]

これは使いたい