如何优化此 LINQ 语句以提高速度？

Question

以下 LINQ 语句查找具有特定属性重复值的项，按计数对它们进行分组，然后进行另一个分组以获取布尔值。

我只是好奇如何改进它，这似乎有点浪费，因为这是对象模型验证库的一部分，我很想尽快得到它。

执行速度是重中之重，但欢迎提出任何其他建议。

var grouped = from g2 in
                 (from i in item.ParentList
                  where _filter(i)
                  group i by propGetter(i) into g
                  select new { Count = g.Count(), Items = g })
              group g2 by g2.Count == 1 into g3
              select new { IsUnique = g3.Key, Items = g3 };

foreach (var g in grouped)
{
    foreach (var grp in g.Items)
    {
        foreach (var itm in grp.Items)
        {
            if (g.IsUnique == false)
                itm.AddPropertyError(_propertyName, (int)Validations.Unique, _message);
            else
                itm.RemovePropertyError(_propertyName, (int)Validations.Unique);
        }
    }
}

Answer 1

正如 Oskar 所说，您可能不希望优化查询以提高速度，除非分析表明它会导致问题。 Premature optimization is the root of all evil。如果你想优化查询的可读性，这里有一种简化第二部分的方法：

var items = grouped
    .SelectMany(group => group.Items)
    .SelectMany(group => group.Items)

foreach (var item in items)
{
    ...
}

编辑您的评论：啊，我没有注意到您在最里面的循环中引用了g.IsUnique。这是解决该问题的一种方法，无需 3 级缩进，但它可能不是最好的方法：

var uniqueItems = grouped
    .Where(group => group.IsUnique)
    .SelectMany(group => group.Items)
    .SelectMany(group => group.Items)

var nonUniqueItems = grouped
    .Where(group => !group.IsUnique)
    .SelectMany(group => group.Items)
    .SelectMany(group => group.Items)

foreach (var item in uniqueItems)
{
    ...
}

foreach (var item in nonUniqueItems)
{
    ...
}