powinienem zablokować "wydarzenie"?

Question

należy zablokować zdarzenie w następującym przypadku:

event foo;

wątek A: wywoła foo + = handler;

wątek B: wywołaj foo - = handler;

Czy powinienem zablokować foo?

Answer 1

Zablokowanie na foo jest złym pomysłem, ponieważ wartość zmieni się za każdym razem. należy zablokować na zmienną, która nie zmiany:

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

pamiętać, że jeśli używasz zdarzenia polowych jak, na przykład:

public event EventHandler Foo; 

Pokochasz więc automatycznie uzyskać " lock (this) "przy dodawaniu/usuwaniu, chociaż musisz ręcznie dodać go podczas pobierania handler'a przed wywołaniem go (zakładając, że chcesz się upewnić, że przeczytałeś ostatnio zapisaną wartość). Osobiście nie jestem zwolennikiem blokowania "tego", ale możesz nie mieć nic przeciwko - i to z pewnością tworzy prostszy kod.