InvalidOperationException VB.Net 通用集合？

Question

我知道修改您正在迭代的集合通常是一个很大的禁忌，但不幸的是我没有设计我要修改的代码。整个地方都完成了以下操作：

for each log in Logs
   logs.Delete(log.LogId)
Next

删除几乎只是从数据库中删除日志并将其从集合中删除。以前，Logs 对象使用非通用集合。我将其更改为使用 Collection(Of Log)，以便我可以 LINQify 对象。现在，每次我在第一次删除后调用 next/.MoveNext 时，都会发生以下错误： 无效操作异常： “集合已修改；枚举操作可能无法执行。”

我明白为什么我收到错误，但我不明白为什么非通用版本从未发生过这种情况。有办法解决这个错误吗？我真的没有办法花时间去改变像这样删除日志的每个地方（代码库很大）。我想删除删除函数中的代码，它将其从当前集合中删除，因为我假设没有代码在完成后对集合执行任何操作，但您知道假设时会发生什么。

Answer 1

简而言之，您的问题：

        Collection<object> stuff = new Collection<object> { 1, 2, 3, 4 };

        foreach (var o in stuff)
            stuff.Remove(o); // causes exception

两种解决方案：

1. 制作集合的副本并对其进行迭代。

    foreach (var o in stuff.ToArray())
           stuff.Remove(o); // 不会引发异常
   

2. 向后迭代集合。

    for (int i = stuff.Count - 1; i >= 0; i--)
           stuff.RemoveAt(i); // 不会引发异常
   

   （这类似于 logs.Delete(logs[i].LogId);）

对于 C# 很抱歉，但概念非常清楚。

Answer 2

您不需要循环来删除 Collection(Of T) 中的所有条目，只需使用 清除方法。

Logs.Clear

Answer 3

查尔斯的回应让我考虑创建一个副本，而不是迭代真正的集合。不幸的是，他的解决方案需要更改删除到处都是活动日志的代码。这是我解决问题的方法，该问题似乎工作得很好（伪代码）：

之前：

Public Function GetEnumerator() as IEnumerator
    Return col.GetEnumerator
End Function

之后：

Public Function GetEnumerator() As IEnumerator
    Return col.ToList().GetEnumerator()
End Function