在 linq 查询中为商品添加订单号

Question

我有以下 Linq 查询。 transactionData 是一个 IEnumerable。

var totalTransactions = 0;
viewModel.GroupedTransactions = transactionData
    .GroupBy(x => new { DocumentId = x.DocumentId ?? "Un Documented" })
    .Select(x => new GroupedTransaction
    {
        DocumentId = x.Key.DocumentId,
        Transactions = x.Select(y => new Transaction
        {
            Amount = y.CommitAmount,
            ActivityType = y.ActivityType,
            Number = totalTransactions++
        })
    })
    .OrderBy(x => x.DocumentId);

我试图将交易记录上的数字设置为递增数字。 这是行不通的，会在数字上留下空白。

查询后我还尝试了以下操作。

foreach (var item in viewModel.GroupedTransactions.SelectMany(x => x.Transactions))
{
     item.Number = totalTransactions;
     totalTransactions++;
}

这甚至没有更新 Number 值。 我做错了什么，或者是否有更简单的方法，使用简洁的 linq 扩展方法？

Answer 1

问题是您正在关闭变量totalTransactions，您必须创建一个本地副本才能使用。 检查关闭循环变量被认为是有害的以获得更详细的解释。

像这样的事情应该有效：

var totalTransactions = 0;
viewModel.GroupedTransactions = transactionData
    .GroupBy(x => new { DocumentId = x.DocumentId ?? "Un Documented" })
    .Select(x => 
    {
      new GroupedTransaction()
      {
        DocumentId = x.Key.DocumentId,
        Transactions = x.Select(y => 
        {
          var currentTransactionId = totalTransactions;
          totalTransactions++;

          return new Transaction
          {
            Amount = y.CommitAmount,
            ActivityType = y.ActivityType,
            Number = currentTransactionId 
          }
        })
      }
    })
    .OrderBy(x => x.DocumentId);

对于使用 foreach 循环的第二种方法 - 您实际上是使用 SelectMany() 创建一个新的枚举，随后将其丢弃：

foreach (var item in viewModel.GroupedTransactions.SelectMany(x => x.Transactions))
{
     item.Number = totalTransactions;
     totalTransactions++;
}

相反，您必须强制对集合进行急切评估通过使用 ToList() 创建一个可以安全修改的集合。

var transactions = viewModel.GroupedTransactions
                            .SelectMany(x => x.Transactions)
                            .ToList();
foreach (var item in transactions)
{
     item.Number = totalTransactions;
     totalTransactions++;
}

Answer 2

另一种思考方式是，您有两个序列：

1. 事务
2. “自动递增”索引

并且您想要获得一个序列，即带有 id 的事务。当我们想要组合两个序列时，我们可以使用 Zip 运算符：

viewModel.GroupedTransactions = transactionData     
    .GroupBy(x => new { DocumentId = x.DocumentId ?? "Un Documented" })
    .Zip(Enumerable.Range(0, int.MaxValue), (x, index) => new GroupedTransaction     
    {         
        DocumentId = x.Key.DocumentId,         
        Transactions = x.Select(y => new Transaction         
        {             
            Amount = y.CommitAmount,             
            ActivityType = y.ActivityType,             
            Number = index         
        })     
    })     
    .OrderBy(x => x.DocumentId); 

这是您想要的吗？

Zip 组合两个序列，直到到达其中一个序列的末尾。这就是为什么可以使用 Enumberable.Range 来获取比我们实际需要的更大范围的数字。