Closed OKAUEND closed 1 year ago
closes #179
コードレビューを行って頂いた時に、レビュアーがコードの意図の把握に時間がかかってしまい、意図を把握し辛いと指摘を頂いた。
先に全件取得するPrismaのロジックを組み、そこに継ぎ足したため。 また元のロジックを変更しないようにするために、検索処理が入った関数を新設したため、似た処理が増えた結果になった。
Prismaを扱う関数で同じロジックの関数を1つにまとめた。 その時に用途名を付けていた関数名もリネームを行う。 クエリ作成関数も、Prisma関数をどこから呼び出しても扱えるようにするために、Prisma関数内に収めた。
Issue / Ticket
作業チケット
179
closes #179
課題/何が起こったか
コードレビューを行って頂いた時に、レビュアーがコードの意図の把握に時間がかかってしまい、意図を把握し辛いと指摘を頂いた。
仮説/どうしてそうなったのか
先に全件取得するPrismaのロジックを組み、そこに継ぎ足したため。 また元のロジックを変更しないようにするために、検索処理が入った関数を新設したため、似た処理が増えた結果になった。
どういう作業を行ったか
Prismaを扱う関数で同じロジックの関数を1つにまとめた。 その時に用途名を付けていた関数名もリネームを行う。 クエリ作成関数も、Prisma関数をどこから呼び出しても扱えるようにするために、Prisma関数内に収めた。
Next Point
変更画面のサンプル
参考資料