Udostępnij za pośrednictwem


CA2214: Nie należy wywoływać nadpisywalnych metod w konstruktorach

TypeName

DoNotCallOverridableMethodsInConstructors

CheckId

CA2214

Kategoria

Microsoft.Usage

Zmiana kluczowa

Niekluczowa

Przyczyna

Konstruktor o niezapieczętowanym typie, wywołuje metodę wirtualną zdefiniowaną w jej klasie.

Opis reguły

Gdy wywoływana jest metoda wirtualna, rzeczywisty typ, który wykonuje metodę nie jest zaznaczony, aż do czasu wykonywania.Kiedy konstruktor wywołuje metodę wirtualną, jest możliwe, że konstruktor wystąpienia, które wywołuje metodę, nie został wykonany.

Jak naprawić naruszenia

Aby naprawić naruszenie tej zasady, nie należy wywoływać metod wirtualnych typu wewnątrz konstruktora typu.

Kiedy pominąć ostrzeżenia

Nie pomijaj ostrzeżeń dla tej reguły.Konstruktor powinien zostać przeprojektowany aby wyeliminować wywołanie metody wirtualnej.

Przykład

Poniższy przykład demonstruje efekt naruszenia tej zasady.Aplikacja testowa tworzy wystąpienie DerivedType, co powoduje wykonanie konstruktora jego klasy bazowej (BadlyConstructedType).Konstruktor BadlyConstructedType niepoprawnie wywołuje metodę wirtualną DoSomething.Jak pokazują dane wyjściowe, wykonuje się DerivedType.DoSomething() zanim zostanie wywołany konstruktor 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();
        }
    }
}

Ten przykład generuje następujące wyniki.