在这个例子中如何避免写三个循环？

Question

在下面的代码示例中，如何避免编写 3 个循环。我不确定如何保持代码可读性并仍然对其进行优化。这不是我正在处理的确切功能，但为简洁起见，我对其进行了更新，供大家阅读。

int OrderQuantity = 5;
List<LineItem> Items = GetLineItems();
List<int> UniqueOrderIDs = new List<int>();

foreach (LineItem i in Items) {
    if (UniqueOrderIDs.Contains(i.OrderID) == false) {
        PurchaseOrder Order = GetOrder(i.OrderId);
        Order.ModifiedDate = Now;
        UpdateOrder(Order);
        UniqueOrderIDs.Add(i.OrderID);
    }
}

foreach (int id in UniqueOrderIDs) {
    decimal TaxableAmount = 0;

    foreach (LineItem i in Items) {
        If(i.OrderID == id){
            i.OrderQuantity = OrderQuantity;
            UpdateItem(i)

            TaxableAmount += i.Cost;
        }
    }

    UpdateTaxAmount(id, TaxableAmount);
}

Answer 1

你可以这样做（我在这里省略了边界检查）

List<LineItem> items = GetLineItems(OrderID);
var sorted = items.OrderBy(i=>i.OrderID);

int id = sorted.First().OrderID; //get 1st element's id

foreach (LineItem i in sorted.Skip(1)) //skip 1st one
{
    if(i.OrderID == id)
    {
        i.OrderQuantity = OrderQuantity;
        UpdateItem(i)    
        TaxableAmount += i.Cost;            
    }
    else 
       id = i.OrderID;
}

....

在简单地迭代它之后，您可以按OrderID 对LineItem 列表进行排序，虽然它是相同的，但您可以做任何您需要做的事情。

可能会有人选择LINQ 解决方案，但在我看来，这看起来更简单明了，然后LINQ 解决方案可能就是在这种情况下。

Answer 2

我可能遗漏了一些东西（在这里迟到了），但你为什么不能这样做：

var orderIdAndTaxableAmount = new Dictionary<int, decimal>();

foreach (LineItem i in Items) {
    if (UniqueOrderIDs.Contains(i.OrderID) == false) {
        PurchaseOrder Order = GetOrder(i.OrderId);
        Order.ModifiedDate = Now;
        UpdateOrder(Order);
        UniqueOrderIDs.Add(i.OrderID);
    }

    i.OrderQuantity = OrderQuantity;
    UpdateItem(i);

    if (!orderIdAndTaxableAmount.ContainsKey(i.OrderId))
    {
        orderIdAndTaxableAmount.Add(i.OrderId, 0.0);
    }
    orderIdAndTaxableAmount[i.OrderId] += i.Cost;
}

foreach (var kvp in orderIdAndTaxableAmount)
{
    UpdateTaxAmount(kvp.Key, kvp.Value);
}