C#: İş parçacığı karşılaşma olayları

Question

Uygulama, iş parçacığı güvenli midir? Eğer değilse, neyi özlüyorum? volatile anahtar kelimesini bir yerde mi olmalıyım? Ya da OnProcessingCompleted yönteminde bir yerde kilit var mı? Öyleyse nerede?

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

---

Not: soyut bir temel sınıftır çünkü özel olayı ve açık arabirim şeyler var neden nedeni vardır. Ve ondan gelen sınıflar, doğrudan bu olayla ilgili hiçbir şey yapmamalıdır.

Answer 1

bir event olmaya özel ProcessCompleted üyesi için gerek yoktur) = daha açıkça belirtecek şekilde sınıf sarmalayıcı Eklenenler - bu sadece bir alan olabilir: - her zaman sahada düz gider sınıf içinde Yani event şeyler zaten kayboluyor.

Eğer açık bir kilit nesnesi ile gösterdiğin yaklaşım değildir çok parçacığı güvenli bir alan benzeri bir olay olmasından daha (yani public event EventHandler<ProcessCompletedEventArgs> ProcessCompleted; - Tek fark "Bu" kilit olmamasıdır (iyi bir şey hangi - eğer ideal önlemek this üzerinde kilitleme olmalıdır). Eğer çok işleyicisi getirme zaman .. "işleyicisi değişkeni" yaklaşımı doğru biridir, ama yine de side-effects you should be aware of vardır

Answer 2

sen kilitlemek gerekiyor aksi halde en son değere sahip olmayabilirsiniz:

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

Not biz işleyicileri seti ve ardından bir işleyici abonelikten yürütmek için gidiyoruz karar verdik bir yarış durumu engelleyemedi bu gelmez. Hala çağrılacak, çünkü onu içeren çok noktaya yayın temsilcisini handler değişkenine getirdik.

Bu konuda yapabileceğiniz pek bir şey yok, çünkü işleyicinin artık çağrılmaması gerektiğini fark etmesini sağlayın.

Sadece değil olaylar parçacığı güvenli yapmak için denemek için tartışmasız iyidir - belirttiğiniz abonelik gerektiği olayı yükseltecektir dizisindeki sadece değişim.