2

我想检查两个具有one-to-many内部关系的实体之间的相等性。

所以很明显我覆盖了这个Object.Equals方法，但是我得到了CS0659编译器警告：'class' overrides Object.Equals(object o) but does not override Object.GetHashCode().

我覆盖了Object.GetHashCode，但是 Resharper 告诉我该GetHashCode方法应该为所有对象生命周期返回相同的结果，并将用于可变对象。（文档）

public class Computer
{
    public long Id { get; set; }
    public ICollection<GPU> GPUs { get; set; } = new List<GPU>();

    public override bool Equals(object obj)
    {
        return obj is Computer computer &&
               GPUs.All(computer.GPUs.Contains);
    }

    public override int GetHashCode()
    {
        return HashCode.Combine(GPUs);
    }
}

public class GPU
{
    public long Id { get; set; }
    public int? Cores { get; set; } = null;

    public override bool Equals(object obj)
    {
        return obj is GPU gpu &&
               Cores == gpu.Cores;
    }

    public override int GetHashCode()
    {
        return HashCode.Combine(Cores);
    }
}

我不知道我应该更喜欢什么：

  • 覆盖Equals方法而不覆盖GetHashCode，或
  • 用不可变数据覆盖GetHashCode？
4

1 回答 1

5

Entity Framework 使用自己的智能方法来检测对象相等性。例如，如果您调用SaveChanges：获取对象的值与更新对象的值匹配，以检测是否需要 SQL 更新。

我不确定您对相等性的定义是否会与此相等性检查混淆，从而导致数据库中更新一些未更改的项目，或者更糟糕的是，数据库中未更新某些更改的数据。

数据库平等

请记住，您的实体类（您放在 中的类DbSet<...>）代表数据库中的表以及表之间的关系。

什么时候从你的数据库中提取的两个项目被认为代表同一个对象？是不是当它们具有相同的值时？我们不能在一个数据库中拥有两个名为“John Doe”、出生于 7 月 4 日的人吗？

您可以用来检测Persons从数据库中提取的两个表示相同的唯一方法Person是检查 Id。一些非主键值不同的事实仅告诉您更改的数据没有在数据库中更新，而不是它是不同的Person.

覆盖 Equals 与创建 EqualityComparer

我的建议是，让你的表表示尽可能简单：只有表的列（非虚拟属性）和表之间的关系（虚拟属性）。没有成员，没有方法，什么都没有。

如果您需要额外的功能，请创建类的扩展功能。如果您需要非标准的相等比较方法，请创建一个单独的相等比较器。您的类的用户可以决定是否要使用默认比较方法或您的特殊比较方法。

这都可以与各种字符串比较器进行比较：StringComparer.OrdinalIgnorCase,StringComparer.InvariantCulture等。

回到你的问题

在我看来，您需要一个不检查 Id 值的 Gpu 比较器：两个具有不同 Id 的项目，但其他属性的相同值被认为是相等的。

class GpuComparer : EqualityComparer<Gpu>
{
    public static IEqualityComparer<Gpu> IgnoreIdComparer {get;} = new GpuComparer()

    public override bool Equals(Gpu x, Gpu y)
    {
        if (x == null) return y == null; // true if both null, false if x null but y not
        if (y == null) return false;     // because x not null
        if (Object.ReferenceEquals(x, y)) return true;
        if (x.GetType() != y.GetType()) return false;

        // if here, we know x and y both not null, and of same type.
        // compare all properties for equality
        return x.Cores == y.Cores;
    }
    public override int GetHasCode(Gpu x)
    {
        if (x == null) throw new ArgumentNullException(nameof(x));

         // note: I want a different Hash for x.Cores == null than x.Cores == 0!

         return (x.Cores.HasValue) ? return x.Cores.Value.GetHashCode() : -78546;
         // -78546 is just a value I expect that is not used often as Cores;
    }
}

请注意，我添加了相同类型的测试，因为如果 y 是 Gpu 的派生类，并且您会忽略它们不是同一类型，那么可能是 Equals(x, y)，但不是 Equals(y, x)，这是相等函数的先决条件之一

用法：

IEqualityComparer<Gpu> gpuIgnoreIdComparer = GpuComparer.IgnoreIdComparer;
Gpu x = new Gpu {Id = 0, Cores = null}
Gpu y = new Gpu {Id = 1, Cores = null}

bool sameExceptForId = gpuIgnoreIdComparer.Equals(x, y);

x 和 y 将被视为相等

HashSet<Gpu> hashSetIgnoringIds = new HashSet<Gpu>(GpuComparer.IgnoreIdComparer);
hashSetIgnoringIds.Add(x);
bool containsY = hashSetIgnoringIds.Contains(y); // expect true

计算机的比较器将是类似的。除了您忘记检查 null 和类型之外，我还发现您希望进行相等性检查的方式还存在一些其他问题：

  • 可以将 null 分配给您的 Gpus 集合。你必须解决这个问题，它不会引发异常。具有零 Gpus 的计算机是否等于具有零 Gpus 的计算机？
  • 显然 Gpus 的顺序对你来说并不重要： [1, 3] 等于 [3, 1]
  • 显然某个GPU出现的次数并不重要：[1,1,3]等于[1,3,3]？

.

class IgnoreIdComputerComparer : EqualityComparer<Computer>
{
    public static IEqualityComparer NoIdComparer {get} = new IgnoreIdComputerCompare();


    public override bool (Computer x, Computer y)
    {
        if (x == null) return y == null;not null
        if (y == null) return false;
        if (Object.ReferenceEquals(x, y)) return true;
        if (x.GetType() != y.GetType())  return false;

        // equal if both GPU collections null or empty,
        // or any element in X.Gpu is also in Y.Gpu ignoring duplicates
        // using the Gpu IgnoreIdComparer
        if (x.Gpus == null || x.Gpus.Count == 0)
            return y.Gpus == null || y.Gpus.Count == 0;

        // equal if same elements, ignoring duplicates:
        HashSet<Gpu> xGpus = new HashSet<Gpu>(x, GpuComparer.IgnoreIdComparer);
        return xGpush.EqualSet(y);
    }

    public override int GetHashCode(Computer x)
    {
        if (x == null) throw new ArgumentNullException(nameof(x));

        if (x.Gpus == null || x.Gpus.Count == 0) return -784120;

         HashSet<Gpu> xGpus = new HashSet<Gpu>(x, GpuComparer.IgnoreIdComparer);
         return xGpus.Sum(gpu => gpu);
    }
}

TODO：如果您将使用大量 Gpus，请考虑使用更智能的 GetHashCode

于 2019-01-02T12:23:33.203 回答