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”提供了一个集合的“仅向前只读”迭代。

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

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

6 个答案: