search

jh6230 / todo-app

タスク管理アプリ
1 stars 0 forks source link

Todo Repository : 各種メソッド名に関して。 #3

Closed Masaya-Abe closed 3 years ago

Masaya-Abe commented 3 years ago

メソッド名は基本動詞にしましょう。 scala のコレクションの メソッド名を参考にしてください。

todoAllByCategory -> filterByCategory todoAllByState -> filterByState

とか

insertNullByCategory このメソッド名も意味不明ですね。 まず insert でないので。 やってることは categoryId に紐づく state を 0 にしているだけなので、 updateStatusToNull とかの方がわかりやすいです。

jh6230 commented 3 years ago

アドバイスと参考に見直しを行いました