क्या बंद होने के माध्यम से किसी ईवेंट से स्वयं को सदस्यता समाप्त करने का निम्न पैटर्न कोई समस्या पैदा करेगा?

Question

उस स्थिति पर विचार करें जिसमें आप एक और केवल एक अधिसूचना के लिए किसी ईवेंट की सदस्यता लेना चाहते हैं। एक बार पहली अधिसूचना भूमि हो जाने के बाद, आप सभी भावी घटनाओं से सदस्यता छोड़ लेते हैं। क्या निम्न पैटर्न किसी स्मृति समस्या को पेश करेगा? यह काम करता है, लेकिन मुझे यकीन नहीं था कि आत्म-संदर्भ बंद करने से चीजों को वांछित से अधिक स्मृति में चारों ओर रख सकता है।

public class Entity 
{ 
    public event EventHandler NotifyEvent; 
} 

// And then, elsewhere, for a listen-once handler, we might do this: 
Entity entity = new Entity(); 
Action<object, EventArgs> listener = null; 
listener = (sender, args) => 
{ 
    // do something interesting 
    // unsubscribe, so we only get 1 event notification 
    entity.NotifyEvent -= new EventHandler(listener); 
}; 
entity.NotifyEvent += new EventHandler(listener); 

नोट आप 'श्रोता' घोषित करने और एक मूल्य (शून्य) आवंटित करने के लिए है। अन्यथा संकलक 'असाइन किए गए स्थानीय चर का उपयोग listener'

Answer 1

इस पैटर्न के साथ कुछ भी गलत नहीं है। यह एक ही पैटर्न है I और कई अन्य एक ईवेंट हैंडलर को लैम्ब्डा अभिव्यक्ति को असाइन करने और हटाने के लिए उपयोग करते हैं।

Answer 2

जबकि मुझे लगता है कि सामान्य पैटर्न ठीक है, तो मैं Action<object, EventArgs> से नहीं जाऊंगा। मैं उपयोग करूंगा:

EventHandler listener = null; 
listener = (sender, args) => 
{ 
    // do something interesting 
    // unsubscribe, so we only get 1 event notification 
    entity.NotifyEvent -= listener; 
}; 
entity.NotifyEvent += listener;