如果我使用 linq 进行投影而不使用范围变量，正确的语法是什么？

Question

我有一个对大量数据进行求和和聚合的查询，如下所示：

var anonType = from x in collection
               let Cars = collection.Where(c=>c.Code == "Cars")
               let Trucks = collection.Where(c=>c.Code == "Trucks")
               select new { 
                           Total = collection.Sum(v=>v.Amount),
                           CarValue = Cars.Sum(v=>v.Amout),
                           TruckValue = Trucks.Sum(v=>v.Amount),
                           CarCount = Cars.Count(),
                           TruckCount = Trucks.Count()
               };

我发现我必须声明范围变量 x 真的很奇怪，特别是如果我不使用它的话。那么，我是否做错了什么，或者我应该遵循不同的格式？

Answer 1

我可能是错的，但从您的使用情况来看，我认为您无论如何都不想对集合进行传统的查询表达式语法查询，因为您似乎只是在寻找聚合。按照您编写的方式，您将提取聚合数据的多个副本，因为您正在为集合中的每个项目执行此操作。如果您愿意，您可以像这样拆分查询（放入示例属性）

var values = collection.Where(c => c.Code == "A");
var anonType = new
               {
                   Sum = values.Sum(v => v.Amount),
                   MinimumStartDate = values.Min(v => v.StartDate),
                   Count = values.Count()
               };

Answer 2

无论循环结构如何，都可以声明范围变量：

foreach(var x in collection)

or

for(var index = 0; index < collection.Count; index++)

或

var index = 0;

while(index < collection.Count)
{
    //...

    index++;
}

查询没有什么不同。只是不要使用该变量，它不会造成任何伤害。

Answer 3

那么，我做错了什么吗？

你的查询不好。对于集合中的每个元素，您将枚举集合 5 次（成本 = 5*n^2）。

我应该遵循不同的格式吗？

您可以枚举集合 5 次（成本 = 5n）。

IEnumerable<X> cars = collection.Where(c => c.Code == "Cars");
IEnumerable<X> trucks = collection.Where(c => c.Code == "Trucks");

var myTotals = new 
{
  Total = collection.Sum(v => v.Amount),
  CarValue = cars.Sum(v => v.Amount),
  TruckValue = trucks.Sum(v => v.Amount,
  CarCount = cars.Count(),
  TruckCount = trucks.Count()
};