Question

我有一个客户类。

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

我继承了EqualityComparer表单MyEqualityComparer（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>。除了同时实现非通用EqualityComparer<T>之外，我不确定IEqualityComparer在这里给你的价值是多少。

还有一些事情：

您应该在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为我生成相等和哈希代码实现。我已经对它们的实现进行了大量的测试，并发现它与我手工编写的任何内容一样好。所以我通常会采用我没有输入的实现。

继承相等比较器

3 个答案: