组合两个简单的相关 linq 查询

Question

我有两个查询，并且我在第二个查询中使用第一个查询的结果，

 var temp  = (ObjectTable.Where(o => o.Category == "Y"));
 var anonymousObjList = temp.Select(o => new {o, IsMax = (o.Value == temp.Max(x => x.Value))});

有没有办法将它们合并到一个查询中？

编辑： 我不能直接链接它们，因为我在第二个查询中使用 temp.Max() 。

Answer 1

为什么？将其改为三个会更清晰（也更有效）：

var temp  = (ObjectTable.Where(o => o.Category == "Y"));
int max = temp.Max(x => x.Value);
var anonymousObjList = temp.Select(o => new {o, IsMax = (o.Value == max)});

Answer 2

您可以使用查询语法并使用let关键字在一条语句中完成此操作。它只计算“max”一次，所以它就像三个单独的语句一样，只是在一行中。

var anonymousObjList = from o in ObjectTable
                       where o.Category == "Y"
                       let max = ObjectTable.Max(m => m.Value)
                       select new { o, IsMax = (o.Value == max) };

这是我唯一一次使用查询语法。您不能使用方法语法来做到这一点！

编辑：ReSharper 建议

var anonymousObjList = ObjectTable.Where(o => o.Category == "Y")
    .Select(o => new {o, max = ObjectTable.Max(m => m.Value)})
    .Select(@t => new {@t.o, IsMax = (@t.o.Value == @t.max)});

但这不是最佳选择。第一个 Select 为 ObjectTable 中的每个项目投影一个 max 属性 - 将针对每个项目评估 Max 函数。如果您使用查询语法，则仅计算一次。

同样，您只能使用查询语法来执行此操作。我不喜欢查询语法，但这使它值得，并且是我使用它的唯一情况。 ReSharper 是错误的。

Answer 3

最直接的重构可能是将所有“temp”实例替换为 temp 的值。由于该值似乎是不可变的，因此重构应该是有效的（但丑陋）：

var anonymousObjList = ObjectTable.Where(o => o.Category == "Y")
    .Select(o => new {o, IsMax = (o.Value == ObjectTable.Where(o => o.Category == "Y").Max(x => x.Value))});

正如已经指出的那样，该查询实际上比原始查询没有任何优势，因为查询使用延迟执行并且可以构建。我实际上建议进一步拆分查询：

var temp  = (ObjectTable.Where(o => o.Category == "Y")); 
var maxValue = temp.Max(x => x.Value);    
var anonymousObjList = temp.Select(o => new {o, IsMax = (o.Value == maxValue)});

这比原来的更好，因为每次调用“Max”都会导致整个数据集的另一次迭代。由于它是在原始的 Select 中调用的，因此 Max 被调用了 n 次。这使得原来的 O(n^2)！