devrais-je verrouiller «événement»?

Question

dois-je verrouiller l'événement dans le cas suivant:

événement foo;

thread A: appelle foo + = gestionnaire;

thread B: appelle foo - = gestionnaire;

devrais-je verrouiller foo?

Answer 1

Verrouiller foo est une mauvaise idée, car la valeur changera à chaque fois. Vous devez verrouiller sur une variable qui ne le fait pas changement:

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

Notez que si vous utilisez un événement comme champ, comme ceci:

public event EventHandler Foo; 

alors vous serez automatiquement obtenir un " lock (this) "sur ajouter/supprimer, bien que vous deviez l'ajouter manuellement lorsque vous récupérerez le gestionnaire avant de l'appeler (en supposant que vous voulez vous assurer de lire la dernière valeur écrite). Personnellement, je ne suis pas un fan de verrouillage sur "ceci", mais cela ne vous dérange pas - et cela rend certainement le code plus simple.