3

我想知道在 C# 中实现正确、灵活和快速的 Equals 的最佳方法，它几乎可以用于任何类和情况。我认为性能需要一个专门的 Equals（将实际类的对象作为参数）。为了避免代码重复，一般的 Equals 应该调用专门的 Equals。空值检查应该只执行一次，即使在继承的类中也是如此。

我终于想出了这个设计：


class MyClass
{
    public Int32 SomeValue1 = 1;
    public Int32 SomeValue2 = 25;

    // Ignoring GetHashCode for simplicity.

    public override bool Equals(object obj)
    {
        return Equals (obj as MyClass);
    }

    public bool Equals(MyClass obj)
    {
        if (obj == null) {
            return false;
        }

        if (!SomeValue1.Equals (obj.SomeValue1)) {
            return false;
        }

        if (!SomeValue2.Equals (obj.SomeValue2)) {
            return false;
        }

        return true;
    }
}

class MyDerivedClass : MyClass
{
    public Int32 YetAnotherValue = 2;

    public override bool Equals(object obj)
    {
        return Equals (obj as MyDerivedClass);
    }

    public bool Equals(MyDerivedClass obj)
    {
        if (!base.Equals (obj)) {
            return false;
        }

        if (!YetAnotherValue.Equals (obj.YetAnotherValue)) {
            return false;
        }

        return true;
    }
}

重要的想法：

  • “as”运算符的用法。这样我们就不必检查一般 Equals 中的空值。错误的类类型会减少为 null，并将在专门的 Equals 中进行排序。
  • 在某一点进行空值检查，即使对于派生类也是如此。
  • 一个接一个地检查属性提供了一个清晰的结构。

这个概念有缺陷，还是我错过了任何条件？

4

1 回答 1

5

当涉及不同类型时，您的 Equals 方法不是自反的：

MyDerivedClass mdc = new MyDerivedClass();
MyClass mc = new MyClass();
Object omdc = mdc;
Object omc = mc;

// mc.Equals(mdc) - true
// mdc.Equals(mc) - true by calling the right overload
// omc.Equals(omdc) - true
// omdc.Equals(omc) - false, the "as" in MyDerivedClass will result in null

正常的方法是使用：

if (GetType() != other.GetType())
{
    return false;
}

请参阅Object.Equals中的文档：“x.Equals(y) 返回与 y.Equals(x) 相同的值。” 依靠重载来给出不同的结果可能会导致可怕的问题，这些问题对于调试来说非常微妙。

于 2009-01-20T11:13:29.947 回答