CA2214 : N'appelez pas de méthodes substituables dans les constructeurs

Cause

Le constructeur d'un type non-sealed (non scellé) appelle une méthode virtuelle définie dans sa classe.

Description de la règle

Lorsqu'une méthode virtuelle est appelée, le type réel qui exécute la méthode n'est pas sélectionné avant le moment de l'exécution. Lorsqu'un constructeur appelle une méthode virtuelle, il est possible que le constructeur de l'instance qui appelle la méthode n'ait pas été exécuté.

Comment corriger les violations

Pour corriger une violation de cette règle, n'appelez pas les méthodes virtuelles d'un type à partir des constructeurs de ce type.

Quand supprimer les avertissements

Ne supprimez aucun avertissement de cette règle. Le design du constructeur doit être refondu pour éliminer l'appel à la méthode virtuelle.

Exemple

L'exemple suivant montre l'effet de la violation de cette règle. L'application de test crée une instance de DerivedType, qui entraîne l'exécution du constructeur de sa classe de base (BadlyConstructedType). Le constructeur de BadlyConstructedType appelle la méthode virtuelle DoSomething de manière incorrecte. Alors que la sortie s'affiche, DerivedType.DoSomething() s'exécute, et ce avant l'exécution du constructeur de DerivedType.

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();
        }
    }
}

Cet exemple génère la sortie suivante :