tukamon5151 / trelllo-copy

お勉強目的でtrelloのコピーを作るリポジトリ

2 stars 0 forks source link

useUserの設計が悪いので治す #17

Closed tukamon5151 closed 3 years ago

tukamon5151 commented 3 years ago

現在useUserはNextAuthのuseSessionの結果を返すhooksになっているが、ネットワークの通信送受部分とそれも含めたグローバルステーツのコンポーネントへのデータのプロバイダーの役割は切り離しておくべき(テストの可用性の観点でも)

tukamon5151 commented 3 years ago

そもそもログインしているユーザーという意味のuserなのか、userというモデルを表すものかがわからず、そもそも命名が悪い。名前を書くならuseCurrentUserとかに変えたほうが良い。で、mypageとかのuserデータはcurrentUserのデータを取るのじゃなくて、currentUser.id経由でデータを取り回すようにする。

tukamon5151 commented 3 years ago

フロントエンドでは、useCurrentUser hooksでcurrentUserオブジェクトを常に持っておく
currentUserオブジェクトはユーザーについての最低限の情報だけ持っておき、それ以上の情報が必要になったら別途のUserクラスシリアライズする
API側のデータIOのインターフェイスはDTOに集約
フロントエンドで使うクライアントModelは別に置いておく

tukamon5151 commented 3 years ago

tukamon5151 / trelllo-copy

useUserの設計が悪いので治す #17

23