集合已修改； ArrayList 中不能执行枚举操作

Question

我试图从 ArrayList 中删除一个项目，但出现此异常：
集合已修改；枚举操作可能无法执行。

有什么想法吗？

Answer 1

您正在 foreach 期间删除该项目，是吗？很简单，你不能。这里有一些常见的选项：

• 使用 List 和 RemoveAll 以及谓词

• 按索引向后迭代，删除匹配项

  for(int i = list.Count - 1; i >= 0; i--) {
      if({一些测试}) list.RemoveAt(i);
  }
  

• 使用foreach，并将匹配项放入第二个列表中；现在枚举第二个列表并从第一个列表中删除这些项目（如果您明白我的意思）

Answer 2

这是一个示例（对于任何拼写错误，我们深表歉意）

var itemsToRemove = new ArrayList();  // should use generic List if you can

foreach (var item in originalArrayList) {
  if (...) {
    itemsToRemove.Add(item);
  }
}

foreach (var item in itemsToRemove) {
  originalArrayList.Remove(item);
}

或者如果您使用的是 3.5，Linq 会使第一步变得更容易：

itemsToRemove = originalArrayList
  .Where(item => ...)
  .ToArray();

foreach (var item in itemsToRemove) {
  originalArrayList.Remove(item);
}

将“...”替换为确定是否应删除项目的条件。

Answer 3

一种方法是将要删除的项目添加到新列表中。然后检查并删除这些项目。

Answer 4

我喜欢使用 for 循环向后迭代，但这与 foreach 相比可能会变得乏味。我喜欢的一种解决方案是创建一个向后遍历列表的枚举器。您可以将其实现为 ArrayList 或 List 上的扩展方法。 ArrayList 的实现如下。

    public static IEnumerable GetRemoveSafeEnumerator(this ArrayList list)
    {
        for (int i = list.Count - 1; i >= 0; i--)
        {
            // Reset the value of i if it is invalid.
            // This occurs when more than one item
            // is removed from the list during the enumeration.
            if (i >= list.Count)
            {
                if (list.Count == 0)
                    yield break;

                i = list.Count - 1;
            }

            yield return list[i];
        }
    }

List 的实现类似。

    public static IEnumerable<T> GetRemoveSafeEnumerator<T>(this List<T> list)
    {
        for (int i = list.Count - 1; i >= 0; i--)
        {
            // Reset the value of i if it is invalid.
            // This occurs when more than one item
            // is removed from the list during the enumeration.
            if (i >= list.Count)
            {
                if (list.Count == 0)
                    yield break;

                i = list.Count - 1;
            }

            yield return list[i];
        }
    }

下面的示例使用枚举器从 ArrayList 中删除所有偶数。

    ArrayList list = new ArrayList() {1, 2, 3, 4, 5, 6, 7, 8, 9, 10};

    foreach (int item in list.GetRemoveSafeEnumerator())
    {
        if (item % 2 == 0)
            list.Remove(item);
    }

Answer 5

不要在迭代列表的循环内修改列表。

相反，使用带有索引的 for() 或 while() ，向后浏览列表。 （这将使您可以删除内容而不会获得无效索引。）

var foo = new List<Bar>();

for(int i = foo.Count-1; i >= 0; --i)
{
  var item = foo[i];
  // do something with item
}

Answer 6

我错过了什么吗？如果我错了，有人纠正我。

list.RemoveAll(s => s.Name == "Fred");

Answer 7

使用带有数字索引的 for() 循环代替 foreach()。

Answer 8

我同意我在这篇文章中读到的几个观点，并且我已将它们合并到我的解决方案中，以解决与原始帖子完全相同的问题。

也就是说，我欣赏的评论是：

• “除非您使用 .NET 1.0 或 1.1，否则请使用 List 而不是 ArrayList。”

• "另外，将项目添加到删除到新列表，然后浏览并删除这些项目。”
  ..就我而言，我刚刚创建了一个新列表并用有效数据值填充它。

例如

private List<string> managedLocationIDList = new List<string>();
string managedLocationIDs = ";1321;1235;;" // user input, should be semicolon seperated list of values

managedLocationIDList.AddRange(managedLocationIDs.Split(new char[] { ';' }));
List<string> checkLocationIDs = new List<string>();

// Remove any duplicate ID's and cleanup the string holding the list if ID's
Functions helper = new Functions();
checkLocationIDs = helper.ParseList(managedLocationIDList);

...
public List<string> ParseList(List<string> checkList)
{
    List<string> verifiedList = new List<string>();

    foreach (string listItem in checkList)
    if (!verifiedList.Contains(listItem.Trim()) && listItem != string.Empty)
        verifiedList.Add(listItem.Trim());

    verifiedList.Sort();
    return verifiedList;
}        

Answer 9

使用 ArrayList 你也可以尝试这样

ArrayList arraylist = ... // myobject data list

ArrayList temp = (ArrayList)arraylist.Clone();

foreach (var item in temp)
{
      if (...)
         arraylist.Remove(item);
}