[オブジェクト指向エクササイズ]「プリミティブ型と文字列型のラップ」について

Question

閲覧ありがとうございます。
http://baba-s.hatenablog.com/entry/2014/01/17/110133
オブジェクト指向エクササイズということで上記のサイトを見ているのですが「プリミティブ型と文字列型のラップ」についての項目で「修正後はこのようになります」という図がこのように表示されています(52ページ目)。

これは一つのクラスにこれらを記入するということなのでしょうか。
だとするととても見づらいし、1クラス50行以内というのが難しくなるような気がするのですが。。

見ていて気になった点です。お時間がありましたら回答・アドバイスよろしくお願いいたします。

Answer 1

こんにちは。

これは一つのクラスにこれらを記入するということなのでしょうか。

文脈的にはそうではないと思います。それぞれ別のクラス定義ではないでしょうか？
プリミティブ型と文字列型をラップすると言う話題ですので、クラスの包含関係については特に触れていないと思います。

---

【蛇足ですが】
内容をざっと見る限り、有用な方針も多いと思いますが原則として全て「常に適用する」と考えるとハマると思います。その方針を適用した方が良い場合と、適用しない方がよい場合の両方が存在するように感じました。
将来的に拡張することが想定されない場面で、BookNameクラスや他の同類クラスを多数作るのは如何なものかと思います。
拡張性を考慮することは好ましいですが、コード量を増やすのは好ましくないです。
一長一短ありますので、拡張が想定されるかどうか一考した上で、適用するしない選択するべきかと。

1クラス50行以内

個人的にはこれはナンセンスと思います。いくらなんでも1クラス50行はないです。
1メソッド50行以内のような方針は時々聞きます。

Answer 2

これは一つのクラスにこれらを記入するということなのでしょうか。

次のスライドにもあるように、「別のクラスに抽出する」作業です。

閉じカッコが省略されているので見づらいですが、クラスが4つある状態です。

Comments

[otabeb]

なるほど！回答ありがとうございます！