CA2214:コンストラクターのオーバーライド可能なメソッドを呼び出しませんCA2214: Do not call overridable methods in constructors

TypeNameTypeName DoNotCallOverridableMethodsInConstructorsDoNotCallOverridableMethodsInConstructors
CheckIdCheckId CA2214CA2214
カテゴリCategory Microsoft.UsageMicrosoft.Usage
互換性に影響する変更点Breaking Change 中断なしNon Breaking

原因Cause

封印されていない型のコンス トラクターは、そのクラスで定義されている仮想メソッドを呼び出します。The constructor of an unsealed type calls a virtual method defined in its class.

規則の説明Rule description

仮想メソッドが呼び出されると、実行時までは、メソッドを実行する実際の型が選択されていません。When a virtual method is called, the actual type that executes the method is not selected until run time. コンス トラクターは、仮想メソッドを呼び出し、メソッドを呼び出すインスタンスのコンス トラクターが実行しないことになります。When a constructor calls a virtual method, it is possible that the constructor for the instance that invokes the method has not executed.

違反の修正方法How to fix violations

この規則違反を修正するには、型のコンス トラクター内からに型の仮想メソッドを呼び出す操作を行います。To fix a violation of this rule, do not call a type's virtual methods from within the type's constructors.

警告を抑制します。When to suppress warnings

この規則による警告は抑制しないでください。Do not suppress a warning from this rule. コンス トラクターは、仮想メソッド呼び出しを削除して再設計する必要があります。The constructor should be redesigned to eliminate the call to the virtual method.

Example

次の例では、この規則違反の効果を示します。The following example demonstrates the effect of violating this rule. テスト アプリケーションのインスタンスを作成するDerivedType、それが原因で、基底クラス (BadlyConstructedType) コンス トラクターが実行されます。The test application creates an instance of DerivedType, which causes its base class (BadlyConstructedType) constructor to execute. BadlyConstructedTypeコンス トラクターが仮想メソッドを正しく呼び出しますDoSomethingします。BadlyConstructedType's constructor incorrectly calls the virtual method DoSomething. 出力をDerivedType.DoSomething()を実行して、前にDerivedTypeのコンス トラクターを実行します。As the output shows, DerivedType.DoSomething() executes, and does so before DerivedType's constructor executes.

using System;

namespace UsageLibrary
{
    public class BadlyConstructedType
    {
        protected  string initialized = "No";
        
        public BadlyConstructedType()
        {
            Console.WriteLine("Calling base ctor.");
            // Violates rule: DoNotCallOverridableMethodsInConstructors.
            DoSomething();
        }
        // This will be overridden in the derived type.
        public virtual void DoSomething()
        {
            Console.WriteLine ("Base DoSomething");
        }
    }
    
    public class DerivedType : BadlyConstructedType
    {
        public DerivedType ()
        {
            Console.WriteLine("Calling derived ctor.");
            initialized = "Yes";
        }
        public override void DoSomething()
        {
            Console.WriteLine("Derived DoSomething is called - initialized ? {0}", initialized);
        }
    }
 
    public class TestBadlyConstructedType
    {
        public static void Main()
        {
            DerivedType derivedInstance = new DerivedType();
        }
    }
}

Imports System

Namespace UsageLibrary

Public Class BadlyConstructedType
    Protected initialized As String = "No"
    
    
    Public Sub New()
        Console.WriteLine("Calling base ctor.")
        ' Violates rule: DoNotCallOverridableMethodsInConstructors.
        DoSomething()
    End Sub 'New
    
    ' This will be overridden in the derived type.
    Public Overridable Sub DoSomething()
        Console.WriteLine("Base DoSomething")
    End Sub 'DoSomething
End Class 'BadlyConstructedType


Public Class DerivedType
    Inherits BadlyConstructedType
    
    Public Sub New()
        Console.WriteLine("Calling derived ctor.")
        initialized = "Yes"
    End Sub 'New
    
    Public Overrides Sub DoSomething()
        Console.WriteLine("Derived DoSomething is called - initialized ? {0}", initialized)
    End Sub 'DoSomething
End Class 'DerivedType


Public Class TestBadlyConstructedType
    
    Public Shared Sub Main()
        Dim derivedInstance As New DerivedType()
    End Sub 'Main
End Class 
End Namespace

この例を実行すると、次の出力が生成されます。This example produces the following output:

Calling base ctor.
Derived DoSomething is called - initialized ? No
Calling derived ctor.