Question

我应该在以下情况下锁定事件：

event foo;

线程A：将调用foo + = handler;

线程B：将调用foo - = handler;

我应该锁定foo吗？

Answer 1

锁定foo是一个坏主意，因为每次都会更改该值。您应该锁定不更改的变量：

private readonly object eventLock = new object();
private EventHandler fooHandler;

public event EventHandler Foo
{
    add
    {
        lock (eventLock)
        {
            fooHandler += value;
        }
    }
    remove
    {
        lock (eventLock)
        {
            fooHandler -= value;
        }
    }
}

private void OnFoo(EventArgs e)
{
    EventHandler handler;
    lock (eventLock)
    {
        handler = fooHandler;
    }
    if (handler != null)
    {
        handler(this, e);
    }
}

请注意，如果您使用类似字段的事件，请执行以下操作：

public event EventHandler Foo;

然后你会在添加/删除时自动获得一个“锁定（这个）”，虽然在调用它之前你必须手动添加它（假设你想确保你阅读最近写的值）。就个人而言，我不是一个锁定“这个”的粉丝，但你可能不介意 - 它肯定会使代码更简单。

我应该锁定'事件'吗？

1 个答案: