Referans türleri için Equals() 'ın "en iyi" kanonik uygulaması nedir?

Question

Referans türleri için Equals() uygulaması göründüğünden daha zordur. Benim şu anki kanonik uygulaması şöyledir:

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(); 
} 

Ben bu her köşesini (devralma ve gibi) durumlarda kapsar ama yanlış olabileceğini düşünüyorum. Siz ne düşünüyorsunuz?

Answer 1

Bunu bir süre önce oldukça kapsamlı bir kılavuz yazdım. Bir başlangıç ​​için eşit uygulamalarınız paylaşılmalıdır (yani, bir nesneyi alan aşırı yüklenme, güçlü bir şekilde yazılan nesneye sahip olana geçmelidir). Ek olarak, GetHashCode'u geçersiz kılmaya ihtiyaç duyduğunuz için nesnenizin değişmez olması gibi şeyleri göz önünde bulundurmanız gerekir. Burada daha fazla bilgi:

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

Answer 2

Değer türü veya başvuru tipi yazıp yazmayacağınıza bağlıdır. Sorgulanabilir bir değer türü için şunu öneririm: A code snippet for Visual Studio 2005 that implements a skeleton value type adhering to Framework Design Guidelines

Answer 3

Mirasla ilgili olarak, OO paradigmasının sihrini yapmasına izin vermelisiniz. Özellikle, GetType() denetiminin kaldırılması gerektiğinden, bu, polimorfizmi çizgiye indirebilir.

Answer 4

Chakrit ile aynı fikirdeyim, farklı türdeki nesnelerin aynı veriye veya kimliğe sahip olduklarında semantik olarak eşit olmasına izin verilmelidir. aynı zamanda bir başvuru türü ise this._data boş olmadığını

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

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

Answer 5

Daha İyi umut:

Şahsen ben şu kullanın.

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(); 
}