C#: eventos seguros contra subprocesos

Question

¿La implementación es segura para subprocesos? Si no, ¿qué es lo que me estoy perdiendo? ¿Debo tener las palabras clave volatile en alguna parte? ¿O un bloqueo en algún lugar del método OnProcessingCompleted? ¿Si es así, donde?

public abstract class ProcessBase : IProcess 
{ 
    private readonly object completedEventLock = new object(); 

    private event EventHandler<ProcessCompletedEventArgs> ProcessCompleted; 

    event EventHandler<ProcessCompletedEventArgs> IProcess.ProcessCompleted 
    { 
     add 
     { 
      lock (completedEventLock) 
       ProcessCompleted += value; 
     } 
     remove 
     { 
      lock (completedEventLock) 
       ProcessCompleted -= value; 
     } 
    } 

    protected void OnProcessingCompleted(ProcessCompletedEventArgs e) 
    { 
     EventHandler<ProcessCompletedEventArgs> handler = ProcessCompleted; 
     if (handler != null) 
      handler(this, e); 
    } 
} 

---

Nota: La razón por la que tengo evento privado y cosas por el interfaz explícita, se debe a que es una clase base abstracta. Y las clases que heredan de él no deberían hacer nada directamente con ese evento. Se ha añadido el contenedor de clase para que sea más claro =)

Answer 1

No hay necesidad de que el ProcessCompleted miembro privado para ser un event - que sólo podría ser un campo: - dentro de la clase que siempre va directamente al campo , así que las cosas event se pierden de todos modos.

El enfoque que has demostrado con un objeto de bloqueo explícito no es mucho más seguro para subprocesos que sólo tener un evento de campo como (es decir public event EventHandler<ProcessCompletedEventArgs> ProcessCompleted; - la única diferencia es que no se está fijando "este" (que es una buena cosa - usted debe idealmente evitar el bloqueo en this). .. el enfoque de "variable de manipulador" es la correcta, pero todavía hay side-effects you should be aware of

Answer 2

debe bloquear al recuperar el manejador demasiado De lo contrario, es posible que no tenga el último valor:

protected void OnProcessingCompleted(ProcessCompletedEventArgs e) 
{ 
    EventHandler<ProcessCompletedEventArgs> handler; 
    lock (completedEventLock) 
    { 
     handler = ProcessCompleted; 
    } 
    if (handler != null) 
     handler(this, e); 
} 

Tenga en cuenta que este no prevenir una condición de carrera en la que hemos decidido que vamos a ejecutar un conjunto de controladores y luego un manejador de baja. Todavía se llamará, porque hemos buscado el delegado de multidifusión que lo contiene en la variable handler.

No hay mucho que pueda hacer al respecto, salvo hacer que el manejador sepa que ya no se debe llamar.

Podría decirse que es mejor no intentar para hacer los eventos flujos seguros - especificar que la suscripción debe única cambio en el hilo que va a provocar el evento.