重载相等运算符时，处理空值的最佳方法是什么？ [复制]

Question

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

假设我有这样的类型：

public class Effect
{
    public static bool operator == ( Effect a, Effect b )
    {
        return a.Equals ( b );
    }

    public static bool operator != ( Effect a, Effect b )
    {
        return !a.Equals ( b );
    }

    public bool Equals ( Effect effect )
    {
        return this.TypeID.Equals ( effect.TypeID );
    }

    public override bool Equals ( object obj )
    {
        return this.TypeID.Equals ( ( ( Effect ) obj ).TypeID );
    }
}

处理空值最可靠、最简洁的方法是什么？

我不确定是否必须同时检查当前实例 (this) 和传递的实例 (effect/obj) 是否为空？如果当前实例 (this) 为 null，编译器还会调用 effect.Equals 还是 Object.Equals？

同样，空值检查应该在哪里进行？我假设只在 Equals 方法中，而不是在相等运算符 (==, !=)。

Answer 1

首先，this 永远不能是 null，至少在 C# 编译器生成的代码中是这样。

其次，使用ReferenceEquals 方法检查null 引用，而不要调用== 的重载版本（或执行((object) sometypeinstance) == null）。

Answer 2

添加这个：

 public static bool operator == ( Effect a, Effect b )     
 {  
     return a is Effect && b is Effect && a.TypeID.Equals (b.TypeID);    
 }  
 public static bool operator != ( Effect a, Effect b )
 {         
     return !(a == b );     
 }      
 public bool Equals ( Effect effect )     
 {         
    return this == effect );     
 }      
 public override bool Equals ( object obj )     
 {   
    return obj is Effect && this == obj); 
 } 

或代替最后放置：

 public override bool Equals ( object obj )     
 {   
    if (obj == null) throw new ArgumentNullException(
         "obj", "obj is null");
    if (!(obj is effect)) throw new ArgumentException(
         "obj", "obj is not an effect");
    return obj is Effect && this == obj); 
 }

Answer 3

怎么样

public static bool operator == ( Effect a, Effect b )     
{  
    return object.Equals(a, b);
}

Object.Equals() 的默认实现会为您执行 null 检查。

如果您好奇，下面是 Object.Equals() 的工作方式（由 .NET Reflector 提供）：

public static bool Equals(object objA, object objB)
{
    return ((objA == objB) || (((objA != null) && (objB != null)) && objA.Equals(objB)));
}

Answer 4

是的，您应该检查 null。你为什么害怕那个？

请记住，当您使用 equals 时，您可能还想看看 hashcode 方法！这两种方法是相互交织的。

Answer 5

有必要检查每个带参数的方法是否存在空值。

public class Effect
{
    public static bool operator == ( Effect a, Effect b )
    {
        if (a == null) && (b == null) return true;
            if (a == null) return false;
            return a.Equals ( b );
    }

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

    public bool Equals ( Effect effect )
    {
            if (b == null) return false;
        return this.TypeID.Equals ( effect.TypeID );
    }

    public override bool Equals ( object obj )
    {
            if (obj == null) return false;
        return this.TypeID.Equals ( ( ( Effect ) obj ).TypeID );
    }
}

其他需要注意的事情是 GetHashCode，如果要实现相等，则应该实现 GetHashCode，如果要在字典或比较项目的类似对象中使用该对象，则 GetHashCode 应该只在不可变属性上实现。使用哈希码。

Answer 6

Visual Studio 有一个 sn-p，它为您提供了一个基本的 Equals() 实现。我会遵循这一点，除非你有充分的理由不这样做。

// override object.Equals
public override bool Equals(object obj)
{
    //       
    // See the full list of guidelines at
    //   http://go.microsoft.com/fwlink/?LinkID=85237  
    // and also the guidance for operator== at
    //   http://go.microsoft.com/fwlink/?LinkId=85238
    //

    if (obj == null || GetType() != obj.GetType())
    {
        return false;
    }

    // TODO: write your implementation of Equals() here
    throw new NotImplementedException();
    return base.Equals(obj);
}

// override object.GetHashCode
public override int GetHashCode()
{
    // TODO: write your implementation of GetHashCode() here
    throw new NotImplementedException();
    return base.GetHashCode();
}