覆盖 == 运算符。如何与空值进行比较？ [复制]

Question

可能重复：
How do I check for nulls in an ‘==’ operator overload without infinite recursion?

这可能有一个简单的答案......但它似乎在逃避我。这是一个简化的例子：

public class Person
{
   public string SocialSecurityNumber;
   public string FirstName;
   public string LastName;
}

假设对于这个特定的应用程序，如果社会安全号码匹配，并且两个名字匹配，那么我们指的是同一个“人”。

public override bool Equals(object Obj)
{
    Person other = (Person)Obj;
    return (this.SocialSecurityNumber == other.SocialSecurityNumber &&
        this.FirstName == other.FirstName &&
        this.LastName == other.LastName);
}

为了保持一致，我们也为团队中不使用 .Equals 方法的开发人员覆盖 == 和 != 运算符。

public static bool operator !=(Person person1, Person person2)
{
    return ! person1.Equals(person2);
}

public static bool operator ==(Person person1, Person person2)
{
    return person1.Equals(person2);
}

很好，花花公子，对吧？

但是，当 Person 对象为 null 时会发生什么？

你不能写：

if (person == null)
{
    //fail!
}

因为这将导致 == 运算符覆盖运行，并且代码将在以下位置失败：

person.Equals()

方法调用，因为您不能在空实例上调用方法。

另一方面，您不能在 == 覆盖中明确检查此条件，因为它会导致无限递归（以及 Stack Overflow [dot com]）

public static bool operator ==(Person person1, Person person2)
{
    if (person1 == null)
    {
         //any code here never gets executed!  We first die a slow painful death.
    }
    return person1.Equals(person2);
}

那么，如何覆盖 == 和 != 运算符以实现值相等并仍然考虑空对象？

我希望答案不是简单得令人痛苦。 :-)

Answer 1

使用object.ReferenceEquals(person1, null) 或新的is operator 代替== 运算符：

public static bool operator ==(Person person1, Person person2)
{
    if (person1 is null)
    {
         return person2 is null;
    }

    return person1.Equals(person2);
}

Answer 2

我一直都是这样做的（对于 == 和 != 运算符），我为我创建的每个对象重复使用此代码：

public static bool operator ==(Person lhs, Person rhs)
{
    // If left hand side is null...
    if (System.Object.ReferenceEquals(lhs, null))
    {
        // ...and right hand side is null...
        if (System.Object.ReferenceEquals(rhs, null))
        {
            //...both are null and are Equal.
            return true;
        }

        // ...right hand side is not null, therefore not Equal.
        return false;
    }

    // Return true if the fields match:
    return lhs.Equals(rhs);
}

"!=" 然后是这样的：

public static bool operator !=(Person lhs, Person rhs)
{
    return !(lhs == rhs);
}

编辑
我修改了== 运算符函数以匹配微软建议的实现here。

Answer 3

你总是可以覆盖并放置

(Object)(person1)==null

我想这会起作用，但不确定。

Answer 4

比任何这些方法都更容易使用

public static bool operator ==(Person person1, Person person2)   
{   
    EqualityComparer<Person>.Default.Equals(person1, person2)
} 

这与其他人提出的方法具有相同的空相等语义，但找出细节是框架的问题:)

Answer 5

最终的（假设的）例程如下。这与@cdhowie 第一次接受的回复非常相似。

public static bool operator ==(Person person1, Person person2)
{
    if (Person.ReferenceEquals(person1, person2)) return true;
    if (Person.ReferenceEquals(person1, null)) return false; //*
    return person1.Equals(person2);
}

感谢您的精彩回复！

//* - .Equals() 对 person2 执行空检查

Answer 6

将Person 实例转换为object：

public static bool operator ==(Person person1, Person person2)
{
    if ((object)person1 == (object)person2) return true;
    if ((object)person1 == null) return false;
    if ((object)person2 == null) return false;
    return person1.Equals(person2);
}

Answer 7

将 Person 转换为 Object，然后执行比较：

object o1 = (object)person1;
object o2 = (object)person2;
if(o1==o2) //compare instances.
   return true;
if (o1 == null || o2 == null)  //compare to null.
   return false;
//continue with Person logic.

Answer 8

始终重载这些运算符非常困难。 My answer to a related question 可以作为模板。

基本上，你首先需要做一个引用（object.ReferenceEquals）测试，看看对象是不是null。 然后你打电话给Equals。

Answer 9

cdhowie 使用ReferenceEquals 赚钱，但值得注意的是，如果有人将null 直接传递给Equals，您仍然可以获得例外。此外，如果您要覆盖Equals，几乎总是值得实现IEquatable<T>，所以我会改为。

public class Person : IEquatable<Person>
{
  /* more stuff elided */

  public bool Equals(Person other)
  {
    return !ReferenceEquals(other, null) &&
      SocialSecurityNumber == other.SocialSecurityNumber &&
      FirstName == other.FirstName &&
      LastName == other.LastName;
  }
  public override bool Equals(object obj)
  {
    return Equals(obj as Person);
  }
  public static bool operator !=(Person person1, Person person2)
  {
    return !(person1 == person2);
  }
  public static bool operator ==(Person person1, Person person2)
  {
    return ReferenceEquals(person1, person2)
      || (!ReferenceEquals(person1, null) && person1.Equals(person2));
  }
}

当然，你永远不应该覆盖Equals，也不要覆盖GetHashCode()

public override int GetHashCode()
{
   //I'm going to assume that different
   //people with the same SocialSecurityNumber are extremely rare,
   //as optimise by hashing on that alone. If this isn't the case, change this
   return SocialSecurityNumber.GetHashCode();
}

同样值得注意的是，身份需要平等（也就是说，对于任何有效的“平等”概念，某物总是等于它自己）。由于相等性测试可能很昂贵并且发生在循环中，并且由于在实际代码中将某些内容与自身进行比较往往很常见（尤其是如果对象在多个地方传递），因此值得将其添加为快捷方式：

  public bool Equals(Person other)
  {
    return !ReferenceEquals(other, null) &&
      ReferenceEquals(this, other) ||
      (
        SocialSecurityNumber == other.SocialSecurityNumber &&
        FirstName == other.FirstName &&
        LastName == other.LastName
      );
  }

ReferenceEquals(this, other) 上的捷径有多少好处可能会因课程的性质而有很大差异，但是否值得这样做是人们应该始终考虑的事情，所以我在这里包含了这项技术.