お気に入りユーザー一覧画面実装 #49
お気に入りユーザー一覧画面実装 #49
Conversation
| // ボタンを押したら、すぐに画面遷移できるようにinitStateでAPIからデータを取得 | ||
| Future(() async { | ||
| await ref | ||
| .read(favoriteUserListProvider.notifier) | ||
| .fetchFavoriteUserList(widget.statusId); | ||
| }); |
| @freezed | ||
| class FavoriteUserList with _$FavoriteUserList { | ||
| const factory FavoriteUserList({ |
There was a problem hiding this comment.
このクラス名だと、Listだけ持ってるクラスを連想しちゃう人も少なからずいると思うので、FavoriteUserListStateのような命名にしてあげると、状態管理で使うクラスであることをもっと想像しやすくなるんじゃないかなと思いました！
修正は任意です。
| class PgMobileDefaultButton extends StatelessWidget { | ||
| const PgMobileDefaultButton({ | ||
| required this.backgroundColor, | ||
| required this.onPressed, | ||
| required this.buttonText, | ||
| required this.buttonTextStyle, | ||
| required this.buttonHeight, | ||
| required this.buttonWidth, | ||
| super.key, | ||
| }); | ||
| final Color backgroundColor; | ||
| final void Function() onPressed; | ||
| final String buttonText; | ||
| final TextStyle buttonTextStyle; | ||
| final double buttonHeight; | ||
| final double buttonWidth; |
There was a problem hiding this comment.
クラスの命名について
AppElevatedButtonクラスとかでも良いのかなと思いました。
他のクラスはAppColorsクラスのように、アプリ内で利用するクラスをAppと表現しています。
また、Defaultという単語に関してですが、アプリ内に登場するボタンが1種類の場合はDefaultでも良いのかな思います。ただ、今回のアプリはボタンが複数種類登場するため、Defaultがどのボタンを示すのか他のエンジニアがちょっと想像しにくいです。
クラス引数について
アプリ内で頻繁に使うWidgetにしてはrequiredの引数が少し多い気がします。引数でWidgetを自由にカスタマイズできる点はとても良いと思うのですが、クラスのインスタンスを生成するたびに6種類の引数を必ず入れないといけないのは、ちょっと使いづらい気がします。こういうクラスを設計する場合は、requiredだけでなく、デフォルト値とかも使ってみるとrequiredが減っていいんじゃないかなと思います。
UIコンポーネントの必要性について
そもそもUIコンポーネントのクラスを作る必要があるか考えてみてほしいです。同じ部分を共通化しようという考えはめちゃくちゃ大事ですし、これからも大事にしてほしいです。
ただ、今回の場合はカスタムなUIコンポーネントのクラスを作成するよりも、ElevatedButtonのThemeを一括で定義した方が、ElevatedButtonをそのまま使用できるため、他のエンジニアから見た可読性も使いやすさも向上するんじゃないかなと思いました（多分そのためにFlutter公式がThemeクラスを用意してくれている）。
| horizontal: 16, | ||
| vertical: 8, |
There was a problem hiding this comment.
余白もレイアウトの一種なので、他の箇所と同じように16.wと8.hで指定していただけると嬉しいです🙏
| // 画面のチラつきを防ぐために今までのデータをリセットする | ||
| ref.read(favoriteUserListProvider.notifier).resetData(); |
概要
お気に入りユーザー一覧画面を実装しました！
動作確認
create_favorite_user_list_page.mp4
参考