从通用 List<t> 中删除项目答案

【问题标题】：removing items from a generic List<t>从通用 List<t> 中删除项目
【发布时间】：2010-06-08 08:37:49
【问题描述】：

我有以下方法，我希望从我的收藏中删除与产品 ID 匹配的项目。似乎相当简单，但我有一个例外。基本上我的收藏不同步了。那么从集合中删除项目的最佳方法是什么。

public void RemoveOrderItem(Model.Order currentOrder, int productId)
{

    foreach (var orderItem in currentOrder.OrderItems)
    {
        if (orderItem.Product.Id == productId)
        {
            currentOrder.OrderItems.Remove(orderItem);
        }
    }
}

异常详细信息：System.InvalidOperationException：集合已修改；枚举操作可能无法执行

【问题讨论】：

标签： c# linq generics list

【解决方案1】：

在循环中修改集合不起作用。为了解决这个问题，List 提供了一些允许“批量”修改集合的方法。在你的情况下，使用：

currentOrder.OrderItems.RemoveAll(x => x.Product.Id == productId)

【讨论】：

感谢康拉德，奇怪的是我无法让 lamda 工作。它无法识别“x.Product.Id”部分。奇怪因为以下工作 var query = from x in currentOrder.OrderItems where x.Product.Id == productId select x;集合类型为 ISet。
好的，我回答了我自己的问题 :) 我已将其更改为 List

【解决方案2】：

您不能在迭代集合时对其进行修改。只需使用普通的 for 循环而不是 foreach 循环。

【讨论】：

:) 这对我来说是最好的解决方案

【解决方案3】：

通过这种循环方式，您无法删除项目，因为它在集合中会跟踪存储的项目。

简单的方法：

   authorsList.RemoveAll(x => x.ProductId == productId);

或

   authorsList = authorsList.Where(x => x.ProductId!= productId).ToList();

【讨论】：

【解决方案4】：

您无法从正在迭代的集合中删除项目，您可以跟踪 orderItem，然后在完成循环后将其删除

【讨论】：

【解决方案5】：

正如您所意识到的，当您循环访问某个项目时，您无法从集合中移除它。我确信有人能够提供更简洁的 LINQ 解决方案，但以下内容应该可以帮助您开始：

public void RemoveOrderItem(Model.Order currentOrder, int productId)
{
    var selectedOrderItem = null;
    foreach (var orderItem in currentOrder.OrderItems)
    {
        if (orderItem.Product.Id == productId)
        {
            selectedOrderItem = orderItem;
            break;
        }
    }

    if(selectedOrderItem != null)
        currentOrder.OrderItems.Remove(selectedOrderItem);
}

【讨论】：

Konrad Rudolph 给出了 LINQ 解决方案
请注意，List<T>.RemoveAll 在 .NET 2.0 中可用，并且在技术上不是 LINQ，尽管它看起来很相似。对于那些必须保持 Windows 2000 兼容性的可怜人来说，这对于 LINQ 来说是一个巨大的优势。

【解决方案6】：

“foreach”提供集合的“Forward-only read-only”迭代。

作为一种解决方法，您可以将引用复制到另一个集合，然后迭代复制的集合并从原始集合中删除项目。

【讨论】：