c# - 值对象的 IEqualityComparer

Question

我有一个不可变的值对象 IPathwayModule，其值定义为:

• (int) block ；
• (实体)模块，由(字符串)ModuleId 标识；
• (枚举)状态；和
• (实体)类，由(字符串)ClassId 标识 - 可以为 null。

这是我当前的 IEqualityComparer 实现，它似乎在一些单元测试中有效。但是，我认为我对自己做得很好的了解还不足以知道我做的是否正确。以前的实现有时会在重复测试运行时失败。

private class StandardPathwayModuleComparer : IEqualityComparer<IPathwayModule>
{
    public bool Equals(IPathwayModule x, IPathwayModule y)
    {
        int hx = GetHashCode(x);
        int hy = GetHashCode(y);
        return hx == hy;
    }

    public int GetHashCode(IPathwayModule obj)
    {
        int h;
        if (obj.Class != null)
        {
            h = obj.Block.GetHashCode() + obj.Module.ModuleId.GetHashCode() + obj.Status.GetHashCode() + obj.Class.ClassId.GetHashCode();
        }
        else
        {
            h = obj.Block.GetHashCode() + obj.Module.ModuleId.GetHashCode() + obj.Status.GetHashCode() + "NOCLASS".GetHashCode();
        }
        return h;
    }
}

IPathwayModule 绝对是不可变的，具有相同值的不同实例应该相等并产生相同的 HashCode，因为它们被用作 HashSet 中的项目。

我想我的问题是:

• 在这种情况下，我是否正确使用了界面？
• 是否存在我可能看不到预期行为的情况？
• 有什么方法可以提高稳健性和性能吗？
• 是否有任何我没有遵循的良好做法？

Answer 1

不要根据 Hash 函数的结果执行 Equals，它太脆弱了。而是对每个字段进行字段值比较。像这样的东西:

return x != null && y != null && x.Name.Equals(y.Name) && x.Type.Equals(y.Type) ...

此外，哈希函数的结果并不真正适合加法。尝试改用 ^ 运算符。

return obj.Name.GetHashCode() ^ obj.Type.GetHashCode() ...

您不需要在 GetHashCode 中进行空检查。如果该值为 null，则您遇到了更大的问题，尝试从您无法控制的事情中恢复是没有用的...