我應該鎖定'事件'嗎？

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」的粉絲，但你可能不介意 - 它確實使代碼更簡單。