断言列表中字段的唯一性

Question

我在 C# 中创建了一个列表，我想进行测试以查看 Id 字段的所有值是否都是唯一的。

public static List<RestaurantReview> _reviews = new List<RestaurantReview>
{
    new RestaurantReview
    {
        Id = 1,
        Name = "McDonalds",
    },
    new RestaurantReview
    {
        Id = 2,
        Name = "Burger King",
    },
}

因为我做了一些调试，我发现它正在通过列表运行，但我没有得到正确的测试值。有人可以解释一下我在这里做错了什么吗？

[TestMethod()]
public void CheckReviewIds()
{
    var FirstReview = ReviewsController._reviews.First();
    bool AllUniqueIds = ReviewsController._reviews.All(s => s.Id == FirstReview.Id);

    Assert.IsFalse(AllUniqueIds);

}

提前致谢。

Answer 1

另一个在时间方面具有更好性能的解决方案（特别是如果您拥有的数据集很大）是使用HashSet，如下所示：

bool IsAllUnique<T>(IEnumerable<T> values)
{
    HashSet<T> hash_set = new HashSet<T>();

    return values.All(x => hash_set.Add(x));
}

然后你可以像这样使用它：

bool unique = IsAllUnique(ReviewsController._reviews.Select(x => x.Id));

此解决方案取决于HashSet.Add 将在我们尝试添加的值已经存在时返回 false 的事实。

此解决方案具有更好性能的一个原因是，如果它检测到重复项，则不必遍历列表中的其余项。

Answer 2

您正在检查所有值是否不等于第一个值。如果值是例如[1, 2, 3, 3]，则可能是这种情况，没有一个等于第一个但它本身，但3 == 3。

相反，您可以GroupBy 按值对它们进行分组，然后检查它们是否不同。我假设性能在这里不是一个大问题（如果列表少于我假设的 100000 个项目，就是这种情况）：

 ReviewsController._reviews.GroupBy(x => x.Id).Count() == ReviewsController._reviews.Count; 

---

请注意，测试组件的内部状态可能不是最好的主意，而是测试它公开的 API。否则，您通过单元测试定义的合同受到您的实施细节的限制。不过这最后一段只是个人意见。

Answer 3

我发现验证这一点的最简单方法是计算所有不同的值并将它们与实际值的数量进行比较：

var actual = ReviewsController._reviews.Select(r => r.Id).Distinct().Count();

var expected = ReviewsController._reviews.Count();
Assert.Equal(expected, actual);