重载的运算符参数始终为 null

Question

所以我有一个类，它重写 Equals(object obj) 和 GetHashCode() 以及实现 IEquatable。为了在检查相等性时使使用这种类型更加自然，我想，哎呀，我会重载相等运算符和不等运算符，不用担心......

呃哦，担心......考虑以下内容 - 其中两个 myType 实例不为空：

if (myType != container.myType) //NullReferenceException
{
    //never get here
}
//never get here either

现在，容器只是另一个类，用于保存 myType 的实例以及用于缓存项目的其他内容。

这是来自 myType 的实际（相关）代码：

public class MyType : IEquatable<MyType>
{
    public static bool operator ==(MyType myTypeA, MyType myTypeB)
    {
        return myTypeA.Equals(myTypeB);
    }

    public static bool operator !=(MyType myTypeA, MyType myTypeB)
    {
        return !(myTypeA == myTypeB);
    }

    public override bool Equals(object obj)
    {
        if (obj != null && obj is MyType)
        {
            return Equals((MyType)obj);
        }
        return false;
    }

    public bool Equals(MyType other)
    {
        if (other != null)
        {
            return other.ToString() == ToString();
        }
        return false;
    }
}

在这方面有任何经验吗？

谢谢。

Answer 1

正如其他人所说，您需要仔细检查空值，因为它会再次调用您的相等函数，通常会导致 StackOverflowException。

当我在类上使用 IEquatable 接口时，我通常使用以下代码：

public override bool Equals(object obj)
{
    // If obj isn't MyType then 'as' will pass in null
    return this.Equals(obj as MyType);
}

public bool Equals(MyType other)
{
    if (object.ReferenceEquals(other, null))
    {
        return false;
    }

    // Actual comparison code here
    return other.ToString() == this.ToString();
}

Answer 2

几个指针 -

1. 如果您在类上重写了 == 和 !=，请确保使用 ReferenceEquals 检查重载内是否为 null实现而不是 ==，因为这会调用重载的运算符，然后进入循环或尝试在 null this 引用上调用 Equals ，这可能就是这里发生的情况。
2. 不要重写类上的 == 和 != 。这些运算符旨在实现值相等，而类并不是真正设计为具有值相等。删除运算符重载，或将 MyType 设为结构体。

Answer 3

棘手的一个...问题是您在 Equal 覆盖中使用相等运算符，如下所示：

public bool Equals(MyType other)
{
    if (other != null)

它转到您重载的 != 运算符，该运算符又转到您的 == 运算符，该运算符试图执行 null.Equals...