收藏已修改；枚举操作可能无法在 ArrayList 中执行 [重复]

Question

我正在尝试从 ArrayList 中删除一个项目，但我得到了这个异常：
Collection was modified; enumeration operation may not execute.

有什么想法吗？

Answer 1

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

• 将List<T> 和RemoveAll 与谓词一起使用

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

  for(int i = list.Count - 1; i >= 0; i--) {
      if({some test}) 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<T> 上的扩展方法来实现。 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<T> 的实现类似。

    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

我同意我在这篇文章中读到的一些观点，并将它们纳入我的解决方案中，以解决与原始帖子完全相同的问题。

也就是说，我欣赏的 cmets 是：

• “除非您使用 .NET 1.0 或 1.1，否则请使用 List<T> 而不是 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);
}