Closed IkumaTadokoro closed 2 years ago
Simulation系のクラスはすべてPOROで実装している。
「サービスオブジェクト等のPOROは慣習的にcallやexecでパブリックAPIを統一するべき」という言説に盲目的に従って従来はcallを使用していた。
call
しかし以下の点からよりクラスが持つ責務に合わせた名前が好ましいと判断し、メソッド名のみをcallからcalc(calculate)に変更した。
calc
PR提出前のチェックリスト:
main
bin/lint
bin/rspec
あくまでAPI呼び出しをcallからcalcにするだけであって、privateの領域で行なっている計算や、このサービス全体でcalculateではなくcalcを使わないといけない、という話ではない。
calculate
Simulation系のクラスはすべてPOROで実装している。
「サービスオブジェクト等のPOROは慣習的にcallやexecでパブリックAPIを統一するべき」という言説に盲目的に従って従来は
call
を使用していた。しかし以下の点からよりクラスが持つ責務に合わせた名前が好ましいと判断し、メソッド名のみを
call
からcalc
(calculate)に変更した。PR提出前のチェックリスト:
main
ブランチに追随している (そうでなければrebaseすること)bin/lint
とbin/rspec
を実行した