書籍「現場で役立つシステム設計の原則」写経用リポジトリ

疑問点メモ

addメソッドで100という数字がハードコーディングされているが、定数MAXの方がよいのでは？
また、100は業務的に正しい値なので、例外メッセージの"以上"は、"超"では？

【修正前】
throw new IllegalArgumentException("不正：合計が" + 100 + "以上");
【修正後】
throw new IllegalArgumentException("不正：合計が" + MAX + "超");

→ 正誤表に反映いただいた。

quantityが一定量より多いかの判定ロジックをisDiscountableに閉じ込めるなら、
isDiscountableはフィールド（プロパティ）ではなくメソッドになるのでは？

■修正前
if(quantity.isDiscountable)

■修正後
if(quantity.isDiscountable())

→ 正誤表に反映いただいた。

コードリストで、Customersコンストラクタの引数にresult.add(customer)を指定しているが、
ArrayListのaddメソッドの戻り値はboolean型なので、意図した動作にならないのでは？

■修正前（Customerコンストラクタにboolean型を渡している）
return new Customers(result.add(customer));

■修正後（CustomerコンストラクタにArrayList型を渡す）
result.add(customer);
return new Customers(result);

→ 正誤表に反映いただいた。

Map型のフィールド types だが、変数宣言しかしておらずインスタンス生成をしていないので、
staticブロック内のtypes.putをするタイミングでNullPointerExceptionになるのでは？

■修正後 (staticブロックの先頭でMapのインスタンスを作成する)
types = new HashMap<String, Fee>();

→ 正誤表に反映いただいた。