基礎的なレビュー項目

final = 実行時不変, const = コンパイル時不変（定数）

可読性・保守性が低下してしまうため スコープが狭く今後も広くなりづらい場合や、final で書くことによって記述量が増えてしまい可読性が低下する場合は使用してもいいかもしれない ただし、そのような場合はほとんどない

スコープは小さくすると安全性が高まって良い

目的によるが、print() では不十分な場合がある ref: https://rizumita.medium.com/logging-in-dart-dd9c01eb459a

ref: https://medium.com/flutter-jp/immutable-d23bae5c29f8

if(条件式) { return … } (早期リターン) などでネストが深くなるのを回避する

単純に return するだけの場合、わざわざ関数化しない

後から enum に要素が追加される場合、if-else で判定すると修正の必要性に気づけない可能性があるため、 switch - case 使う

意味のある数値を何も説明なく使用しない → 意味のわかる命名で定数を定義して利用するのが良い

ref: https://zenn.dev/iwaku/articles/2020-12-19-iwaku

• 通常はコードの変更にも安全に開発できるようにするために、ローカル変数にして型昇格を行ってから利用する • スコープが狭くほとんど変更されない箇所でnullじゃないと確信できるところは、 “!” を使うことを許容する

ビルドできる単位で、など

Prefix がついていると見やすい 「エラー直した」など具体性が低いものは NG

ずっとブランチを残していると、ブランチを切り替える際に徐々にコストがかかるようになってしまう

アーキテクチャグラフや環境構築の手順が記載されているか

*.freezed.dart・*.g.dart 等

VS Code では、settings.json に Code Spell Checker の設定を行う