search

GotaSuzuki / spotify_music_imageChange

0 stars 0 forks source link

[must]App.jsxのリファクタ #30

Open hasegawa-kaname-opst opened 4 hours ago

hasegawa-kaname-opst commented 4 hours ago 
          [must]:https://img.shields.io/badge/review-must-red.svg

![must]

  1. App.jsxで定数を定義すると、他の定数も定義することとなり、肥大化します。モックデータであれば、srcと分離しましょう。
  2. 仮に定数だとすると、関数外に定義し、useStateを使う必要がありません。useStateはレンダリングを跨いで動的なデータを保持するために使用します。APIレスポンスのモックだとすれば、API呼び出し層 + APIキャッシュ層を切り出してください。APIの変更が本来変更影響のないUI実装にまで影響を及ぼしてしまいます。APIキャッシュは、自前でカスタムフックを用意し、useStateを使用しても良いし、ライブラリを使用しても良いです。API呼び出し時はローディングを実装しましょう。

_Originally posted by @iida-naoki-opst in https://github.com/GotaSuzuki/spotify_music_imageChange/pull/23#discussion_r1814728933_