在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++; 
} 

這甚至沒有更新數值。 我做錯了什麼，或者有一個簡單的方法，用一個整潔的linq擴展方法？

Answer 1

問題是您正在關閉變量totalTransactions，您必須創建一個本地副本才能使用。 Check Closing over the loop variable considered harmful獲取更詳細的解釋。

像這樣的東西應該工作：

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. 「自動遞增」指數

而且你想獲得一個序列，交易與ids。當我們想兩個序列結合，我們可以使用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將兩個序列組合到一個序列的末尾。這就是爲什麼它可以允許。範圍比我們實際需要的範圍大得多。