IEquatable 接口检查 null 时做什么

Question

我使用以下代码在类中实现了 IEquatable 接口。

        public bool Equals(ClauseBE other)
        {
            if (this._id == other._id)
            {
                return true;
            }
            return false;
        }

        public override bool Equals(Object obj)
        {
            if (obj == null)
            {
                return base.Equals(obj);
            }

            if (!(obj is ClauseBE))
            {
                throw new InvalidCastException("The 'obj' argument is not a ClauseBE object.");
            }

            return Equals(obj as ClauseBE);
        }

        public override int GetHashCode()
        {
            return this._id.GetHashCode();
        }

        public static bool operator ==(ClauseBE a, ClauseBE b)
        {
            // cast to object so we call the overloaded Equals function which appropriately checks when b is null.
            return a.Equals(b as object);
        }

        public static bool operator !=(ClauseBE a, ClauseBE b)
        {
            // cast to object so we call the overloaded Equals function which appropriately checks when b is null.
            return !a.Equals(b as object);
        }

这段代码适用于大多数情况。 但是，以下检查会在相等运算符重载方法中引发异常，因为 a 为 null，因此没有 Equals 方法。

if(this.Clause != null)
{

}

解决这个问题的标准方法是什么？

编辑

我已经做到了这一点，但它看起来很麻烦。 我希望有一种更优雅的方式来实现这一点。

    public static bool operator ==(ClauseBE a, ClauseBE b)
    {
        if (a as object == null && b as object == null)
        {
            return true;
        }

        if ((a as object == null && b as object != null)
            || (b as object == null && a as object != null))
        {
            return false;
        }

        // cast to object so we call the overloaded Equals function which appropriately checks when b is null.
        return a.Equals(b as object);
    }

    public static bool operator !=(ClauseBE a, ClauseBE b)
    {
        if (a as object == null && b as object == null)
        {
            return false;
        }

        if((a as object == null && b as object != null)
            || (b as object == null && a as object != null))
        {
            return true;
        }

        // cast to object so we call the overloaded Equals function which appropriately checks when b is null.
        return !a.Equals(b as object);
    }

解决办法

谢谢大家。 我从大家那里得到了很多好的建议，我真的很感激。 这就是我最终决定的，它比我开始时要优雅得多。 除了运算符重载之外，所有代码都是相同的。

public static bool operator ==(ClauseBE a, ClauseBE b)
{
    if (ReferenceEquals(a, null) && ReferenceEquals(b, null))
    {
        return true;
    }

    if (ReferenceEquals(a, null) || ReferenceEquals(b, null))
    {
        return false;
    }

    return a.Equals(b);
}

public static bool operator !=(ClauseBE a, ClauseBE b)
{
    return !(a == b);
}

Answer 1

我总是发现使用 null 处理编写静态运算符更容易，并且让 Equals 重写以“this”作为参数之一调用重载运算符。

来自重载 Equals() 和运算符 == 的指南（C# 编程指南）

//add this code to class ThreeDPoint as defined previously
//
public static bool operator ==(ThreeDPoint a, ThreeDPoint b)
{
    // If both are null, or both are same instance, return true.
    if (System.Object.ReferenceEquals(a, b))
    {
        return true;
    }

    // If one is null, but not both, return false.
    if (((object)a == null) || ((object)b == null))
    {
        return false;
    }

    // Return true if the fields match:
    return a.x == b.x && a.y == b.y && a.z == b.z;
}

public static bool operator !=(ThreeDPoint a, ThreeDPoint b)
{
    return !(a == b);
}

Answer 2

这就是 ReSharper 创建相等运算符并实现 IEquatable 的方式，当然，我盲目地相信它；-)

public class ClauseBE : IEquatable<ClauseBE>
{
    private int _id;

    public bool Equals(ClauseBE other)
    {
        if (ReferenceEquals(null, other))
            return false;
        if (ReferenceEquals(this, other))
            return true;
        return other._id == this._id;
    }

    public override bool Equals(object obj)
    {
        if (ReferenceEquals(null, obj))
            return false;
        if (ReferenceEquals(this, obj))
            return true;
        if (obj.GetType() != typeof(ClauseBE))
            return false;
        return Equals((ClauseBE)obj);
    }

    public override int GetHashCode()
    {
        return this._id.GetHashCode();
    }

    public static bool operator ==(ClauseBE left, ClauseBE right)
    {
        return Equals(left, right);
    }

    public static bool operator !=(ClauseBE left, ClauseBE right)
    {
        return !Equals(left, right);
    }
}

Answer 3

检查是否为 null 并返回 false。 如果操作数之一为 null，则等于应始终为 false；

Answer 4

我认为这比在检查 null 之前转换为 Object 稍微麻烦一些：

ReferenceEquals(a, null)

Answer 5

其他答案为一般问题提供了很好的解决方案。

但是，您自己的代码可以简化为相对简单的解决方案...

首先，在您的 == 运算符的开头，您有这样的内容：

    // First test
    if (a as object == null && b as object == null)
    {
        return true;
    }

这符合“工作太努力”的条件。

如果ClauseBE是引用类型，那么你只需要与null进行比较——“as object”是多余的； 同样，如果 ClauseBE 是值类型，那么它永远不可能是 null。

假设 ClauseBE 是引用类型（最有可能的情况），那么您可以简化为此 - 请注意，我们使用 Object.Equals() 来避免无限递归和堆栈井喷。

    // First test
    if (Object.Equals(a, null) && Object.Equals(b, null))
    {
        return true;
    }

一种有用的快捷方式是使用 Object.ReferenceEquals() - 它可以为您处理空值。

所以你可以这样写：

    // First test
    if (Object.ReferenceEquals(a, b))
    {
        return true;
    }

这样做的好处是它还可以处理 a 和 b 是同一个对象的情况。

一旦通过了 Object.ReferenceEquals() 测试，您就会知道 a 和 b 是不同的。

因此，您的下一个测试：

    // Second test
    if ((a as object == null && b as object != null)
        || (b as object == null && a as object != null))
    {
        return false;
    }

可以简化 - 因为您知道如果 a 为 null，则 b 不能为 null，依此类推。

    // Second test
    if (Object.Equals(a, null) || Object.Equals(b, null))
    {
        return false;
    }

如果此测试失败，那么您就知道 a 和 b 不同，并且都不为 null。 现在是调用重写的 Equals() 的好时机。

    // Use the implementation of Equals() for the rest
    return a.Equals(b as object);

Answer 6

public class Foo : IEquatable<Foo>
{
    public Int32 Id { get; set; }

    public override Int32 GetHashCode()
    {
        return this.Id.GetHashCode();
    }

    public override Boolean Equals(Object obj)
    {
        return !Object.ReferenceEquals(obj as Foo, null)
            && (this.Id == ((Foo)obj).Id);

        // Alternative casting to Object to use == operator.
        return ((Object)(obj as Foo) != null) && (this.Id == ((Foo)obj).Id);
    }

    public static Boolean operator ==(Foo a, Foo b)
    {
        return Object.Equals(a, b);
    }

    public static Boolean operator !=(Foo a, Foo b)
    {
        return !Object.Equals(a, b);
    }

    public Boolean Equals(Foo other)
    {
        return Object.Equals(this, other);
    }
}

Answer 7

我使用了以下方法，它似乎对我很有效。 事实上，Resharper 建议采用这种方法。

public bool Equals(Foo pFoo)
{
        if (pFoo == null)
            return false;
        return (pFoo.Id == Id);
}

public override bool Equals(object obj)
{
        if (ReferenceEquals(obj, this))
            return true;

        return Equals(obj as Foo);
}

Answer 8

我更喜欢在 Equals(T) 方法中执行所有比较逻辑，并将运算符重载中的“if this or that is null, else ...”留给框架。

覆盖运算符重载的唯一棘手的事情是您不能再在 Equals 实现中使用这些运算符，例如与 null 进行比较。 相反，object.ReferenceEquals可以达到同样的效果。

遵循 MSDN 中的 TwoDPoint 示例 重写 Equals() 和运算符 ==< 的指南/a> 文章，这是我在实现类型的值相等时生成的模式：

public override bool Equals( object obj ) {
  // Note: For value types, would use:
  // return obj is TwoDPoint && this.Equals( (TwoDPoint)obj );
  return this.Equals( obj as TwoDPoint );
}

public bool Equals( TwoDPoint other ) {
  // Note: null check not needed for value types.
  return !object.ReferenceEquals( other, null )
      && EqualityComparer<int>.Default.Equals( this.X, other.X )
      && EqualityComparer<int>.Default.Equals( this.Y, other.Y );
}

public static bool operator ==( TwoDPoint left, TwoDPoint right ) {
  // System.Collections.Generic.EqualityComparer<T> will perform the null checks 
  //  on the operands, and will call the Equals overload if necessary.
  return EqualityComparer<TwoDPoint>.Default.Equals( left, right );
}

public static bool operator !=( TwoDPoint left, TwoDPoint right ) {
  return !EqualityComparer<TwoDPoint>.Default.Equals( left, right );
}

上面的形式是最安全的实现，因为它只是将字段相等性检查转发到框架，并且不需要知道字段是否重载相等运算符。 在您知道存在过载的情况下简化此操作是完全可以的：

public bool Equals( TwoDPoint other ) {
  return !object.ReferenceEquals( other, null )
      && this.X == other.X
      && this.Y == other.Y;
}

您还可以替换 EqualityComparer运算符中的调用会通过调用静态 object.Equals 方法，当比较引用类型或装箱值类型无关紧要时：

public static bool operator ==( TwoDPoint left, TwoDPoint right ) {
  return object.Equals( left, right );
}

public static bool operator !=( TwoDPoint left, TwoDPoint right ) {
  return !object.Equals( left, right );
}

另请参阅 重写 GetHashCode 的最佳算法是什么？ 用于实现 GetHashCode。