1

可能重复:
比较引用类型的两个实例的“最佳实践”是什么?

我的应用程序有这个自定义类。这个类有两个实例(A 和 B),我试图比较它们。但是,我遇到了问题;我正在 IEquatable<T>使用重写的Equals方法实现接口以进行此比较。

这些Equals方法调用了ReferenceEquals行为不端的函数。

以下是测试用例:

案例1: A和B是不同的实例,包含不同的数据..ReferenceEquals说:它们是不同的(这是正确的!)

案例 2: A 和 B 是不同的实例,但 B 是使用 A 的变量值实例化的(即 A 和 B 都包含完全相同的数据!)..ReferenceEquals说:它们不同(这是错误的!)

案例 3: A 和 B 是相同的实例(即 A 被传入两次,例如Equals (A, A)ReferenceEquals说:它们是相同的(这是正确的!)

那么如何让案例 2 的结果也正确呢?

实现的类IEquatable<T>

namespace DBtestApp1
{
    class QuantityBasedDiscount : IEquatable<QuantityBasedDiscount>
    {
        public string pType { get; set; }
        public string pSubType { get; set; }
        public DataTable quantityDiscountsDT { get; set; }

        public QuantityBasedDiscount()
        {
            pType = "";
            pSubType = "";
            quantityDiscountsDT = new DataTable();
        }

        public QuantityBasedDiscount(string iProdType, string iProdSubType, DataTable iQuantitiesDT)
        {
            pType = iProdType;
            pSubType = iProdSubType;
            quantityDiscountsDT = iQuantitiesDT;
        }

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

        public override bool Equals(Object obj)
        {

            var other = obj as QuantityBasedDiscount;
            if (other == null) return false;

            return Equals(other);
        }

        public bool Equals(QuantityBasedDiscount other)
        {
            if (other == null)
            {
                return false;
            }

            if (ReferenceEquals(this, other))
            {
                return true;
            }

            return false;
        }
    }
}

调用该Equals方法的代码(此处为案例 2 配置):

private bool AnyUnsavedChanges()
{
    QuantityBasedDiscount copyB = new QuantityBasedDiscount(copyA.pType, copyA.pSubType, copyA.quantityDiscountsDT);

    if (copyA.Equals(copyB))
    {
         return false; //They are the same!
    }
    else
    {
         return true; //They are NOT the same!
    }

}

那么这段代码有什么问题呢?

4

2 回答 2

2

虽然不是完全重复,但您可以在这里找到所有答案:比较引用类型的两个实例的“最佳实践”是什么?. 从该线程中获取提供 Konrad 答案的片段,以免出错。


简而言之,你做错了。您不是在比较通用Equals方法中的实际值。此外,在方法内部调用==运算符也不安全Equals。像这样做:

public override int GetHashCode()
{
    return pType.GetHashCode() ^ pSubType.GetHashCode() ^ quantityDiscountsDT.GetHashCode();
    //or something similar, but fast.
}

public bool Equals(QuantityBasedDiscount other)
{
   if (ReferenceEquals(null, other))
    {
        return false;
    }

    if (ReferenceEquals(this, other))
    {
        return true;
    }

    return pType == other.pType && pSubType == other.pSubType && 
           quantityDiscountsDT == other.quantityDiscountsDT;
}

如果您的 DataTable 字段被修改,这可能仍然会给您带来不一致的结果。这一切都取决于如何==为 DataTable 实现运算符。要对此进行更多控制,您必须派生自己的 DataTable..

此外,您可能还想重载==!=运算符。对于所有检查提供的链接。

于 2012-12-23T19:27:51.717 回答
0

您必须重写 Equals 和 GetHashCode,如果您喜欢/需要语法,您还需要重载 '==' 和 '!=' 运算符。在覆盖 Equals 时,您可以首先检查 ReferenceEquals 是否返回 true,如果不是,则比较对象的内容。

我个人更喜欢在这种情况下避免使用 ReferenceEquals。

于 2012-12-23T15:37:31.770 回答