issues
search
IkumaTadokoro
/
quitcost
「無職になったらいくらかかる?」を簡単に計算できるサービス
https://quitcost.herokuapp.com/
25
stars
2
forks
source link
useValidationSchemaをリファクタリング
#188
Closed
IkumaTadokoro
closed
2 years ago
IkumaTadokoro
commented
2 years ago
やったこと
数値、日付の必須チェックをメソッドに抽出
変数
validationSchema
はチェック対象のみを保持するオブジェクトに修正
validationSchemaにチェック対象とチェック内容の双方を含んでいると、全体としてどのカラムにチェックがかかっているかがわかりづらい。そのためvalidationSchemaが保持しているチェック内容を、各メソッドに抽出した
ビルドサイズ削減のため、yupのimport対象を最小限に変更
バリデーションに必要な翌々年度を別モジュールから取得するように変更
会計年度に関する責務をuseFinancialYearに集約するため。また愚直な日付の加減算は結果がわかりづらく、privateゆえにテストもできないため、テスタブルな別モジュールに処理を移譲した
変更後の命名でも意味がとれるものは、スタイルを優先して短い名前に変更した
PR提出前のチェックリスト:
[x] PRの関心は
ただ一つ
だけになっている & 文法的に正しく、明確かつ完全なタイトルと本文になっている
[x]
良いコミットメッセージ
を書いている
[ ] 関連issueがある場合、コミットメッセージに
Closing Keywords
を使っている
[x] Featureブランチは最新版の
main
ブランチに追随している (そうでなければrebaseすること)
[x] 関連するコミットはsquashした
[ ] テストを追加した
[x]
bin/lint
と
bin/rspec
を実行した
やったこと
validationSchema
はチェック対象のみを保持するオブジェクトに修正PR提出前のチェックリスト:
main
ブランチに追随している (そうでなければrebaseすること)bin/lint
とbin/rspec
を実行した