继承相等比较器

Question

我有一个客户类。

public class Customer
{
    private string _id;
    private string _name;
    // some more properties follow 

我继承了 EqualityComparer 表单 MyEqualityComparer(of Customer)。
我打算在 LINQ 查询中使用它。
MyEqualityComparer 用于两个对象之间的部分检查。
如果 customer.id 和 customer.name 匹配，我将对象视为平等。

public class MyComparer : System.Collections.Generic.EqualityComparer<Customer>
{
    public override bool Equals(Customer x, Customer y)
    {
        if (x.Id == y.Id && x.Name == y.Name)
            return true;
        else
            return false;
    }

    public override int GetHashCode(Customer obj)
    {
        return string.Concat(obj.Id,obj.Name).GetHashCode();
    }

}

我提到了generating hashcode。
我不太确定连接字符串并将其用作哈希码。
这是我想要做的安全和健全的事情吗？

Answer 1

请参阅this question on hashcodes，了解基于多个字段返回一个哈希码的非常简单的方法。

话虽如此，我自己不会从EqualityComparer<T> 派生 - 我只是直接实现IEqualityComparer<T>。除了实现非泛型 IEqualityComparer 之外，我不确定 EqualityComparer<T> 在这里真正给你带来了什么价值。

还有几件事：

• 您应该在 Equals 中处理无效性

• 您现在的 Equals 代码可以简化为：

  return x.Id == y.Id && x.Name == y.Name;
  

Equals 的更完整实现可能是：

public override bool Equals(Customer x, Customer y)
{
    if (object.ReferenceEquals(x, y))
    {
        return true;
    }
    if (x == null || y == null)
    {
        return false;
    }
    return x.Id == y.Id && x.Name == y.Name;
}

Answer 2

您应该从可能的“碰撞”的角度来看待它，例如当两个不同的对象获得相同的哈希码时。这可能是“1,2any”和“12, any”这样的对的情况，对中的值是“id”和“name”。如果您的数据无法做到这一点，那么您就可以开始了。否则，您可以将其更改为：

return obj.Id.GetHashCode() ^ obj.Name.GetHashCode();

Answer 3

Resharper（来自 JetBrains 的出色重构插件）认为它应该是：

public override int GetHashCode(Customer obj)
{
    unchecked
    {
        return ((obj.Id != null ? obj.Id.GetHashCode() : 0) * 397) 
            ^ (obj.Name != null ? obj.Name.GetHashCode() : 0);
    }
}

我不得不承认，我几乎总是让 Resharper 为我生成相等和哈希码实现。我已经对他们的实现进行了大量测试，发现它与我手写的任何东西一样好，如果不是更好的话。所以我通常会采用我不必键入的实现。