सी # एक ईवेंट हैंडलर को रोकने के लिए पैटर्न के दो बार

Question

डुप्लिकेट झुका: How to ensure an event is only subscribed to once और Has an event handler already been added?

मैं एक सिंगलटन कुछ सेवा और मेरी कक्षाओं उस पर कुछ घटनाओं में हुक, कभी कभी एक वर्ग घटना के लिए दो बार hooking है प्रदान करता है और फिर दो बार बुलाया जाता है। मैं इसे होने से रोकने के लिए एक शास्त्रीय तरीके की तलाश में हूं। किसी भी तरह से मुझे यह जांचने की ज़रूरत है कि क्या मैंने पहले से ही इस घटना से जुड़ा हुआ है ...

Answer 1

घटना को स्पष्ट रूप से कार्यान्वित करें और आमंत्रण सूची की जांच करें। तुम भी अशक्त के लिए जाँच करने की आवश्यकता होगी:, एक फोन करने वाले घटना कई बार की सदस्यता लेता है, तो

using System.Linq; // Required for the .Contains call below: 

... 

private EventHandler foo; 
public event EventHandler Foo 
{ 
    add 
    { 
     if (foo == null || !foo.GetInvocationList().Contains(value)) 
     { 
      foo += value; 
     } 
    } 
    remove 
    { 
     foo -= value; 
    } 
} 

उपरोक्त कोड का उपयोग करना, यह केवल नजरअंदाज कर दिया जाएगा।

Answer 2

आपको ईवेंट पर एक्सेसर्स को जोड़ने और निकालने की आवश्यकता है, और उसके बाद प्रतिनिधि की लक्षित सूची देखें, या लक्ष्य को स्टोर करें सूची।

ऐड विधि में, आप प्रतिनिधि को पहले से जोड़े गए लक्ष्यों की एक सूची प्राप्त करने के लिए Delegate.GetInvocationList विधि का उपयोग कर सकते हैं।

चूंकि प्रतिनिधियों को समान लक्ष्य की तुलना करने के लिए परिभाषित किया जाता है, यदि वे एक ही लक्ष्य वस्तु से उसी विधि से जुड़े होते हैं, तो आप शायद उस सूची के माध्यम से भाग सकते हैं और तुलना कर सकते हैं, और यदि आपको कोई भी तुलना करने वाला कोई नहीं मिलता है, तो आप नया जोड़ते हैं ।

यहाँ नमूना कोड है, सांत्वना आवेदन के रूप में संकलन:

using System; 
using System.Linq; 

namespace DemoApp 
{ 
    public class TestClass 
    { 
     private EventHandler _Test; 

     public event EventHandler Test 
     { 
      add 
      { 
       if (_Test == null || !_Test.GetInvocationList().Contains(value)) 
        _Test += value; 
      } 

      remove 
      { 
       _Test -= value; 
      } 
     } 

     public void OnTest() 
     { 
      if (_Test != null) 
       _Test(this, EventArgs.Empty); 
     } 
    } 

    class Program 
    { 
     static void Main() 
     { 
      TestClass tc = new TestClass(); 
      tc.Test += tc_Test; 
      tc.Test += tc_Test; 
      tc.OnTest(); 
      Console.In.ReadLine(); 
     } 

     static void tc_Test(object sender, EventArgs e) 
     { 
      Console.Out.WriteLine("tc_Test called"); 
     } 
    } 
} 

आउटपुट:

tc_Test called 

(यानी केवल एक बार।)

Answer 3

अपने सिंगलटन वस्तु की जांच यह यह जो सूचित करता है की सूची दी गई है है और डुप्लिकेट होने पर केवल एक बार कॉल करें। वैकल्पिक रूप से यदि संभव हो तो घटना अनुलग्नक अनुरोध को अस्वीकार कर दें।

Answer 4

हो सकता है कि इसी तरह के एक पद के लिए मेरा उत्तर में मदद मिलेगी:

Differentiating between events raised by user interaction and my own code

Answer 5

आप वास्तव में सिंक स्तर और नहीं स्रोत स्तर पर इस संभाल चाहिए। यही है, ईवेंट स्रोत पर ईवेंट हैंडलर तर्क निर्धारित न करें - इसे हैंडलर (सिंक) पर छोड़ दें।

सेवा के डेवलपर के रूप में, आप कौन कहेंगे कि सिंक केवल एक बार पंजीकरण कर सकते हैं? क्या होगा यदि वे किसी कारण से दो बार पंजीकरण करना चाहते हैं? और यदि आप स्रोत को संशोधित करके सिंक में बग को सही करने का प्रयास कर रहे हैं, तो यह सिंक-स्तर पर इन समस्याओं को ठीक करने का एक अच्छा कारण है।

मुझे यकीन है कि आपके पास आपके कारण हैं; एक घटना स्रोत जिसके लिए डुप्लिकेट सिंक अवैध हैं, वह अतुलनीय नहीं है। लेकिन शायद आपको एक वैकल्पिक वास्तुकला पर विचार करना चाहिए जो किसी घटना के अर्थशास्त्र को बरकरार रखे।

Answer 6

के बारे में सिर्फ घटना, -= के साथ पहली बार को हटाने अगर यह नहीं पाया जाता है एक अपवाद फेंका नहीं है

/// -= Removes the event if it has been already added, this prevents multiple firing of the event 
((System.Windows.Forms.WebBrowser)sender).Document.Click -= new System.Windows.Forms.HtmlElementEventHandler(testii); 
((System.Windows.Forms.WebBrowser)sender).Document.Click += new System.Windows.Forms.HtmlElementEventHandler(testii); 

Answer 7

माइक्रोसॉफ्ट के Reactive Extensions (Rx) framework भी ऐसा करने के लिए "केवल एक बार सदस्यता ले" इस्तेमाल किया जा सकता कैसे।

एक माउस घटना foo.Clicked को देखते हुए यहां सदस्यता लें और केवल एक ही मंगलाचरण प्राप्त करने के लिए बताया गया है:

Observable.FromEvent<MouseEventArgs>(foo, "Clicked") 
    .Take(1) 
    .Subscribe(MyHandler); 

... 

private void MyHandler(IEvent<MouseEventArgs> eventInfo) 
{ 
    // This will be called just once! 
    var sender = eventInfo.Sender; 
    var args = eventInfo.EventArgs; 
} 

"एक बार सदस्यता ले" कार्यक्षमता प्रदान करने के अलावा, RX दृष्टिकोण घटनाओं को एक साथ रचना करने की क्षमता प्रदान करता है या घटनाओं को फ़िल्टर करें। यह काफी निफ्टी है।

Answer 8

चांदी की रोशनी में आपको ई। हैंडल = सत्य कहना होगा; घटना कोड में।

void image_MouseLeftButtonDown(object sender, MouseButtonEventArgs e) 
{ 
    e.Handled = true; //this fixes the double event fire problem. 
    string name = (e.OriginalSource as Image).Tag.ToString(); 
    DoSomething(name); 
} 

अगर यह मदद करता है तो कृपया मुझे टिकटें।

Answer 9

किसी ईवेंट की बजाय कोई कार्रवाई बनाएं।

public class MyClass 
{ 
       // sender arguments  <-----  Use this action instead of an event 
    public Action<object, EventArgs> OnSomeEventOccured; 

    public void SomeMethod() 
    { 
      if(OnSomeEventOccured!=null) 
       OnSomeEventOccured(this, null); 
    } 

} 

Answer 10

मैं प्रत्येक समाधान और सबसे अच्छा एक (पर विचार प्रदर्शन) परीक्षण किया है है:: अपने वर्ग की तरह लग सकता है

private EventHandler _foo; 
public event EventHandler Foo { 

    add { 
     _foo -= value; 
     _foo += value; 
    } 
    remove { 
     _foo -= value; 
    } 
} 

नहीं Linq आवश्यक का उपयोग कर। सदस्यता रद्द करने से पहले शून्य की जांच करने की आवश्यकता नहीं है (विवरण के लिए एमएस इवेंट हैंडलर देखें)। हर जगह सदस्यता रद्द करने की याद रखने की जरूरत नहीं है।