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

Artigo
06/08/2015

TypeName	DoNotCallOverridableMethodsInConstructors
CheckId	CA2214
Categoria	Microsoft.Usage
Alteração Significativa	Sem Quebra

Causa

O construtor de um tipo não lacrado 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 está selecionado até o tempo de execução.Quando um construtor chama um método virtual, é possível que o construtor para a instância do que invoca o método não foi executada.

Como Corrigir Violações

Para corrigir uma violação desta regra, não chamar os métodos virtuais de um tipo de nos construtores do tipo.

Quando Suprimir Alertas

Não elimine um alerta desta regra.O construtor deve ser reprojetado para eliminar a chamada para o método virtual.

Exemplo

O exemplo a seguir demonstra o efeito de violar esta regra.O aplicativo de teste a seguir cria uma instância de DerivedType, que faz com que o construtor de classe baseBadlyConstructedType() é executado.o construtor deBadlyConstructedType incorretamente chama o método virtual DoSomething.Como a saída mostra, DerivedType.DoSomething() , executa e faz isso antes do construtor de DerivedType seja executado.

Imports System

Namespace UsageLibrary

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

using System;

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

O exemplo produz a seguinte saída.

Partilhar via