CA2214: Nie należy wywoływać nadpisywalnych metod w konstruktorach

Właściwości	Wartość
Identyfikator reguły	CA2214
Tytuł	Nie wywołuj w konstruktorach metod, które można przesłaniać
Kategoria	Użycie
Poprawka łamiąca lub nienaruszająca	Niezgodność
Domyślnie włączone na platformie .NET 10	Nie.
Zastosowane języki	C# i Visual Basic

Przyczyna

Konstruktor niezaziemowanego typu wywołuje metodę wirtualną zdefiniowaną w swojej klasie.

Opis reguły

Po wywołaniu metody wirtualnej rzeczywisty typ, który wykonuje metodę, nie jest wybierany do czasu wykonania. Kiedy konstruktor wywołuje metodę wirtualną, istnieje możliwość, że konstruktor dla instancji, która wywołuje tę metodę, nie został jeszcze wykonany. Może to prowadzić do błędów lub nieoczekiwanego zachowania, jeśli zastępowana metoda wirtualna opiera się na inicjowaniu i innej konfiguracji konstruktora.

Jak naprawić naruszenia

Aby naprawić naruszenie tej reguły, nie należy wywoływać wirtualnych metod danego typu wewnątrz jego konstruktorów.

Kiedy pomijać ostrzeżenia

Nie pomijaj ostrzeżeń dla tej reguły. Konstruktor powinien zostać przeprojektowany, aby wyeliminować wywołanie metody wirtualnej.

Przykład

W poniższym przykładzie pokazano efekt naruszenia tej reguły. Aplikacja testowa tworzy wystąpienie DerivedType, co powoduje wykonanie konstruktora klasy bazowej BadlyConstructedType. BadlyConstructedType konstruktor niepoprawnie wywołuje wirtualną metodę DoSomething. Jak pokazano w danych wyjściowych, DerivedType.DoSomething() wykonuje się przed wykonaniem konstruktora DerivedType.

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 ? {initialized}");
    }
}

public class TestBadlyConstructedType
{
    public static void Main2214()
    {
        DerivedType derivedInstance = new();
    }
}

Imports System

Namespace ca2214

    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 Main2214()
            Dim derivedInstance As New DerivedType()
        End Sub 'Main
    End Class
End Namespace

Ten przykład generuje następujące wyniki:

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