केवल पढ़ने योग्य संपत्ति को लागू करने के लिए

Question

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

मैं दो तरीकों से यह करने के लिए देखें:

1. class MyClass 
   { 
       public readonly object MyProperty = new object(); 
   } 
   

2. class MyClass 
   { 
       private readonly object my_property = new object(); 
       public object MyProperty { get { return my_property; } } 
   } 
   

इन सब FxCop त्रुटियों कह रही है कि मैं सार्वजनिक सदस्य चर नहीं करना चाहिए था, ऐसा लगता है कि एक दूसरे के ऐसा करने का सही तरीका है। सही बात?

क्या केवल एक संपत्ति प्राप्त करने और इस मामले में केवल पढ़ने वाले सदस्य के बीच कोई अंतर है?

मैं किसी भी टिप्पणी/सलाह/आदि की सराहना करता हूं।

Answer 1

संशोधन:
मुझे लगता है कि यह बहुत फर्क नहीं पड़ता कि आप एक मात्र स्रोत अनुकूलता में रुचि रखते हैं।
एक संपत्ति का उपयोग बाइनरी संगतता के लिए बेहतर है क्योंकि आप इसे उस संपत्ति द्वारा प्रतिस्थापित कर सकते हैं जिसमें आपकी लाइब्रेरी के आधार पर संकलित कोड को तोड़ने के बिना एक सेटटर है।

कन्वेंशन:
आप सम्मेलन का पालन कर रहे हैं। इस तरह के मामलों में जहां सम्मेलन बेहतर होने के बाद दोनों संभावनाओं के बीच मतभेद अपेक्षाकृत मामूली हैं। एक मामला जहां यह काटने के लिए वापस आ सकता है आप प्रतिबिंब आधारित कोड है।यह केवल गुण स्वीकार कर सकता है न कि फ़ील्ड, उदाहरण के लिए एक संपत्ति संपादक/दर्शक।

क्रमबद्धता
संपत्ति के क्षेत्र से बदल रहा है शायद serializers का एक बहुत टूट जाएगा। और AFAIK XmlSerializer केवल सार्वजनिक गुणों को क्रमबद्ध करता है, न कि सार्वजनिक फ़ील्ड।

एक Autoproperty
एक अन्य आम परिवर्तन का उपयोग एक निजी सेटर के साथ एक autoproperty उपयोग कर रहा है। हालांकि यह छोटा है और एक संपत्ति यह पठनीयता को लागू नहीं करती है। तो मैं दूसरे को पसंद करता हूं।

केवल पढ़ने के लिए क्षेत्र
selfdocumenting है, हालांकि क्षेत्र में से एक फायदा है:
यह सार्वजनिक इंटरफ़ेस है कि यह वास्तव अपरिवर्तनीय है (छोड़कर प्रतिबिंब) पर एक नज़र में स्पष्ट करता है। जबकि किसी संपत्ति के मामले में आप केवल देख सकते हैं कि आप इसे बदल नहीं सकते हैं, इसलिए आपको दस्तावेज़ या कार्यान्वयन का संदर्भ लेना होगा।

लेकिन ईमानदार होने के लिए मैं आलसी हूं क्योंकि मैं अक्सर आवेदन कोड में पहले व्यक्ति का उपयोग करता हूं। पुस्तकालयों में मैं आम तौर पर अधिक गहन हूं और सम्मेलन का पालन करता हूं।

सी # 6.0 केवल पढ़ने के लिए स्वत: गुण

public object MyProperty { get; } 

कहते हैं तो जब आप पुराने compilers आपको लगता है कि बस के रूप में एक केवल पढ़ने के लिए क्षेत्र के रूप में संक्षिप्त है कोड के साथ वास्तव में केवल पढ़ने के लिए संपत्ति हो सकती है समर्थन करने के लिए की जरूरत नहीं है।

Answer 2

दूसरा तरीका पसंदीदा विकल्प है।

private readonly int MyVal = 5; 

public int MyProp { get { return MyVal;} } 

यह सुनिश्चित होगा कि MyVal केवल (यह भी एक निर्माता में सेट किया जा सकता) प्रारंभ में सौंपा जा सकता है।

जैसा कि आपने नोट किया था - इस तरह आप एक आंतरिक सदस्य का पर्दाफाश नहीं कर रहे हैं, जिससे आप भविष्य में आंतरिक कार्यान्वयन को बदल सकते हैं।

Answer 3

दूसरी विधि को encapsulation की वजह से प्राथमिकता दी जाती है। आप निश्चित रूप से पठनीय क्षेत्र सार्वजनिक हो सकते हैं, लेकिन यह सी # मुहावरे के खिलाफ चला जाता है जिसमें आपके पास डेटा एक्सेस होता है और गुणों के माध्यम से नहीं होता है।

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

Answer 4

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

Answer 5

आप ऐसा कर सकते हैं:

public int Property { get { ... } private set { ... } } 

Answer 6

सी # 6 की शुरूआत के साथ

(वी.एस. 2015 में), अब आप get -only स्वत: गुण है, जो में निहित समर्थन क्षेत्र readonly (यानी मूल्यों निर्माता में सौंपा जा सकता है लेकिन और कहीं नहीं) है हो सकता है:

public string Name { get; } 

public Customer(string name) // Constructor 
{ 
    Name = name; 
} 

private void SomeFunction() 
{ 
    Name = "Something Else"; // Compile-time error 
} 

और तुम अब भी गुण (के साथ या बिना एक सेटर) इनलाइन आरंभ कर सकते हैं:

public string Name { get; } = "Boris"; 

वापस सवाल की चर्चा करते हुए यह आप विकल्प 2 के फायदे देता है (सार्वजनिक सदस्य एक संपत्ति नहीं है, एक क्षेत्र) ब्रेवटी के साथ विकल्प 1.

दुर्भाग्यवश, यह सार्वजनिक इंटरफ़ेस के स्तर पर अपरिवर्तनीयता की गारंटी प्रदान नहीं करता है (जैसे कि कोड के उपभोक्ता को, क्लास के उपभोक्ता के लिए, कोई सेटटर नहीं है एक निजी सेटर होने से अलग करने योग्य।