VB.NET For-Each 循环错误

Question

在编写一些代码来加载应用程序的设置时遇到问题 - 以下代码会产生 InvalidOperationException 错误：

For Each c As MapInfo In CurrentMaps
    AddMap(c, False)
Next

调试器突出显示 Next 部分，并从 MSDN 页面上的错误，看起来这是因为集合（CurrentMaps）与上次碰撞枚举器（处理For Each）时相比发生了某种变化。

但不确定为什么会发生这种情况，因为 AddMap 采用 MapInfo ByVal 并且不触及输入（因此集合不会更改）。

我将其更改为这样，效果非常好：

For i As Integer = 0 To CurrentMaps.Count - 1
    AddMap(CurrentMaps(i), False)
Next

那些在功能上应该是相同的，对吧？知道为什么 For Each 不起作用吗？

编辑：

啊，明白了——问题是我使用 CurrentMaps 作为临时变量（跟踪当前地图的集合），而 AddMap 通常在应用程序中使用来添加新地图（其中意味着更新 UI 并将其添加到 CurrentMaps）。

在这里（加载应用程序时），我只是使用 AddMap 来更新 UI，但是将项目添加到 CurrentMaps 的子调用仍然存在，因此它确实在修改 CurrentMaps。

感谢大家的意见！

Answer 1

不，它们的工作方式不应该相同。基本上，当你迭代集合时，你不能修改它 - 看起来这就是正在发生的事情。

来自 List.GetEnumerator() 文档：

只要集合保持不变，枚举器就保持有效。如果对集合进行更改，例如添加、修改或删除元素，则枚举器将不可恢复地失效，并且其行为未定义。

您还没有显示您的 AddMap 方法的作用，但我怀疑它向 CurrentMaps 添加了一个映射，从而使迭代器无效。

如果您可以向我们提供有关 AddMap 的确切更多信息，我们可以为您提供更多帮助。 False 是什么意思？

Answer 2

不，这些功能的工作方式非常不同。第一个版本使用 .Net IEnumerable(Of T) + IEnumerator(Of T) 模式。使用此模式枚举集合时改变集合是不合法的。许多 BCL 类型强制执行此要求。

第二个简单地使用计数+索引模式。如果边界保持正确，则在使用此模式进行枚举时改变集合是合法的。

这里的问题几乎可以肯定是 AddMap 间接改变了 CurrentMaps 集合，从而产生了这个错误。

Answer 3

啊，明白了 - 问题是我使用 CurrentMaps 作为临时变量（跟踪当前地图的集合），并且 AddMap 通常在应用程序内使用来添加新地图（这意味着更新 UI 并将其添加到 CurrentMaps）。

在这里（加载应用程序时），我只是使用 AddMap 来更新 UI，但是将项目添加到 CurrentMaps 的子调用仍然存在，所以它确实是修改CurrentMaps。

感谢大家的意见！

Answer 4

Private Maps as list(of MapInfo)

For each iMaps as MapInfo in CurrentMaps

  Maps.add(iMaps)

Next