参照型のためのEquals（）の "最良の"正規の実装は何ですか？

Question

リファレンスタイプのEquals（）の実装は、それよりも難しくなります。私の現在の標準的な実装では、このように書きます：

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

私はこれまでかなりの包括的なガイドを書いていました。まず、equalsの実装を共有する必要があります（オブジェクトを取るオーバーロードは、厳密に型指定されたオブジェクトを取るオブジェクトに渡す必要があります）。さらに、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

継承に関しては、私はOOパラダイムにその魔法をさせるべきだと思います。

特に、GetType()のチェックは削除する必要があります。これにより、多形性が破られる可能性があります。

Answer 4

私はchakritに同意します。異種のオブジェクトは、同じデータまたはIDを持つ場合、意味的に等価にする必要があります。それはまた、参照型かどうthis._dataがnullでないことを

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

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

Answer 5

良いの希望：

は個人的に、私は次のように使用します。

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