Compartir a través de

Patrones de constructor seguros para DependencyObjects

Hay un principio general en la programación de código administrado, a menudo aplicado por las herramientas de análisis de código, que los constructores de clase no deben llamar a métodos invalidables. Si un constructor de clase base llama a un método reemplazable y una clase derivada invalida ese método, el método de invalidación de la clase derivada se puede ejecutar antes del constructor de clase derivada. Si el constructor de clase derivada realiza la inicialización de clases, el método de clase derivada podría tener acceso a miembros de clase no inicializados. Las clases de propiedades de dependencia deben evitar establecer valores de propiedad de dependencia en un constructor de clase para evitar problemas de inicialización en tiempo de ejecución. En este artículo se describe cómo implementar DependencyObject constructores de una manera que evite esos problemas.

Métodos virtuales y devoluciones de llamada del sistema de propiedades

Los métodos virtuales y las devoluciones de llamada de propiedades de dependencia son parte del sistema de propiedades de Windows Presentation Foundation (WPF) y amplían la versatilidad de las propiedades de dependencia.

Una operación básica, como establecer un valor de propiedad de dependencia mediante SetValue, invocará el evento OnPropertyChanged y potencialmente varias devoluciones de llamada del sistema de propiedades WPF.

OnPropertyChanged es un ejemplo de un método virtual del sistema de propiedades de WPF que pueden ser sobreescritos por las clases que tienen DependencyObject en su jerarquía de herencia. Si establece un valor de una propiedad de dependencia dentro de un constructor que se llama durante la instanciación de su clase de propiedad de dependencia personalizada, y si una clase derivada de ella invalida el método virtual OnPropertyChanged, entonces el método OnPropertyChanged de la clase derivada se ejecutará antes que cualquier constructor de dicha clase derivada.

PropertyChangedCallback y CoerceValueCallback son ejemplos de devoluciones de llamada del sistema de propiedades de WPF que se pueden registrar mediante clases de propiedad de dependencia y invalidadas por clases que derivan de ellas. Si establece un valor de propiedad de dependencia en el constructor de su clase personalizada de propiedad de dependencia, y una clase que deriva de ella sobrescribe una de esas funciones de retorno en los metadatos de propiedad, entonces la función de retorno de la clase derivada se ejecutará antes que cualquier constructor de la clase derivada. Este problema no es relevante para ValidateValueCallback , ya que no forma parte de los metadatos de propiedad y solo se puede especificar mediante la clase de registro.

Para obtener más información sobre las devoluciones de llamada de propiedades de dependencia, consulte Devoluciones de llamada de propiedades de dependencia y validación.

Analizadores de .NET

Los analizadores de la plataforma del compilador de .NET inspeccionan el código de C# o Visual Basic para detectar problemas de calidad y estilo de código. Si llama a métodos sobrescribibles en un constructor cuando la regla de analizador CA2214 está activa, obtendrá la advertencia CA2214: Don't call overridable methods in constructors. Sin embargo, la regla no marcará métodos virtuales ni llamadas de retorno que el subyacente sistema de propiedades WPF invoque cuando se establece un valor de propiedad de dependencia en el constructor.

Problemas causados por clases derivadas

Si sellas tu clase de propiedad de dependencia personalizada, o sabes que de otro modo tu clase no será derivada, entonces los problemas de inicialización en tiempo de ejecución de las clases derivadas no se aplican a esa clase. Sin embargo, si crea una clase de propiedad de dependencia que se puede heredar, por ejemplo, si va a crear plantillas o un conjunto de bibliotecas de controles expandibles, evite llamar a métodos reemplazables o establecer valores de propiedad de dependencia desde un constructor.

El siguiente código de prueba demuestra un patrón de constructor inseguro, donde un constructor de clase base asigna un valor a una propiedad de dependencia, lo que desencadena llamadas a métodos virtuales y callbacks.

    private static void TestUnsafeConstructorPattern()
    {
        //Aquarium aquarium = new();
        //Debug.WriteLine($"Aquarium temperature (C): {aquarium.TempCelcius}");

        // Instantiate and set tropical aquarium temperature.
        TropicalAquarium tropicalAquarium = new(tempCelcius: 25);
        Debug.WriteLine($"Tropical aquarium temperature (C): " +
            $"{tropicalAquarium.TempCelcius}");

        /* Test output:
        Derived class static constructor running.
        Base class ValidateValueCallback running.
        Base class ValidateValueCallback running.
        Base class ValidateValueCallback running.
        Base class parameterless constructor running.
        Base class ValidateValueCallback running.
        Derived class CoerceValueCallback running.
        Derived class CoerceValueCallback: null reference exception.
        Derived class OnPropertyChanged event running.
        Derived class OnPropertyChanged event: null reference exception.
        Derived class PropertyChangedCallback running.
        Derived class PropertyChangedCallback: null reference exception.
        Aquarium temperature (C): 20
        Derived class parameterless constructor running.
        Derived class parameter constructor running.
        Base class ValidateValueCallback running.
        Derived class CoerceValueCallback running.
        Derived class OnPropertyChanged event running.
        Derived class PropertyChangedCallback running.
        Tropical aquarium temperature (C): 25
        */
    }
}

public class Aquarium : DependencyObject
{
    // Register a dependency property with the specified property name,
    // property type, owner type, property metadata with default value,
    // and validate-value callback.
    public static readonly DependencyProperty TempCelciusProperty =
        DependencyProperty.Register(
            name: "TempCelcius",
            propertyType: typeof(int),
            ownerType: typeof(Aquarium),
            typeMetadata: new PropertyMetadata(defaultValue: 0),
            validateValueCallback: 
                new ValidateValueCallback(ValidateValueCallback));

    // Parameterless constructor.
    public Aquarium()
    {
        Debug.WriteLine("Base class parameterless constructor running.");

        // Set typical aquarium temperature.
        TempCelcius = 20;

        Debug.WriteLine($"Aquarium temperature (C): {TempCelcius}");
    }

    // Declare public read-write accessors.
    public int TempCelcius
    {
        get => (int)GetValue(TempCelciusProperty);
        set => SetValue(TempCelciusProperty, value);
    }

    // Validate-value callback.
    public static bool ValidateValueCallback(object value)
    {
        Debug.WriteLine("Base class ValidateValueCallback running.");
        double val = (int)value;
        return val >= 0;
    }
}

public class TropicalAquarium : Aquarium
{
    // Class field.
    private static List<int> s_temperatureLog;

    // Static constructor.
    static TropicalAquarium()
    {
        Debug.WriteLine("Derived class static constructor running.");

        // Create a new metadata instance with callbacks specified.
        PropertyMetadata newPropertyMetadata = new(
            defaultValue: 0,
            propertyChangedCallback: new PropertyChangedCallback(PropertyChangedCallback),
            coerceValueCallback: new CoerceValueCallback(CoerceValueCallback));

        // Call OverrideMetadata on the dependency property identifier.
        TempCelciusProperty.OverrideMetadata(
            forType: typeof(TropicalAquarium),
            typeMetadata: newPropertyMetadata);
    }

    // Parameterless constructor.
    public TropicalAquarium()
    {
        Debug.WriteLine("Derived class parameterless constructor running.");
        s_temperatureLog = new List<int>();
    }

    // Parameter constructor.
    public TropicalAquarium(int tempCelcius) : this()
    {
        Debug.WriteLine("Derived class parameter constructor running.");
        TempCelcius = tempCelcius;
        s_temperatureLog.Add(tempCelcius);
    }

    // Property-changed callback.
    private static void PropertyChangedCallback(DependencyObject depObj, 
        DependencyPropertyChangedEventArgs e)
    {
        Debug.WriteLine("Derived class PropertyChangedCallback running.");
        try
        {
            s_temperatureLog.Add((int)e.NewValue);
        }
        catch (NullReferenceException)
        {
            Debug.WriteLine("Derived class PropertyChangedCallback: null reference exception.");
        }
    }

    // Coerce-value callback.
    private static object CoerceValueCallback(DependencyObject depObj, object value)
    {
        Debug.WriteLine("Derived class CoerceValueCallback running.");
        try
        {
            s_temperatureLog.Add((int)value);
        }
        catch (NullReferenceException)
        {
            Debug.WriteLine("Derived class CoerceValueCallback: null reference exception.");
        }
        return value;
    }

    // OnPropertyChanged event.
    protected override void OnPropertyChanged(DependencyPropertyChangedEventArgs e)
    {
        Debug.WriteLine("Derived class OnPropertyChanged event running.");
        try
        {
            s_temperatureLog.Add((int)e.NewValue);
        }
        catch (NullReferenceException)
        {
            Debug.WriteLine("Derived class OnPropertyChanged event: null reference exception.");
        }

        // Mandatory call to base implementation.
        base.OnPropertyChanged(e);
    }
}

    Private Shared Sub TestUnsafeConstructorPattern()
        'Aquarium aquarium = new Aquarium();
        'Debug.WriteLine($"Aquarium temperature (C): {aquarium.TempCelcius}");

        ' Instantiate And set tropical aquarium temperature.
        Dim tropicalAquarium As New TropicalAquarium(tempCelc:=25)
        Debug.WriteLine($"Tropical aquarium temperature (C): 
            {tropicalAquarium.TempCelcius}")

        ' Test output:
        ' Derived class static constructor running.
        ' Base class ValidateValueCallback running.
        ' Base class ValidateValueCallback running.
        ' Base class ValidateValueCallback running.
        ' Base class parameterless constructor running.
        ' Base class ValidateValueCallback running.
        ' Derived class CoerceValueCallback running.
        ' Derived class CoerceValueCallback: null reference exception.
        ' Derived class OnPropertyChanged event running.
        ' Derived class OnPropertyChanged event: null reference exception.
        ' Derived class PropertyChangedCallback running.
        ' Derived class PropertyChangedCallback: null reference exception.
        ' Aquarium temperature(C):  20
        ' Derived class parameterless constructor running.
        ' Derived class parameter constructor running.
        ' Base class ValidateValueCallback running.
        ' Derived class CoerceValueCallback running.
        ' Derived class OnPropertyChanged event running.
        ' Derived class PropertyChangedCallback running.
        ' Tropical Aquarium temperature (C): 25

    End Sub
End Class

Public Class Aquarium
    Inherits DependencyObject

    'Register a dependency property with the specified property name,
    ' property type, owner type, property metadata with default value,
    ' and validate-value callback.
    Public Shared ReadOnly TempCelciusProperty As DependencyProperty =
        DependencyProperty.Register(
        name:="TempCelcius",
        propertyType:=GetType(Integer),
        ownerType:=GetType(Aquarium),
        typeMetadata:=New PropertyMetadata(defaultValue:=0),
        validateValueCallback:=
            New ValidateValueCallback(AddressOf ValidateValueCallback))

    ' Parameterless constructor.
    Public Sub New()
        Debug.WriteLine("Base class parameterless constructor running.")

        ' Set typical aquarium temperature.
        TempCelcius = 20

        Debug.WriteLine($"Aquarium temperature (C): {TempCelcius}")
    End Sub

    ' Declare public read-write accessors.
    Public Property TempCelcius As Integer
        Get
            Return GetValue(TempCelciusProperty)
        End Get
        Set(value As Integer)
            SetValue(TempCelciusProperty, value)
        End Set
    End Property

    ' Validate-value callback.
    Public Shared Function ValidateValueCallback(value As Object) As Boolean
        Debug.WriteLine("Base class ValidateValueCallback running.")
        Dim val As Double = CInt(value)
        Return val >= 0
    End Function

End Class

Public Class TropicalAquarium
    Inherits Aquarium

    ' Class field.
    Private Shared s_temperatureLog As List(Of Integer)

    ' Static constructor.
    Shared Sub New()
        Debug.WriteLine("Derived class static constructor running.")

        ' Create a new metadata instance with callbacks specified.
        Dim newPropertyMetadata As New PropertyMetadata(
                defaultValue:=0,
                propertyChangedCallback:=
                    New PropertyChangedCallback(AddressOf PropertyChangedCallback),
                coerceValueCallback:=
                    New CoerceValueCallback(AddressOf CoerceValueCallback))

        ' Call OverrideMetadata on the dependency property identifier.
        TempCelciusProperty.OverrideMetadata(
                forType:=GetType(TropicalAquarium),
                typeMetadata:=newPropertyMetadata)
    End Sub

    ' Parameterless constructor.
    Public Sub New()
        Debug.WriteLine("Derived class parameterless constructor running.")
        s_temperatureLog = New List(Of Integer)()
    End Sub

    ' Parameter constructor.
    Public Sub New(tempCelc As Integer)
        Me.New()
        Debug.WriteLine("Derived class parameter constructor running.")
        TempCelcius = tempCelc
        s_temperatureLog.Add(TempCelcius)
    End Sub

    ' Property-changed callback.
    Private Shared Sub PropertyChangedCallback(depObj As DependencyObject,
        e As DependencyPropertyChangedEventArgs)
        Debug.WriteLine("Derived class PropertyChangedCallback running.")

        Try
            s_temperatureLog.Add(e.NewValue)
        Catch ex As NullReferenceException
            Debug.WriteLine("Derived class PropertyChangedCallback: null reference exception.")
        End Try
    End Sub

    ' Coerce-value callback.
    Private Shared Function CoerceValueCallback(depObj As DependencyObject, value As Object) As Object
        Debug.WriteLine("Derived class CoerceValueCallback running.")

        Try
            s_temperatureLog.Add(value)
        Catch ex As NullReferenceException
            Debug.WriteLine("Derived class CoerceValueCallback: null reference exception.")
        End Try

        Return value
    End Function

    ' OnPropertyChanged event.
    Protected Overrides Sub OnPropertyChanged(e As DependencyPropertyChangedEventArgs)
        Debug.WriteLine("Derived class OnPropertyChanged event running.")

        Try
            s_temperatureLog.Add(e.NewValue)
        Catch ex As NullReferenceException
            Debug.WriteLine("Derived class OnPropertyChanged event: null reference exception.")
        End Try

        ' Mandatory call to base implementation.
        MyBase.OnPropertyChanged(e)
    End Sub

End Class

El orden en el que se llama a los métodos en la prueba del patrón de constructor inseguro es:

  1. Constructor estático de clase derivada, que invalida los metadatos de propiedad de dependencia de Aquarium para registrar PropertyChangedCallback y CoerceValueCallback.

  2. Constructor de clase base, que establece un nuevo valor de propiedad de dependencia que da como resultado una llamada al SetValue método . La SetValue llamada desencadena devoluciones de llamada y eventos en el orden siguiente:

    1. ValidateValueCallback, que se implementa en la clase base. Esta devolución de llamada no forma parte de los metadatos de la propiedad de dependencia y no se puede implementar en la clase derivada sobrescribiendo los metadatos.

    2. PropertyChangedCallback, que se implementa en la clase derivada reemplazando los metadatos de la propiedad de dependencia. Esta devolución de llamada provoca una excepción de referencia nula cuando llama a un método en el campo de clase sin inicializar s_temperatureLog.

    3. CoerceValueCallback, que se implementa en la clase derivada reemplazando los metadatos de la propiedad de dependencia. Esta devolución de llamada provoca una excepción de referencia nula cuando llama a un método en el campo de clase sin inicializar s_temperatureLog.

    4. OnPropertyChanged event, que se implementa en la clase derivada reemplazando el método virtual. Este evento produce una excepción de referencia nula cuando llama a un método en el campo s_temperatureLogde clase sin inicializar .

  3. Constructor sin parámetros de clase derivada, que inicializa s_temperatureLog.

  4. Constructor de parámetros de clase derivado, que establece un nuevo valor de propiedad de dependencia que da lugar a otra llamada al SetValue método . Puesto que s_temperatureLog ahora se inicializa, las devoluciones de llamada y los eventos se ejecutarán sin causar excepciones de referencia nulas.

Estos problemas de inicialización se pueden evitar mediante el uso de patrones de constructor seguros.

Patrones de constructor seguros

Los problemas de inicialización de clases derivadas que se muestran en el código de prueba se pueden resolver de maneras diferentes, entre las que se incluyen:

  • Evite establecer un valor para una propiedad de dependencia en el constructor de su clase de propiedad de dependencia personalizada si su clase podría utilizarse como clase base. Si necesita inicializar un valor de propiedad de dependencia, considere la posibilidad de establecer el valor necesario como el valor predeterminado en los metadatos de propiedad durante el registro de propiedades de dependencia o al invalidar los metadatos.

  • Inicialice los campos de clase derivadas antes de su uso. Por ejemplo, con cualquiera de estos enfoques:

    • Cree instancias y asigne campos de instancia en una sola instrucción. En el ejemplo anterior, la instrucción List<int> s_temperatureLog = new(); evitaría la asignación tardía.

    • Realice la asignación en el constructor estático de clase derivada, que se ejecuta antes de cualquier constructor de clase base. En el ejemplo anterior, colocar la instrucción s_temperatureLog = new List<int>(); de asignación en el constructor estático de clase derivada evitaría la asignación tardía.

    • Utilice la inicialización diferida y la creación de instancias, que inicializa los objetos en el momento en que son necesarios. En el ejemplo anterior, instanciar y asignar s_temperatureLog usando la inicialización perezosa evitaría la asignación tardía. Para obtener más información, consulte Inicialización diferida.

  • Evite el uso de variables de clase sin inicializar en funciones de retorno y eventos del sistema de propiedades de WPF.

Consulte también