Non chiamare metodi sottoponibili a override nei costruttori

Articolo
09/12/2008

Aggiornamento: novembre 2007

TypeName	DoNotCallOverridableMethodsInConstructors
CheckId	CA2214
Category	Microsoft.Usage
Breaking Change	Non sostanziale

Causa

Il costruttore di un tipo non sealed chiama un metodo virtuale definito nella relativa classe.

Descrizione della regola

Quando viene chiamato un metodo virtuale, il tipo effettivo che esegue il metodo non viene selezionato fino alla fase di esecuzione. Quando un costruttore chiama un metodo virtuale, è possibile che il costruttore per l'istanza che richiama il metodo non sia stato eseguito.

Correzione di violazioni

Per correggere una violazione di questa regola, non chiamare i metodi virtuali di un tipo dall'interno dei costruttori del tipo.

Esclusione di avvisi

Non escludere un avviso da questa regola. Il costruttore deve essere riprogettato per eliminare la chiamata al metodo virtuale.

Esempio

Nell'esempio riportato di seguito viene illustrato l'effetto della violazione di questa regola. L'applicazione di test crea un'istanza di DerivedType che causa l'esecuzione del costruttore della relativa classe base (BadlyConstructedType). Il costruttore di BadlyConstructedType chiama erroneamente il metodo virtuale DoSomething. Come illustrato nell'output, viene eseguito DerivedType.DoSomething() prima dell'esecuzione del costruttore di 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();
        }
    }
}

Questo esempio produce l'output che segue.

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

Condividi tramite