重载 operator== 与 Equals()

Question

我正在开发一个 C# 项目，到目前为止，我一直在使用不可变对象和工厂来确保 Foo 类型的对象始终可以与 == 进行比较是否相等。

Foo 对象一旦创建就无法更改，并且工厂始终为给定的一组参数返回相同的对象。这很好用，并且在整个代码库中，我们假设== 始终用于检查相等性。

现在我需要添加一些引入边缘情况的功能，但这种情况并不总是有效。最简单的做法是为该类型重载operator ==，这样项目中的其他代码都不需要更改。但这让我觉得代码味道很重：重载operator == 而不是Equals 看起来很奇怪，而且我习惯了== 检查引用相等性和Equals 检查对象相等性（或任何术语）的约定是）。

这是一个合理的担忧，还是我应该继续超载operator ==？

Answer 1

根据 Microsoft 自己的最佳实践，Equals 方法和等号 (==) 重载的结果应该相同。

CA2224: Override equals on overloading operator equals

Answer 2

显示如何根据MSFT guidelines（下）实现此功能的示例。请注意，在覆盖 Equals 时，您还需要覆盖 GetHashCode()。希望对大家有所帮助。

public class Person
{
    public Guid Id { get; private set; }

    public Person(Guid id)
    {
        Id = id;
    }

    public Person()
    {
        Id = System.Guid.NewGuid();
    }

    public static bool operator ==(Person p1, Person p2)
    {
        bool rc;

        if (System.Object.ReferenceEquals(p1, p2))
        {
            rc = true;
        }
        else if (((object)p1 == null) || ((object)p2 == null))
        {
            rc = false;
        }
        else
        {
            rc = (p1.Id.CompareTo(p2.Id) == 0);
        }

        return rc;
    }

    public static bool operator !=(Person p1, Person p2)
    {
        return !(p1 == p2);
    }

    public override bool Equals(object obj)
    {
        bool rc = false;
        if (obj is Person)
        {
            Person p2 = obj as Person;
            rc = (this == p2);
        }
        return rc;
    }

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

Answer 3

重载 == 和覆盖 Equals 之间有很大区别。

当你有表情时

if (x == y) {

用于比较变量 x 和 y 的方法在编译时决定。这是运算符重载。声明 x 和 y 时使用的类型用于定义用于比较它们的方法。 x 和 y 中的实际类型（即子类或接口实现）是无关紧要的。请考虑以下内容。

object x = "hello";
object y = 'h' + "ello"; // ensure it's a different reference

if (x == y) { // evaluates to FALSE

还有以下

string x = "hello";
string y = 'h' + "ello"; // ensure it's a different reference

if (x == y) { // evaluates to TRUE

这表明用于声明变量 x 和 y 的类型用于确定使用哪种方法来评估 ==。

相比之下，Equals 是在运行时根据变量 x 中的实际类型确定的。 Equals 是 Object 上的一个虚方法，其他类型可以并且确实可以覆盖。因此，以下两个示例的计算结果都为真。

object x = "hello";
object y = 'h' + "ello"; // ensure it's a different reference

if (x.Equals(y)) { // evaluates to TRUE

还有以下

string x = "hello";
string y = 'h' + "ello"; // ensure it's a different reference

if (x.Equals(y)) { // also evaluates to TRUE

Answer 4

对于不可变类型，我认为重载== 以支持值相等没有什么问题。我认为我不会在不覆盖 Equals 的情况下覆盖 == 以获得相同的语义。如果您确实覆盖了==，并且由于某种原因需要检查引用相等性，您可以使用Object.ReferenceEquals(a,b)。

看到这个Microsoft article for some useful guidelines

Answer 5

我相信标准是，对于大多数类型，.Equals 检查对象相似性，运算符 == 检查引用相等性。

我认为最佳实践是对于不可变类型，运算符 == 和 .Equals 应该检查相似性。如果您想知道它们是否真的是同一个对象，请使用.ReferenceEquals。有关此示例，请参阅 C# String 类。

Answer 6

确实有味道。当重载== 时，您应该确保Equals() 和GetHashCode() 也是一致的。请参阅MSDN guidelines。

这看起来完全没问题的唯一原因是您将您的类型描述为不可变的。