我应该锁定'事件'吗？

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; 

那么你会自动获得一个“在添加/删除时锁定（this）“，尽管您在调用处理程序之前必须手动添加它（假设您要确保读取最近写入的值）。就我个人而言，我不是锁定“this”的粉丝，但你可能不介意 - 它确实使代码更简单。