第08章 実装を隠す

[at-grandpa]

TODO

• $5 + 10 CHF = $10（レートが2:1に場合）
• $5 * 2 = $10
• amountをprivateにする
• Dollarの副作用どうする？
• Moneyの丸め処理をどうする？
• equals
• hashCode
• nullとの等価性比較
• 他のオブジェクトとの等価性比較
• 5CHF*2=10CHF
• DollarとFrancの重複
• equalsの一般化
• timesの一般化
• FrancとDollarを比較する
• 通貨の概念

[at-grandpa]

FrancのtimesとDollarのtimesがにているので重複を省いていく。

[at-grandpa]

もう二つのサブクラスは大した仕事をしていないので、この際消してしまいたい。が、一気に消すのはステップが大きすぎる。小さく進めていく。

[at-grandpa]

crystalの場合はいい感じに型推論っぽいことをしているので、この時点でコンパイラはエラーを吐かない。（Money 型には times メソッドが定義されていない、、などのコンパイルエラー）

[at-grandpa]

利用者側からDollarのサブクラスを消すことができた。テストからサブクラスを消去したことで、既存のコードに影響を及ぼさずに継承構造を変える権利を得た。

[at-grandpa]

あー、なるほど。こういうところになかなか気づけないから、「こうした方が良い」というイメージがわかないんだな。今回のこの修正で、疎結合になっていく具体的なステップがわかった。

[at-grandpa]

TODO

• $5 + 10 CHF = $10（レートが2:1に場合）
• $5 * 2 = $10
• amountをprivateにする
• Dollarの副作用どうする？
• Moneyの丸め処理をどうする？
• equals
• hashCode
• nullとの等価性比較
• 他のオブジェクトとの等価性比較
• 5CHF*2=10CHF
• DollarとFrancの重複
• equalsの一般化
• timesの一般化
• FrancとDollarを比較する
• 通貨の概念
• testFrancMultiplicationを削除する?

[at-grandpa]

振り返り

• 重複を除去できる状態に一歩近づける為に、DollarとFrancにある二つのtimesのシグネチャを合わせた
• せめてメソッド宣言だけは親クラスに移動した
• FactoryMethodパターンを導入して、テストコードから二つのサブクラスの存在を消した
• サブクラスを隠した結果、幾つかのテストが冗長なものになったと気づいたが、そのままにした