CA2214: No llamar a métodos reemplazables en constructores
Nombre de tipo |
DoNotCallOverridableMethodsInConstructors |
Identificador de comprobación |
CA2214 |
Categoría |
Microsoft.Usage |
Cambio problemático |
No |
Motivo
El constructor de un tipo no sellado llama a un método virtual definido en su clase.
Descripción de la regla
Cuando se llama a un método virtual, el tipo real que ejecuta el método no se selecciona hasta el tiempo de ejecución.Cuando un constructor llama a un método virtual, es posible que no se haya ejecutado el constructor para la instancia que invoca el método.
Cómo corregir infracciones
Para corregir una infracción de esta regla, no llame a los métodos virtuales del tipo desde los constructores del tipo.
Cuándo suprimir advertencias
No suprima las advertencias de esta regla.El constructor se debería rediseñar para eliminar la llamada al método virtual.
Ejemplo
El ejemplo siguiente muestra el efecto de la infracción de esta regla.La aplicación de prueba crea una instancia de DerivedType, que hace que se ejecute el constructor de la clase base (BadlyConstructedType).El constructor de BadlyConstructedTypellama incorrectamente al método virtual DoSomething.Tal como muestra el resultado, DerivedType.DoSomething() se ejecuta y así lo hace antes de que se ejecute el constructor 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();
}
}
}
Este ejemplo produce el siguiente resultado.