C#: потокобезопасные события

Question

Является ли реализация ниже потокобезопасной? Если нет, чего я не хватает? Должен ли я иметь ключевые слова volatile? Или замок где-нибудь в методе OnProcessingCompleted? Если да, то где?

public abstract class ProcessBase : IProcess 
{ 
    private readonly object completedEventLock = new object(); 

    private event EventHandler<ProcessCompletedEventArgs> ProcessCompleted; 

    event EventHandler<ProcessCompletedEventArgs> IProcess.ProcessCompleted 
    { 
     add 
     { 
      lock (completedEventLock) 
       ProcessCompleted += value; 
     } 
     remove 
     { 
      lock (completedEventLock) 
       ProcessCompleted -= value; 
     } 
    } 

    protected void OnProcessingCompleted(ProcessCompletedEventArgs e) 
    { 
     EventHandler<ProcessCompletedEventArgs> handler = ProcessCompleted; 
     if (handler != null) 
      handler(this, e); 
    } 
} 

---

Примечание: Причины, почему у меня есть частный случай и явный интерфейс вещь, потому что это абстрактный базовый класс. И классы, которые наследуют от него, не должны делать ничего с этим событием напрямую. Добавлен класс обертку, так что это более понятно =)

Answer 1

Там нет необходимости для частного ProcessCompleted члена быть event - это может быть просто поле: - внутри класса, она всегда идет прямо на поле , так что все равно теряется материал event.

подход вы показали с явной блокировки объекта не более потокобезопасной, чем просто полевую подобные события гораздо (т.е. public event EventHandler<ProcessCompletedEventArgs> ProcessCompleted; - единственное различие заключается в том, что вы не блокировки «это» (что это хорошая вещь - вы должны идеально избежать блокировки на this). .. «переменной обработчик» подход является правильным, но есть еще side-effects you should be aware of

Answer 2

вы должны зафиксировать, когда вы запрашиваете обработчик слишком , в противном случае вы не можете иметь последнее значение:

protected void OnProcessingCompleted(ProcessCompletedEventArgs e) 
{ 
    EventHandler<ProcessCompletedEventArgs> handler; 
    lock (completedEventLock) 
    { 
     handler = ProcessCompleted; 
    } 
    if (handler != null) 
     handler(this, e); 
} 

Обратите внимание, что это не предотвращает состояние гонки, когда мы решили, что мы выполним набор обработчиков и , а затем один обработчик отменен. Он все равно будет вызван, потому что мы получили делегат многоадресной передачи, содержащий его, в переменную handler.

Существует не так много, что вы можете сделать по этому поводу, кроме того, что сам хендлер знал, что его больше не следует вызывать.

Это, возможно, лучше просто не попробовать сделать событие потокобезопасным - указать, что подписка должна только изменение в потоке, который поднимет событие.