संदर्भ प्रकारों के लिए बराबर() के "सर्वोत्तम" कैननिकल कार्यान्वयन क्या है?

Question

संदर्भ प्रकारों के लिए समानता() कार्यान्वित करना ऐसा लगता है जितना कठिन लगता है। मेरे वर्तमान विहित कार्यान्वयन इस प्रकार है:

public bool Equals(MyClass obj) 
{ 
    // If both refer to the same reference they are equal. 
    if(ReferenceEquals(obj, this)) 
    return true; 

    // If the other object is null they are not equal because in C# this cannot be null. 
    if(ReferenceEquals(obj, null)) 
    return false; 

    // Compare data to evaluate equality  
    return _data.Equals(obj._data); 
} 

public override bool Equals(object obj) 
{ 
    // If both refer to the same reference they are equal. 
    if(ReferenceEquals(obj, this)) 
    return true; 

    // If the other object is null or is of a different types the objects are not equal. 
    if(ReferenceEquals(obj, null) || obj.GetType() != GetType()) 
    return false; 

    // Use type-safe equality comparison 
    return Equals((MyClass)obj); 
} 

public override int GetHashCode() 
{ 
    // Use data's hash code as our hashcode 
    return _data.GetHashCode(); 
} 

मुझे लगता है कि यह सब कोने (विरासत और इस तरह के) के मामलों को शामिल किया गया लेकिन मैं गलत हो सकता है। आप लोग क्या सोचते हैं?

Answer 1

मैंने कुछ समय पहले इस बारे में काफी व्यापक मार्गदर्शिका लिखी थी। शुरुआत के लिए आपके बराबर कार्यान्वयन को साझा किया जाना चाहिए (यानी एक ऑब्जेक्ट लेना अधिभार एक दृढ़ता से टाइप की गई वस्तु को लेना चाहिए)। इसके अतिरिक्त आपको GetHashCode को ओवरराइड करने की आवश्यकता के कारण आपकी वस्तु जैसे चीजों को अपरिवर्तनीय होना चाहिए। यहाँ और जानकारी:

http://gregbeech.com/blog/implementing-object-equality-in-dotnet

Answer 2

यह इस बात पर निर्भर करता है कि आप एक मूल्य प्रकार या संदर्भ प्रकार लिख रहे हैं या नहीं। एक क्रमबद्ध मान प्रकार के लिए, मैं इसकी अनुशंसा करता हूं: A code snippet for Visual Studio 2005 that implements a skeleton value type adhering to Framework Design Guidelines

Answer 3

विरासत के बारे में, मुझे लगता है कि आपको केवल ओओ प्रतिमान को जादू करना चाहिए।

विशेष रूप से, GetType() चेक हटाया जाना चाहिए, यह लाइन के नीचे polymorphism तोड़ सकता है।

Answer 4

मैं चक्र के साथ सहमत हूं, विभिन्न प्रकार की वस्तुओं को समान रूप से बराबर होने की अनुमति दी जानी चाहिए यदि उनके पास एक ही डेटा या आईडी है।

public override bool Equals(object obj) 
    { 
     var other = obj as MyClass; 
     if (other == null) return false; 

     return this.data.Equals(other.data); 
    } 

Answer 5

बेहतर आशा व्यक्त की कि this._data अशक्त अगर यह भी एक संदर्भ प्रकार है नहीं है:

व्यक्तिगत रूप से, मैं निम्नलिखित का उपयोग करें।

public bool Equals(MyClass obj) 
{ 
    if (obj == null) { 
     return false; 
    } 
    else { 
     return (this._data != null && this._data.Equals(obj._data)) 
         || obj._data == null; 
    } 
} 

public override bool Equals(object obj) 
{ 
    if (obj == null || !(obj is MyClass)) { 
     return false; 
    } 
    else { 
     return this.Equals((MyClass)obj); 
    } 
} 

public override int GetHashCode() { 
    return this._data == null ? 0 : this._data.GetHashCode(); 
}