CA2214: Não chamar métodos substituíveis em construtores

Artigo
11/14/2023

Propriedade	valor
ID da regra	CA2214
Cargo	Não chame métodos substituíveis em construtores
Categoria	Utilização
A correção está quebrando ou não quebrando	Sem quebra
Habilitado por padrão no .NET 8	Não

Causa

O construtor de um tipo sem lacre chama um método virtual definido em sua classe.

Descrição da regra

Quando um método virtual é chamado, o tipo real que executa o método não é selecionado até o tempo de execução. Quando um construtor chama um método virtual, é possível que o construtor para a instância que invoca o método não tenha sido executado. Isso pode levar a erros ou comportamento inesperado, se um método virtual substituído depende de inicialização e outra configuração no construtor.

Como corrigir violações

Para corrigir uma violação dessa regra, não chame os métodos virtuais de um tipo de dentro dos construtores do tipo.

Quando suprimir avisos

Não suprima um aviso desta regra. O construtor deve ser redesenhado para eliminar a chamada para o método virtual.

Exemplo

O exemplo a seguir demonstra o efeito de violar essa regra. O aplicativo de teste cria uma instância de , que faz com que seu construtor de DerivedTypeclasse base (BadlyConstructedType) seja executado. BadlyConstructedType's construtor incorretamente chama o método DoSomethingvirtual . Como mostra a saída, DerivedType.DoSomething() executa antes DerivedTypede executar o construtor do .

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 exemplo produz a seguinte saída:

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

Partilhar via