CA1034: नेस्टेड प्रकार दिखाई

Question

Here's नियम यह है कि है कि मैं समझने की कोशिश कर रहा हूँ की एक विवरण नहीं होना चाहिए। यहाँ सरल कोड है कि कोड विश्लेषक के बारे में शिकायत की गई थी:

Public Class CustomerSpeed 

    Public Enum ProfitTypeEnum As Integer 
     NotSpecified = 0 
     FlatAmount = 1 
     PercentOfProfit = 2 
    End Enum 

    Private _ProfitTypeEnum As ProfitTypeEnum 

    Public Sub New(ByVal profitType As ProfitTypeEnum) 

     _ProfitTypeEnum = profitType 

    End Sub 

End Class 

enum केवल वर्ग से संबंधित है, तो क्यों यह एक बुरी बात यह वर्ग के भीतर एक निहित प्रकार बनाने के लिए है? मेरे लिए neater लगता है ...

किसी को भी पता है क्या निम्न पंक्ति ?:

नेस्टेड प्रकार सदस्य पहुँच की धारणा है, जो कुछ प्रोग्रामर स्पष्ट रूप से

समझ में नहीं आता का उपयोग करना शामिल मतलब है समूह के लिए नेमस्पेस क्लास और enum एक उपयोगी तरीका की तरह प्रतीत नहीं होता है इस चेतावनी को हल करने, के बाद से मैं दोनों enum वर्ग के नाम के रूप में एक ही मूल स्तर के हैं करना चाहते हैं।

Answer 1

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

Answer 2

मैं इसे इंगित करता है कि नेस्टेड प्रकार स्थिर चर के साथ भ्रमित किया जा सकता है, क्योंकि वे दोनों के बाद IntelliSense में (स्वत: पूर्ण) दिखाई देगा। "" लगता है।

भले ही आप .NET के बीसीएल (बेस क्लास लाइब्रेरी) को देखते हैं, सभी enums, भले ही वे केवल एक वर्ग में उपयोग किए जाते हैं, वे घोंसला नहीं हैं, कारण है, जब आप तुलना करना चाहते हैं या तत्काल, कक्षा नाम + "।" + enum नाम नीचे दिखाए गए भ्रमित होंगे, माइक्रोसॉफ्ट भी सही है कि यह लोगों को भ्रमित करेगा।

- उदाहरण

class RootClass 
{ 
    enum NestedEnum 
    { 
     StaticItem = 0  
    } 

    static string NestedString = ""; 
} 


RootClass.NestedEnum <-- represents enum, 
RootClass.NestedString <-- represents static variable 

एक एक प्रकार है और अन्य चर रहा है, सबसे एमएस कोड विश्लेषक चेतावनी, बेहतर डिजाइन के लिए कर रहे हैं लेकिन यह @George के लिए एक बड़ा अपवाद है, यदि आप इसे पसंद नहीं, न इसका इस्तेमाल करें, बस आगे बढ़ें और चेतावनियों को अक्षम करें। आप नामस्थानों के बजाय निश्चित रूप से बड़े वर्ग के नामों का उपयोग कर सकते हैं, इसकी बस आपकी पसंद है। लेकिन अच्छे प्रोग्रामिंग कौशल इस बारे में है कि दूसरों को आपके कोड को कैसे प्राप्त करें बल्कि आपको क्या पसंद है !!!

और नामस्थान वहाँ व्यवस्थित करने के लिए कर रहे हैं और हम कम टाइप का आयोजन करके, thats क्या सभी के बारे में कम प्रयासों के साथ अधिक काम कर रही। लेकिन अगर आपको टाइप करने के लिए बड़े नाम पसंद हैं, तो कोई भी आपको रोक नहीं रहा है।

Answer 3

Enumerations आमतौर पर निर्दिष्ट जहां गणन है होने के लिए, वर्ग उन्हें का उपयोग करता है में नहीं रखा जाता है, ताकि लोगों का इस्तेमाल नहीं कर रहे हैं:

Dim speed As New CustomerSpeed(CustomerSpeed.ProfitTypeEnum.FlatAmount) 

गणन रखने कक्षा के बाहर यह आसान उपयोग करने के लिए बनाता है:

Dim speed As New CustomerSpeed(ProfitTypeEnum.FlatAmount) 

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