从通用List 中删除项目

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

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

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

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

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