2

好的,我有一个类,我想在其中覆盖相等运算符,所以我有以下代码:

/// <summary>
/// Over-ride of the equality operator.
/// </summary>
/// <param name="credential1">The left hand object to test for equality.</param>
/// <param name="credential2">The right hand object to test for equality.</param>
/// <returns>True if the objects are equal.</returns>
public static bool operator ==(KnownServerCredential credential1, KnownServerCredential credential2)
{
    // ok check if either is null
    bool cell1Null = Equals(null, credential1);
    bool cell2Null = Equals(null, credential2);

    // if both are, then it's true
    if (cell1Null && cell2Null)
    {
        return true;
    }

    // if only one is, then how can they be the same?
    if (cell1Null || cell2Null)
    {
        return false;
    }

    // neither are - so we can now go to full on equality
    return credential1.IsEqualTo(credential2);
}

这很好用,我很满意。然而,静态分析工具(Resharper 和 VS2010 代码分析)发誓最后一行可能会引发空引用异常,因为我在前两行中检查空值的方式。如果我将前两行从 更改为Equals(null, credentialX)credentialX == null则静态分析工具会很高兴,但它会创建堆栈溢出异常,因为然后我会递归调用相等覆盖。我可以通过使用两全其美(object)credentialX == null,但这似乎不是最干净的方式。

所以简单的问题是,我是否遗漏了某些东西,或者演员和比较是实现我正在寻找的最佳方式?

4

2 回答 2

3

检查一对对象是否相等有两种方法:引用相等和结构/值相等。引用相等是所有引用类型(类)的默认值,结构相等是所有值类型的默认值(但默认实现不是最优的)。使用以下指南为引用类型和值类型实现结构相等。

平等

相等性检查应遵循以下规则:

  • 一个对象等于它自己(身份)
  • 比较x返回y与比较相同的y事实x。(对称)
  • 如果x等于y并且y等于z,则x必须等于z。(传递性)
  • 一个对象永远不等于null
  • null等于null
  • 不应抛出任何异常。

让您的类或结构实现IEquatable<T>自定义相等检查的接口,然后实现IEquatable<T>.Equals(T)andObject.Equals()方法。

引用类型(类)的相等性

对于引用类型,实现如下IEquatable<T>.Equals(T)方法:

public bool Equals(MyType other)
{
    if (Object.ReferenceEquals(other, null) ||      // When 'other' is null
        other.GetType() != this.GetType())          // or of a different type
        return false;                               // they are not equal.
    return this.field1 == other.field1
        && this.field2 == other.field2;
}

然后像这样覆盖Object.Equals()

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

值类型(结构)的平等

由于值类型不能是null,因此实现如下IEquatable<T>.Equals(T)方法:

public bool Equals(MyType other)
{
    return this.field == other.field
        && this.field2 == other.field2;
}

然后像这样覆盖Object.Equals()

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

等式运算符

对于引用类型和值类型,您可能希望覆盖默认的相等和不等运算符。根据Jon Skeet的这篇文章,等式和不等式运算符可以这样实现:

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

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

请注意, when和left/或rightis不调用覆盖(因此不调用方法)。nullObject.Equals(object, object)Object.Equals(object)IEquatable<T>.Equals(T)

哈希码

有时对象的哈希码很重要,例如当对象可能被放入字典或哈希表时。一般来说,当你重写Equals()方法时,重写GetHashCode()方法。哈希码应遵循以下规则

  • 哈希码永远不应该改变,即使在修改了对象中的某些字段之后也是如此。
  • 对于被认为相等的对象,哈希码必须相等。
  • 对于被认为不相等的对象,哈希码可以是任何东西(包括相等)。
  • 哈希码应该是随机分布的。
  • 哈希码函数绝不能抛出异常,并且必须始终返回。
  • 哈希码的计算速度应该非常快。

因此,要实现Object.GetHashCode()使用结构相等的类或结构,请从对象中选择一些不可变的字段并标记它们readonly。仅使用这些字段来计算哈希码。覆盖该Object.GetHashCode()方法并像这样实现它

public override int GetHashCode()
{
    unchecked
    {
        int hash = 17;
        // Don't forget to check for null values.
        hash = hash * 29 + field1.GetHashCode();
        hash = hash * 29 + field2.GetHashCode();
        // ...
        return hash;
    }
}

或者,如果您只有一个不可变字段,您可以考虑只使用:

public override int GetHashCode()
{
    // Don't forget to check for null values.
    return field1.GetHashCode();
}

如果您没有不可变字段,请返回一个常量哈希码。例如,类型本身的哈希码

public override int GetHashCode()
{
    return GetType().GetHashCode();
}

集合的结构平等

不应使用默认Equals()方法比较集合。相反,集合的默认相等性应该是引用相等性。要实现结构相等,请实现IStructuralEquatable接口。例如:

bool IStructuralEquatable.Equals(object obj, IEqualityComparer comparer)
{
    var other = obj as MyType;
    if (other == null)
        return false;
    return ((IStructuralEquatable)this.innerArray)
        .Equals(other.innerArray, comparer);
}

int IStructuralEquatable.GetHashCode(IEqualityComparer comparer)
{
    return ((IStructuralEquatable)this.innerArray).GetHashCode(comparer);
}
于 2012-07-19T11:28:47.920 回答
2

通常,当我必须检查是否等于 null 时,我使用:

if (object.ReferenceEquals(myref, null))
{

它的优点是可以绕过重载Equalsoperator==.

于 2012-07-19T11:26:24.143 回答