コード検査: 重複した記述

このインスペクションは、if/else、switch ブロックなどの条件付きブランチブロック内の繰り返し文を検出します。これらの構造におけるロジックの繰り返しは、コードの可読性と保守性に悪影響を及ぼす可能性があります。重複文を削除すると、コードが簡素化され、エラーが発生しにくくなり、変更への適応が容易になります。

以下の例では、両方の分岐フローで同じ文が実行されます。if 文内の重複文を削除してもセマンティクスは変わりませんが、コードの明瞭性と可読性は大幅に向上します。

            
public string CheckResult(bool result)
{
    if (result)
    {
        Console.WriteLine("Checking result");
        return "OK";
    }
    Console.WriteLine("Checking result");
    return "OK";
}

        

            
public string CheckResult(bool result)
{
    if (result)
    {
    }
    Console.WriteLine("Checking result");
    return "OK";
}

        

2025 年 3 月 24 日

関連ページ：

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

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

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

このインスペクションは、同一の本体を持つ連続する文を検出します。このような冗長性は、コードの可読性と保守性に悪影響を及ぼします。これは、該当する条件を統合することでロジックを簡素化できる可能性があることを示しています。以下の例では、2 つの連続するステートメントが同じ本体を共有しています。ReSharper では、繰り返しを減らすために、これらの条件を 1 つのステートメントにまとめることが推奨されています。public string ValidateInput(int input) { if...