警告：“...覆盖 Object.Equals(object o) 但不覆盖 Object.GetHashCode()”

Question

我覆盖了我班级的 Equals() 来比较 Guid 类型的 ID 值。

然后 Visual Studio 发出警告：

... 覆盖 Object.Equals(object o) 但 不覆盖 Object.GetHashCode()

然后我也像这样覆盖它的 GetHashCode()：

public partial class SomeClass
{
    public override bool Equals(Object obj)
    {
        //Check for null and compare run-time types.
        if (obj == null || this.GetType() != obj.GetType()) return false;

        return this.Id == ((SomeClass)obj).Id;
    }

    public override int GetHashCode()
    {
        return this.Id.GetHashCode();
    }
}

它似乎工作。 我做对了吗？记住 Id 的类型是 Guid。 我的类是实体框架对象有关系吗？

Answer 1

正如其他人所说，在 Equals 中使用反射似乎很狡猾。撇开这些不谈，让我们专注于 GetHashCode。

您不得违反 GetHashCode 的主要规则是如果两个对象相等，则它们必须具有相同的哈希码。或者，一种等效的说法是如果两个对象具有不同的哈希码，那么它们一定是不相等的。你的实现在那里看起来不错。

您可以随意违反相反的规定。也就是说，如果两个对象具有相同的哈希码，那么它们被允许相等或不相等，只要你认为合适。

我假设“Id”是一个不可变的属性。如果“Id”可以在对象的生命周期内发生变化，那么在将对象放入哈希表时可能会遇到问题。考虑确保在计算相等性和哈希码时仅使用不可变属性。

您的实现看起来不错，但您提出这个问题的事实表明您可能没有牢牢掌握构建 GetHashCode 实现的所有微妙因素。一个很好的起点是我关于这个主题的文章：

http://ericlippert.com/2011/02/28/guidelines-and-rules-for-gethashcode/

Answer 2

在我看来是正确的。每当我做这样的事情时，我通常也会实现IEquatable，这样相同编译时类型的变量之间的比较会更有效率。

public partial class SomeClass : IEquatable<SomeClass>
{
     public override bool Equals(Object obj)
     {
         return Equals(obj as SomeClass);
     }
     public bool Equals(SomeClass obj)
     {
         if (obj == null) 
             return false;
         return Id == obj.Id;
     }
     public override int GetHashCode()
     {
         return Id.GetHashCode();
     }
} 

此结构还允许将具有相同 Id 的更多派生对象与派生程度较低的对象进行比较。如果这不是所需的行为，那么您还必须像在问题中那样比较类型。

if (obj.GetType() != typeof(SomeClass)) return false;

Answer 3

由于您不是在处理密封类，因此我建议您不要像 this.GetType() != obj.GetType() 这样检查类是否相等。 SomeClass 的任何子类也应该能够参与 Equals，因此您可能希望改用它：

if (obj as SomeClass == null) return false;

Answer 4

传统上，Equals 的实现方式是两个对象只有在各方面完全相同时才会“相等”。例如，如果您有两个对象代表数据库中的同一对象，但其中一个对象的Name 属性与另一个对象不同，则这些对象不被视为“相等”，应避免产生相同的“哈希码”如果可能的话。

最好选择“不相等”而不是冒险调用两个不相等的对象。这就是对象的默认实现使用对象本身的内存位置的原因：除非它们是完全相同的对象，否则不会将两个对象视为“相等”。所以我想说，除非你想同时编写 GetHashCode 和 Equals 以检查它们所有属性的相等性，否则你最好不要覆盖任何一种方法。

如果您有一个数据结构（如HashSet），您特别希望根据 ID 值确定相等性，您可以为该数据结构提供特定的IEqualityComparer 实现。

Answer 5

你的第一个问题得到了很好的答案：

我做对了吗？

我会回答你的第二个问题

我的类是实体框架对象有关系吗？

是的，这很重要。实体框架在内部大量使用HashSet。例如，动态代理使用HashSet 表示集合导航属性，EntityObjects 使用EntityCollection，后者又在内部使用HashSet。