C#：线程安全事件

Question

下面的实现是线程安全的吗？如果不是，我错过了什么？我应该在某处使用volatile 关键字吗？还是OnProcessingCompleted 方法中某处的锁？如果有，在哪里？

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);
    }
}

---

注意：我之所以有私有事件和显式接口的东西，是因为它是一个抽象基类。从它继承的类不应该直接对该事件做任何事情。添加了类包装器，使其更清晰 =)

Answer 1

你在获取handler的时候也需要加锁，否则你可能没有最新的值：

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

请注意，这不会阻止我们决定执行一组处理程序并且然后取消订阅一个处理程序的竞争条件。它仍会被调用，因为我们已将包含它的多播委托提取到 handler 变量中。

除了让处理程序本身意识到不应再调用它之外，您无能为力。

最好不要尝试使事件线程安全 - 指定订阅应该仅在将引发事件的线程中更改。 p>

Answer 2

私有ProcessCompleted 成员不需要是event - 它可以只是一个字段：private EventHandler<ProcessCompletedEventArgs> ProcessCompleted; - 在类内部它总是直接进入字段，所以event 的东西是还是输了。

您使用显式锁定对象显示的方法更多比仅具有类似字段的事件（即public event EventHandler<ProcessCompletedEventArgs> ProcessCompleted; - 唯一的区别是您是'不锁定“this”（这是一件好事——你应该理想地避免锁定this）..“处理程序变量”方法是正确的，但仍有side-effects you should be aware of .