“集合已修改...”问题

Question

我有一个函数可以检查对象列表以查看它们是否已被单击并相应地触发 OnClick 事件。我相信该函数工作正常，但是我遇到了一个问题：

当我连接到 OnClick 事件之一并将该元素删除并插入到列表中的不同位置（该程序的典型功能）时，我得到“集合已修改...”错误。

我相信我明白发生了什么：

• 该函数在必要时循环遍历每个对象，触发 OnClick 事件
• 触发事件，并且对象根据挂钩函数更改列表中的位置
• 在迭代集合时会抛出异常以修改集合

我的问题是，如何允许函数迭代所有对象，在适当的时间触发必要的事件，并且仍然为用户提供操纵对象在列表中的位置的选项？

Answer 1

此类问题有两种通用解决方案：

• 复制该列表。迭代副本。
• 列出需要发生的改变。完成迭代后应用更改。

如果您想将进行更改的代码与执行循环的代码分离，“使用索引”选项听起来并不合适。

Answer 2

• 使用集合的副本而不是初始集合进行迭代。

• 如果您有一个支持索引的集合（如 List），您可以使用“for”循环，而不是使用“foreach”进行迭代。

Answer 3

如果您使用 foreach 循环来操作集合，请尝试将其替换为

for (int a = items_count - 1; a >= 0; --a)

Answer 4

您看到的行为是设计使然：

C# 的 foreach 语句
语言（对于 Visual Basic 中的每种语言）
隐藏了复杂性
枚举器。因此，使用 foreach
推荐而不是直接
操作枚举器。

枚举器可用于读取
集合中的数据，但它们
不能用于修改
底层集合。

触发同步执行的事件与在 foreach 循环中修改集合具有相同的效果。

我的偏好是使用向后循环的 for 循环，这可以避免根据是否插入某些内容来更新循环索引的条件逻辑：

for (var i = collection.Count - 1; i >= 0; i--) {
  if (condition)
    collection.Insert(i, item);
}

相应的递增循环将如下所示：

for (var i = 0; i < collection.Count; i++) {
  if (condition) {
    collection.Insert(i, item);
    i++;
  }
}

Answer 5

集合在迭代时不能被修改。您需要先标记要从集合中删除的项目，然后再将其删除。您可以根据您的要求在删除项目之前或之后触发事件。 相关的内容

此代码片段显示了我与List itemsTobeRemoved=new List();
for (var i = 0; i < collection.Count; i++) {
如果（条件）{
itemsTobeRemoved.Add(i);
}

foreach (var i = 0 in itemsTobeRemoved) {
如果（条件）{
集合.RemoveAt(i);
}