联合两个自定义类返回重复项

Question

我有两个自定义类，ChangeRequest 和 ChangeRequests，其中 ChangeRequests 可以包含许多 ChangeRequest 实例。

public class ChangeRequests : IXmlSerializable, ICloneable, IEnumerable<ChangeRequest>,
    IEquatable<ChangeRequests> { ... }

public class ChangeRequest : ICloneable, IXmlSerializable, IEquatable<ChangeRequest>
    { ... }

我正在尝试对两个 ChangeRequests 实例进行联合。但是，重复项似乎没有被删除。我的 MSTest 单元测试如下：

var cr1 = new ChangeRequest { CRID = "12" };
var crs1 = new ChangeRequests { cr1 };
var crs2 = new ChangeRequests
               {
                   cr1.Clone(),
                   new ChangeRequest { CRID = "34" }
               };
Assert.AreEqual(crs1[0], crs2[0], "First CR in both ChangeRequests should be equal");
var unionedCRs = new ChangeRequests(crs1.Union<ChangeRequest>(crs2));
ChangeRequests expected = crs2.Clone();
Assert.AreEqual(expected, unionedCRs, "Duplicates should be removed from a Union");

最后一行测试失败，并且 unionedCRs 包含两个 cr1 副本。当我尝试调试并单步执行每一行时，我在第一行的 ChangeRequest.Equals(object) 以及 ChangeRequest.Equals(ChangeRequest) 的第一行中都有一个断点），但两人都没有被击中。为什么联合包含重复的 ChangeRequest 实例？

编辑：根据要求，这里是ChangeRequests.Equals(ChangeRequests)：

public bool Equals(ChangeRequests other)
{
    if (ReferenceEquals(this, other))
    {
        return true;
    }

    return null != other && this.SequenceEqual<ChangeRequest>(other);
}

这里是ChangeRequests.Equals(object)：

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

编辑：< /strong> 我在 ChangeRequest 和 ChangeRequests 上覆盖了 GetHashCode，但仍在测试中，如果我执行 IEnumerableunionedCRsIEnum = crs1.Union(crs2);，unionedCRsIEnum 最终得到 ChangeRequest 的两个副本，CRID 为 12

编辑：某些东西必须与我的Equals或GetHashCode实现相一致，因为Assert.AreEqual(expected, unionedCRs.Distinct) (), "Distinct 应删除重复项"); 失败，expected 和 unionedCRs.Distinct() 的字符串表示形式显示 unionedCRs. Distinct() 肯定有 CR 12 的两个副本。

Answer 1

确保您的 GetHashCode 实现与您的 Equals 一致 - Enumerable.Union 方法似乎确实使用了两者。

如果您实现了其中一个但没有实现另一个，您应该会收到编译器的警告；您仍然需要确保这两种方法彼此一致。以下是规则的方便摘要：为什么在重写 Equals 方法时重写 GetHashCode 很重要？

Answer 2

我不相信 Assert.AreEqual() 检查序列的内容 - 它比较序列对象本身，它们显然不相等。

您需要的是一个 SequenceEqual() 方法，它将实际检查两个序列的内容。 这个答案可能对您有帮助。这是对类似问题的回答，描述了如何与 IEnumerable 序列进行比较。

您可以轻松地获取响应者的答案，并创建一个扩展方法以使调用看起来更像断言：

public static class AssertionExt
{
  public static bool AreSequencesEqual<T>( IEnumerable<T> expected, 
                                           IEnumerable<T> sequence )
  {
    Assert.AreEqual(expected.Count(), sequence .Count()); 

    IEnumerator<Token> e1 = expected.GetEnumerator(); 
    IEnumerator<Token> e2 = sequence .GetEnumerator(); 

    while (e1.MoveNext() && e2.MoveNext()) 
    { 
        Assert.AreEqual(e1.Current, e2.Current); 
    }
  }
}

或者您可以使用 SequenceEqual()，比较序列，意识到它不会提供任何关于哪些元素不相等的信息。

Answer 3

正如 LBushkin 所说，Assert.AreEqual 只会在序列上调用 Equals。

您可以使用 SequenceEqual 扩展方法不过：

Assert.IsTrue(expected.SequenceEqual(unionedCRs));

但是，如果失败的话，这不会提供太多信息。

您可能需要使用我们为 MoreLINQ 编写的测试代码，该代码是序列-focused - 如果序列不相等，它将指定它们的不同之处。 （我试图获取相关源文件的链接，但我的网络连接很糟糕。）