CA2214: No llamar a métodos reemplazables en constructores

Propiedad	Value
Identificador de la regla	CA2214
Título	No llamar a métodos reemplazables en constructores
Categoría	Uso
La corrección es problemática o no problemática	Poco problemático
Habilitado de forma predeterminada en .NET 8	No

Causa

El constructor de un tipo no sellado llama a un método virtual definido en su clase.

Descripción de la regla

Cuando se llama a un método virtual, el tipo real que ejecuta el método no se selecciona hasta el tiempo de ejecución. Cuando un constructor llama a un método virtual, es posible que no se haya ejecutado el constructor de la instancia que invoca el método. Esto podría provocar errores o un comportamiento inesperado, si un método virtual invalidado se basa en la inicialización y en otra configuración del constructor.

Cómo corregir infracciones

Para corregir una infracción de esta regla, no llame a los métodos virtuales de un tipo desde dentro de los constructores del tipo.

Cuándo suprimir las advertencias

No suprima las advertencias de esta regla. Se debe rediseñar el constructor para eliminar la llamada al método virtual.

Ejemplo

En el ejemplo siguiente se muestra el efecto de infringir esta regla. La aplicación de prueba crea una instancia de DerivedType, que provoca la ejecución del constructor de su clase base (BadlyConstructedType). El constructor de BadlyConstructedType llama al método virtual DoSomething de forma incorrecta. Como muestra el resultado, DerivedType.DoSomething() se ejecuta antes de que se ejecute el constructor de 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 ? {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

Este ejemplo produce el siguiente resultado:

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