CA2214: Não chamar métodos substituíveis em construtores
Property | Valor |
---|---|
ID da regra | CA2214 |
Título | Não chamar métodos substituíveis em construtores |
Categoria | Usage |
Correção interruptiva ou sem interrupção | Sem interrupção |
Habilitado por padrão no .NET 8 | Não |
Causa
O construtor de um tipo não selado chama um método virtual definido na respectiva 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 da instância que invoca o método não tenha sido executado. Isso pode resultar em erros ou em um comportamento inesperado se um método virtual substituído depende da inicialização e de outras configurações do construtor.
Como corrigir violações
Para corrigir uma violação dessa regra, não chame os métodos virtuais de um tipo nos construtores do tipo.
Quando suprimir avisos
Não suprima um aviso nessa regra. O construtor deve ser reprojetado para eliminar a chamada ao método virtual.
Exemplo
O exemplo a seguir demonstra o efeito da violação dessa regra. O aplicativo de teste cria uma instância de DerivedType
, o que faz com que o construtor da classe base (BadlyConstructedType
) seja executado. O construtor do BadlyConstructedType
chama incorretamente o método virtual DoSomething
. Como a saída mostra, DerivedType.DoSomething()
é executado antes da execução do construtor 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
Esse exemplo gera a saída a seguir:
Calling base ctor.
Derived DoSomething is called - initialized ? No
Calling derived ctor.