क्या यह किसी अन्य घटना के भीतर से किसी ईवेंट को बढ़ाने के लिए सुगंधित है?

Question

public class Basket 
{ 
    private int _unitCount; 

    public int UnitCount 
    { 
     get { return _unitCount; } 
     set 
     { 
      _unitCount = Math.Max(0, value); 
      OnUnitCountChanged(new EventArgs()); 
     } 
    } 

    public event EventHandler UnitCountChanged; 
    public event EventHandler Depleted; 

    protected virtual void OnUnitCountChanged(EventArgs args) 
    { 
     var handler = UnitCountChanged; 
     if(handler!=null) { handler(this, args); } 
     if(_unitCount == 0) { OnDepleted(new EventArgs()); } 
    } 

    protected virtual void OnDepleted(EventArgs args) 
    { 
     var handler = UnitCountChanged; 
     if(handler!=null) { handler(this, args); } 
    } 
} 

वहाँ समाप्त हो गया के लिए शर्तों की जाँच और UnitCountChanged घटना के भीतर यदि आवश्यक हो तो है कि घटना को ऊपर उठाने के साथ एक समस्या है, या मैं दोनों UnitCount सेटर में करना चाहिए (और कहीं और एक गैर तुच्छ उदाहरण में)?

Answer 1

जबकि मैंने इसे देखा है, तो मैं इसके खिलाफ अनुशंसा करता हूं और आपके UnitCount सेटर की तरह होने वाली विधियों में ईवेंट को बढ़ाऊंगा। चूंकि आपके पास virtual एक्सेस संशोधक कीवर्ड है, इसलिए कोई व्यक्ति विधि को ओवरराइड कर सकता है और यदि वे मूल ऑब्जेक्ट को कॉल नहीं करते हैं तो यह अपेक्षित काम नहीं करेगा।

मैं अपने कोड का उपयोग करने के लिए इसे और अधिक जटिल बनाने का प्रशंसक नहीं हूं।

Answer 2

ऐसे कई बार होते हैं जब यह उपयोगी हो सकता है (उदाहरण के लिए, यदि आप बेस क्लास का विस्तार कर रहे हैं और घटनाओं को बढ़ाने वाले तरीकों को ओवरराइड करने की क्षमता नहीं है), लेकिन सामान्य रूप से, मैं इसके खिलाफ अनुशंसा करता हूं ।

इस मामले में, मैं इसे UnitCount सेटर में दोनों घटनाओं को बढ़ाने के लिए बेहतर है कहेंगे:

public int UnitCount 
{ 
    get { return _unitCount; } 
    set 
    { 
     _unitCount = value; 
     OnUnitCountChanged(new EventArgs()); 
     if(_unitCount == 0) { OnDepleted(new EventArgs()); } 
    } 
}