Skip to content

[Flutter lint] unrelated_type_equality_checks を解消 (2件・要注意(バグ)) #313

@taminororo

Description

@taminororo

概要

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 の順を省略しないのがこのチームの正規フローです。

難易度

要注意・バグ修正。 単なる整形ではなく、ロジックの誤りを直すタスクです。比較の意図をコードから読み解く必要があるため、新入生に振る場合はレビューを手厚くするか、経験者と一緒に進めてください。

Metadata

Metadata

Assignees

No one assigned

    Labels

    Size-XS開発時間の目安は3時間以下✨Frontend-MobileMobileのフロントのタスク. 主にDart/Flutterを使用🔨改修改修。バグ修正とはちょっと違うけど完全に新規作成でもないやつとか

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions