从通用列表中删除项目<t>

Question

我有以下方法，我希望从我的集合中删除与产品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：Collection已修改; 枚举操作可能无法执行

Answer 1

修改循环内的集合不起作用。 为了解决这个问题， List有一些方法可以对集合进行“批量”修改。 在您的情况下，使用：

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

Answer 2

迭代时无法修改集合。 只需使用普通for循环而不是foreach循环。

Answer 3

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

简单的方法：

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

要么

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

Answer 4

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

Answer 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);
}

Answer 6

“foreach”提供集合的“仅向前只读”迭代。

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