C#


CSHARP.STRUCT.UUMETH : Unused Method (C#)

要旨

A method or constructor is not called.

Methods or constructors never used in the code are often harmless, except for an increase in size of the application. In some cases, however, unused methods are the sign of a stale application interface that needs cleaning or of the necessity to introduce an abstract method.

プロパティ

クラス名 Unused Method (C#)
日本語クラス名 Unused Method (C#)
クラス分類 信頼性 (reliability)
ニーモニック CSHARP.STRUCT.UUMETH
カテゴリー
CWE CWE:561 Dead Code
対応言語 C# で利用可能です。
有効/無効設定 このワーニングクラスのチェックはデフォルトで有効になっています。チェックを無効にするにはプロジェクト設定ファイル (configuration file)に以下の WARNING_FILTER ルールを追加してください。
WARNING_FILTER += discard class="Unused Method (C#)"

In the following code, the implementation of method m inside class A is never called: only its redefinitions inside classes B and C are called.

using System;

namespace DocumentationExamples {

  public class Deadcode {

    public static void Main(string[] args) {
      DeadcodeA b = new DeadcodeB();
      DeadcodeA c = new DeadcodeC();
      Test(b);
      Test(c);
    }

    private static void Test(DeadcodeA a) {
      Console.WriteLine(a.M(13));
    }
  }

  public class DeadcodeA {
    public virtual int M(int p) { // "Unused Method (C#)" warning issued here
                                  // if CSHARP_ANALYSIS_ENTRY_POINTS_MODE is set to ONLY_EXPLICIT_ENTRIES or ONLY_STANDARD_ENTRIES
      return p + 1;
    }
  }

  public class DeadcodeB : DeadcodeA {
    public override int M(int p) {
      return p + 2;
    }
  }

  public class DeadcodeC : DeadcodeA {
    public override int M(int p) {
      return p * 2;
    }
  }
}

In this example, the programmer should probably introduce an abstract definition for M inside class DeadcodeA and make the class DeadcodeA itself abstract, as follows:

public abstract class DeadcodeA
{
  public abstract int M(int p);
}

解決法

Verify if the application interface can be simplified or an abstract method could replace a non-abstract method.

関連のある設定ファイルパラメータ

設定ファイルの以下のパラメータがこのワーニングクラスのチェックに影響します。