Session 毎のレビュー項目

より意図が分かりやすくなり可読性が向上する

const をつけることができ、パフォーマンスがより向上する

より適切なものを使うことによって、可読性が向上する

縦画面固定は必須ではなく、対応していた場合のレビュー項目 SystemChrome.setPreferredOrientations の方法では制限事項があり、特定の条件では機能しない ref: https://gist.github.com/blendthink/7529efc701be36b9bd1f3a9b55570225

ページ内に補足あり📝

const で定義できる SizedBox や Padding で置き換えられないか確認する

後々、画像追加する際に都度修正する必要がなくなり、保守性が向上する

不要なインスタンス生成を日頃から行っていると、ループ処理などの重たい処理を行う際にも記述してしまってパフォーマンスに影響を及ぼしてしてしまう恐れがあるため、日頃から無駄なことをしないように意識する

APIが修正されて不具合が入り込んだ時のことを想定しておくと、より安全性や保守性が向上する

分岐処理で switch を使用することができ、可読性や保守性が向上する

API が意図せずに変更されたり、不具合が入り込んだりした時のことを想定して、API の処理時に Error が発生しないようにすることで、より安全性や保守性が向上する Enum の .values.byName()でスローされるのは ArgumentError で、 アプリケーションコードで Error をキャッチするのは適切ではないため、Exception をスローする or null を返す拡張関数を自作する ref: https://dart.dev/effective-dart/usage#error-handling

flutter_gen または、独作

予期せぬリビルドが生じる可能性があるため 代わりに、initStateに処理を書いているか

スレッドをブロックしてしまうから

非同期処理で UI を操作しようとするときに mounted のチェックをしていないと、 Widget が Widget Tree に マウントされていない可能性があり、エラーが生じることがある

go_router のバージョン 6.5.0 ~ 6.5.4 だと iOS のスワイプバックで戻ってきた時に、再帰処理が実行されないことがあった。それが原因で、初期画面から天気画面に遷移しないことがあったため、実際の案件ではこのような iOS・Android のスワイプでの挙動の差異がないかというところにも気をつける go_router のバージョン 6.5.5 で不具合修正済み ref: https://github.com/flutter/packages/pull/3613

• void afterDisplayLayout() ⚪︎ • void afterDisplayLayout() {} ×

ネストが深くなるのを防ぎ、可読性・保守性が高まる

可読性・再利用性を高める

https://dart.dev/effective-dart/usage#error-handling

ページ内に補足あり📝

json_serializable or freezed を利用していたら、不要

ページ内に補足あり📝

デフォルト値を記述すると、他のデフォルト値も全て記載しないといけなくなりそうで、メンバーに混乱が生じる

ページ内に補足あり📝

ページ内に補足あり📝

２つのパッケージは関連が強いので、コミットをまとめておいた方が良い

ページ内に補足あり📝

ページ内に補足あり📝

ref: https://docs-v2.riverpod.dev/docs/concepts/reading#dont-use-refread-inside-the-build-method

ref: https://docs-v2.riverpod.dev/docs/concepts/reading#dont-use-refread-inside-the-build-method

特に意図がない限り基本的に状態は保持しつづけるべきではない

• アーキテクチャの説明でなく、実装の説明になっていないか ◦ 具体的なクラス名やファイル名を記載すると、リネームのたびに修正が必要になる

ページ内に補足あり📝

riverpod_graphを使うことでProviderの依存関係図を自動生成できる https://github.com/rrousselGit/riverpod/tree/master/packages/riverpod_graph

• @visibleForTesting だけの場合、警告の表示で無視することもできてしまうため、analyzer の error に設定した方がより安全 • ref: https://kanta-mori.netlify.app/p/visiblefortestingとは/

ref: https://docs-v2.riverpod.dev/docs/cookbooks/testing

依存先を明確にしてテストダブルを利用しているか

ページ内に補足あり📝

ページ内に補足あり📝