स्थिर चर को enums के साथ प्रतिस्थापित किया जाना चाहिए?

Question

तो मैं कुछ कोड देख रहा था जिसमें चेक किया गया था और मुझे सभी परेशान हो गए:

// Amount of days before cancellation can't be done 
enum Cancellation { Limit = 2 }; 

जिस व्यक्ति ने इसे चेक किया है उससे पूछते हुए उसने तर्क दिया कि स्थिर चर के बजाए enums का उपयोग करना बेहतर है, इससे बेहतर :

private static int CANCELLATION_LIMIT = 2; 

तो हमने बहस शुरू कर दी। मेरा तर्क था कि वह मूल्यों को स्टोर करने के तरीके के रूप में enum का उपयोग कर रहा था (यदि एक ही मूल्य के साथ दो enum प्रतीकों थे तो यह तोड़ देगा)। उन्होंने तर्क दिया कि यह एक वर्ग में स्थैतिक चर के लिए एक antipattern था।

मेरा प्रश्न यह है कि या तो सर्वोत्तम अभ्यास का उपयोग किया जाना चाहिए?

Answer 1

Enums लिखे जाते हैं (मैं तार्किक संगत के एक बड़े प्रशंसक हूँ)।

यह है कि, यदि आपके पास कोई तरीका है जहां आपको उदाहरण के लिए किसी विधि को 'निश्चित' पास करना है, तो आप केवल 'मान्य' तर्क ही पारित कर सकते हैं। उदाहरण के लिए:

enum OrderState 
{ 
    pending = 1, 
    shipped = 2 
} 

public IList<Order> GetOrdersInState(OrderState) 
{ 
} 

यह enums का उपयोग करने का -imho- एक अच्छा उदाहरण है। जब ऑर्डरस्टेट एक int है जिसके लिए आप 2 कॉन्स इन्ट्स बनाते हैं, तो आपके पास कोई प्रतिबंध नहीं है और वे अमान्य मानों को पार करने में सक्षम हैं। संकलक शिकायत नहीं करेगा।

हालांकि, जो मामला आप ला रहे हैं, मुझे लगता है कि enums का उपयोग एक वैध समाधान नहीं है। यह एक int का उपयोग करने का दुरुपयोग है, और एक कॉन्स int का उपयोग किया जाना चाहिए।

Enums अच्छे हैं, लेकिन इन्हें इस्तेमाल किया जाना चाहिए जहां उनका उपयोग किया जाना चाहिए। वे हर स्थिति में पसंदीदा उपकरण नहीं हैं। इस मामले में एक कॉन्स या स्थिर var होने के कारण एक एंटीपाटर नहीं है।

Answer 2

मुझे लगता है कि यदि आपके पास सीधे जुड़े हुए मानों का एक सेट है तो आपको enums का उपयोग करना चाहिए।

कुछ ऐसा है: enum status {open = 1, बंद = 2, प्रतीक्षा = 3};

अन्य सभी के लिए, मैं कहूंगा कि स्थैतिक चर जाने का रास्ता है।

Answer 3

मुझे नहीं पता कि यह एक वर्ग (?) में स्थिर चर रखने के लिए एक विरोधी पैटर्न है। उदाहरण के लिए, नेट फ्रेमवर्क में रंग वर्ग में कई स्थिर सार्वजनिक चर हैं, जैसे Color.Red। तो, उस परिप्रेक्ष्य से, मैं आपसे सहमत होगा।

हालांकि, एक समझौता हो सकता है: निजी कॉन्स CANCELLATION_LIMIT = 2 का उपयोग करें; और आप दोनों को खुश होना चाहिए। उनके लिए, कक्षा (?) के लिए कोई वैश्विक चर नहीं होगा, क्योंकि कॉन्स को कंपाइलर द्वारा प्रतिस्थापित किया जाएगा, और आपको एक स्पष्ट नाम के साथ परिवर्तन का एक बिंदु मिलेगा।

Answer 4

वापसी "क्या यह तर्कसंगत रूप से मूल्यों का एक सेट है"? "Enum उचित है": "स्थैतिक स्थिरांक ठीक है"

Answer 5

नहीं, आप enum में स्थिर स्ट्रिंग चर या दशमलव मान कैसे परिभाषित करते हैं?

Answer 6

मुझे नहीं लगता कि CANELLATION_LIMIT एक enum की तरह लगता है, जो आमतौर पर विकल्पों का एक सेट है।

कुछ अलग के लिए, यदि यह था एक const, तो शायद ... लेकिन वर्तमान में यह एक अस्थायी क्षेत्र है?

ध्यान दें कि enums तक ही सीमित हैं पूर्णांक आधारित प्रकार, तो यह, आदि

Answer 7

अद्वितीय होने के लिए लक्षित अपरिवर्तनीय मूल्यों के लिए float, string के लिए इस्तेमाल नहीं किया जा सकता है, enumerations जाने का रास्ता है। पूछने का सवाल सरल है: क्या ऑब्जेक्ट को वैल्यू स्टोर करना चाहिए, यहां तक ​​कि स्थैतिक रूप से भी? कई मामलों में, जैसे त्रुटियों या कार्रवाई का वर्णन करते समय, जवाब नहीं है। याद रखें, enum रों लिए #define एक स्थानापन्न के रूप में पैदा हुआ था: यह पहचान का ठेठ मूल्यों जुड़े हुए हैं और यह एक प्रकार प्रदान करता है, यह वास्तव में नहीं कहना है "स्टोर इस निरंतर यहाँ"।

मुझे लगता है आप नहीं है वास्तव में कुछ भी स्टोर, लेकिन इस तरह ठेठ मान प्रदान करना चाहते हैं। स्थिर कॉन्स सदस्य केवल तब उपयोगी होते हैं जब आप उनका उपयोग करना चाहते हैं, उदाहरण के लिए यदि आपको किसी विधि के संदर्भ में उन्हें पास करने की आवश्यकता है।