issues
search
IkumaTadokoro
/
quitcost
「無職になったらいくらかかる?」を簡単に計算できるサービス
https://quitcost.herokuapp.com/
25
stars
2
forks
source link
パラメータの処理をParamParserクラスに一任
#138
Closed
IkumaTadokoro
closed
2 years ago
IkumaTadokoro
commented
2 years ago
Closes #100
やったこと
コントローラーから受け渡されるパラメータについて、バケツリレーで受け渡しを行なっており、コードの見通し的にも記述量的にも好ましくなかったため、ParamParserクラスを受け渡すように変更
パラメータから各クラスで導出していた給与、社会保険料についても重複していたため、ParamParser側の責務として抽出。各クラスに不要な引数を渡さないようにした。
PR提出前のチェックリスト:
[x] PRの関心は
ただ一つ
だけになっている & 文法的に正しく、明確かつ完全なタイトルと本文になっている
[x]
良いコミットメッセージ
を書いている
[x] 関連issueがある場合、コミットメッセージに
Closing Keywords
を使っている
[x] Featureブランチは最新版の
main
ブランチに追随している (そうでなければrebaseすること)
[x] 関連するコミットはsquashした
[x] テストを追加した
[x]
bin/lint
と
bin/rspec
を実行した
Closes #100
やったこと
PR提出前のチェックリスト:
main
ブランチに追随している (そうでなければrebaseすること)bin/lint
とbin/rspec
を実行した