CA2214: Nevolejte přepisovatelné metody v konstruktorech

Článek
09/06/2023

Vlastnost	Hodnota
ID pravidla	CA2214
Název	Nevolejte přepisovatelné metody v konstruktorech
Kategorie	Využití
Oprava způsobující chybu nebo chybu způsobující chybu	Nenarušující
Povoleno ve výchozím nastavení v .NET 9	No

Příčina

Konstruktor nezapečetěného typu volá virtuální metodu definovanou ve své třídě.

Popis pravidla

Pokud je volána virtuální metoda, skutečný typ, který spustí metodu, není vybrán do doby běhu. Když konstruktor volá virtuální metodu, je možné, že konstruktor pro instanci, která vyvolá metodu, se nespustí. To může vést k chybám nebo neočekávanému chování, pokud přepsaná virtuální metoda spoléhá na inicializaci a jinou konfiguraci v konstruktoru.

Jak opravit porušení

Chcete-li opravit porušení tohoto pravidla, nevolejte virtuální metody typu z konstruktorů typu.

Kdy potlačit upozornění

Nepotlačujte upozornění na toto pravidlo. Konstruktor by měl být přepracovaný tak, aby eliminoval volání virtuální metody.

Příklad

Následující příklad ukazuje účinek porušení tohoto pravidla. Testovací aplikace vytvoří instanci DerivedType, která způsobí spuštění jeho základní třídy (BadlyConstructedType) konstruktoru. BadlyConstructedTypekonstruktor nesprávně volá virtuální metodu DoSomething. Jak ukazuje výstup, DerivedType.DoSomething() provede se před DerivedTypespuštěním konstruktoru.

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 Main2214()
    {
        DerivedType derivedInstance = new DerivedType();
    }
}


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

Tento příklad vytvoří následující výstup:

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

Sdílet prostřednictvím