क्या मुझे 'ईवेंट' लॉक करना चाहिए?

Question

मुझे निम्नलिखित मामले में ईवेंट लॉक करना चाहिए:

ईवेंट foo;

थ्रेड ए: foo + = हैंडलर को कॉल करेगा;

धागा बी: foo - = हैंडलर कॉल करेगा;

क्या मुझे foo लॉक करना चाहिए?

Answer 1

foo पर लॉक करना एक बुरा विचार है, क्योंकि मूल्य हर बार बदल जाएगा।

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

ध्यान दें कि अगर आप एक क्षेत्र की तरह घटना, इस तरह का उपयोग करें:: तो

public event EventHandler Foo; 

आप स्वचालित रूप से एक प्राप्त "करेंगे आप एक चर जो नहीं परिवर्तन पर ताला चाहिए लॉक (यह) "जोड़ने/हटाने पर, हालांकि इसे कॉल करने से पहले हैंडलर लाने पर आपको इसे मैन्युअल रूप से जोड़ना होगा (मान लें कि आप यह सुनिश्चित करना चाहते हैं कि आपने हाल ही में लिखित मूल्य पढ़ा है)। व्यक्तिगत रूप से मैं "इस" पर लॉक करने का प्रशंसक नहीं हूं, लेकिन आपको कोई फर्क नहीं पड़ता - और यह निश्चित रूप से सरल कोड बनाता है।