devo bloccare "evento"?

Question

dovrei bloccare l'evento nel seguente caso:

evento foo;

thread A: chiamerà foo + = gestore;

thread B: chiamerà foo - = handler;

devo bloccare?

Answer 1

Blocco su foo è una cattiva idea, perché il valore cambierà ogni volta. Si dovrebbe chiudere su una variabile che non lo fa cambiamento:

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

Si noti che se si utilizza un evento di campo simile, in questo modo:

public event EventHandler Foo; 

allora si otterrà automaticamente un " lock (this) "su aggiungi/rimuovi, anche se dovresti aggiungerlo manualmente durante il recupero del gestore prima di chiamarlo (supponendo che tu voglia assicurarti di leggere il valore scritto più di recente). Personalmente non sono un fan del lock su "this", ma non ti dispiacerà, e certamente rende il codice più semplice.