同じメソッドで異なる物を更新しようとしている時点でおかしい
適切な抽象化と関心の分離ができていないんだよ
条件によって筆者を更新するあるいは価格を更新するメソッド
意味不明だろ
このメソッドは筆者に関心があるのか価格に関心があるのか
条件が増えるたびにメソッド名や契約が変わりまくる典型的なクソ仕様だ
同じメソッドにある以上筆者の更新ロジックが変わったら価格の更新についてもテストしなきゃならない
どんだけ工数増やす気だよ
そうじゃなく書籍情報を更新するメソッドとするんだよ
条件なんていらない
筆者も価格も他の属性も単に上書きするだけ
条件によって変更が無いものは条件分岐を消して同じ値で更新するだけ
更新する値を計算する処理に条件をつける
適度に抽象的で全ての関心ごとが分離された美しい世界だ
