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

Article
09/07/2023

Propriété	Value
Identificateur de la règle	CA2214
Titre	N'appelez pas de méthodes substituables dans les constructeurs
Catégorie	Utilisation
Le correctif est cassant ou non cassant	Sans rupture
Activé par défaut dans .NET 8	Non

Cause

Le constructeur d’un type unsealed 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 sélectionné qu’au 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é. Cela peut entraîner des erreurs ou un comportement inattendu si une méthode virtuelle substituée dépend de l’initialisation et d’autres configurations dans le constructeur.

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 du type.

Quand supprimer les avertissements

Ne supprimez aucun avertissement de cette règle. Le constructeur doit être repensé de façon à éliminer l’appel à la méthode virtuelle.

Exemple

L’exemple suivant illustre l’effet d’une violation de cette règle. L’application de test crée une instance de DerivedType, ce qui entraîne l’exécution de son constructeur de classe de base (BadlyConstructedType). Le constructeur de BadlyConstructedType appelle incorrectement la méthode virtuelle DoSomething. Comme le montre la sortie, DerivedType.DoSomething() s’exécute avant le constructeur 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

Cet exemple produit la sortie suivante :

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

Partage via