概要
flutter_lints の unrelated_type_equality_checks 違反を解消します。現状 2件 です。
静的解析とは、プログラムを実行せずソースを読むだけで「エラーではないが良くない書き方」を見つける仕組みです。基礎は親issue #286 を参照してください。このissueでは下記の1ルールだけを直します。
このルールについて(何を・なぜ・どう直す)
何を検出するか: 互いに無関係な型どうしを == で比較している箇所を検出します(例: String と int の比較)。
なぜ直すのか: 無関係な型の比較は常に false になります。つまり、ほぼ確実に「変数を取り違えている」「比較する値を間違えている」というバグのサインです。書き方の問題ではなく、ロジックの誤りである可能性が高い箇所です。
どう直すか:
// Before(status が String、定数が int → 常に false)
if (status == 0) { ... }
// After(正しい型・値で比較)
if (status == 'pending') { ... }
機械的には直せません。なぜこの比較になっているのか、どちらの型が正しいのかをコードを追って特定し、本来意図した比較に直してください。直したら必ず該当機能の動作を確認します。
対象箇所
lib/pages/manual_list_page.dart:33
lib/pages/users_page.dart:47
直し方
このルールは自動修正の対象外です。上の方針で1箇所ずつ手作業で直します。修正後、当該ルールが消えたか確認します。
cd mobile && fvm flutter analyze
進め方
この issue から branch を切り、PR を作成してください。issue→branch→PR の順を省略しないのがこのチームの正規フローです。
難易度
要注意・バグ修正。 単なる整形ではなく、ロジックの誤りを直すタスクです。比較の意図をコードから読み解く必要があるため、新入生に振る場合はレビューを手厚くするか、経験者と一緒に進めてください。
概要
flutter_lints の
unrelated_type_equality_checks違反を解消します。現状 2件 です。このルールについて(何を・なぜ・どう直す)
何を検出するか: 互いに無関係な型どうしを
==で比較している箇所を検出します(例: String と int の比較)。なぜ直すのか: 無関係な型の比較は常に false になります。つまり、ほぼ確実に「変数を取り違えている」「比較する値を間違えている」というバグのサインです。書き方の問題ではなく、ロジックの誤りである可能性が高い箇所です。
どう直すか:
機械的には直せません。なぜこの比較になっているのか、どちらの型が正しいのかをコードを追って特定し、本来意図した比較に直してください。直したら必ず該当機能の動作を確認します。
対象箇所
直し方
このルールは自動修正の対象外です。上の方針で1箇所ずつ手作業で直します。修正後、当該ルールが消えたか確認します。
進め方
この issue から branch を切り、PR を作成してください。issue→branch→PR の順を省略しないのがこのチームの正規フローです。
難易度
要注意・バグ修正。 単なる整形ではなく、ロジックの誤りを直すタスクです。比較の意図をコードから読み解く必要があるため、新入生に振る場合はレビューを手厚くするか、経験者と一緒に進めてください。