コード検査: 重複した連続 'if' ブランチ

このインスペクションは、同一の本体を持つ連続する if 文を検出します。このような冗長性は、コードの可読性と保守性に悪影響を及ぼします。これは、該当する条件を統合することでロジックを簡素化できる可能性があることを示しています。

以下の例では、2 つの連続する if ステートメントが同じ本体を共有しています。ReSharper では、繰り返しを減らすために、これらの条件を 1 つのステートメントにまとめることが推奨されています。

            
    public string ValidateInput(int input)
    {
        if (input == 10 || input == 20)
            return Validate(input);
        if (input == 30 || input == 40)
            return Validate(input);

        return "invalid input";
    }

    string Validate(int i) => "valid input";

        

            
    public string ValidateInput(int input)
    {
        if (input == 10 || input == 20 || input == 30 || input == 40)
            return Validate(input);

        return "invalid input";
    }

    string Validate(int i) => "valid input";

2025 年 3 月 24 日

関連ページ：

コードインスペクション設定を構成する

設計時コードインスペクションを構成する：デフォルトでは、ReSharper は、サポートされている言語に対応するすべてのファイルで設計時コードインスペクションを有効にします。必要に応じて、無効にすることができます。設計時コードインスペクションが有効かどうかに関係なく、コードインスペクションは常に特定のスコープで実行できます。メインメニューからを選択するか、Alt+R O を押してから、左側のを選択します。設計時コードインスペクションを切り替えるには、コード分析を有効にするチェックボックスを使用し...

コード検査: 重複した連鎖 'if' ブランチ

このインスペクションは、複数の連続する文が同一の本体を持つケースを識別します。このようなコードは冗長性を高め、メンテナンスを困難にし、エラーの発生リスクを高めます。条件を 1 つの文に統合することで、コードが簡素化され、可読性が向上します。以下の例では、連鎖されたステートメントの本体は同一です。ReSharper では、明瞭性を高め、重複を避けるため、条件を 1 つのステートメントに統合することを推奨しています。public void CheckStatus(int status) { if (...