在 foreach循环中删除集合元素

1. 引言

几乎每个刚开始学C#编程的人，迟早都会遇到同一个问题：有个集合（比如对象列表），需要按某个条件删掉不需要的元素。听起来很简单，手就想用熟悉又方便的foreach循环，毕竟这是最“安全”“友好”的遍历方式。但突然，在最意想不到的时候，冒出一个神秘的运行时错误，简单例子里没见过——程序直接卡住。

咱们来搞清楚为啥会这样，集合和迭代器“底层”到底怎么回事，怎么优雅地删元素才能避免踩坑和bug。

为什么foreach不适合删元素

想象一下，有一队人排队（这就是我们的集合）。你一个个问过去：“你留下还是划掉？”如果你在遍历时直接划掉某个人，队伍就会整体往前挪，人都换了位置，你的“下一个人就是下一个”这个计划立马崩了。可能有人没问到，也可能有人被问两次。

C#示例：

List<string> names = new List<string> { "安东", "鲍里斯", "维卡", "格里沙" };

foreach (string name in names)
{
    if (name.StartsWith("V"))
        names.Remove(name); // 嘭！InvalidOperationException
}

当程序走到"维卡"并决定删掉她时，内部迭代器就“失联”了——你会收到提示：
InvalidOperationException: 集合已被修改；枚举操作可能无法执行。

这不是C#“任性”，而是为了保护你不被难以发现的bug和数据结构损坏坑到。

2. 为什么这么简单的代码不行？

内部到底怎么实现的？

你写foreach循环时，编译器会生成一个特殊对象——迭代器（IEnumerator），它会记住集合开始时有多少元素、当前“指针”在哪，并且死死盯着集合遍历时不能变。

只要你在foreach里删或加元素，这个契约就破坏了。为啥？因为删了元素后，索引全变了，迭代器就找不到下一个元素了。有人会被跳过，有人会被重复——结果就是一团乱。所以.NET一发现集合变了，立马抛异常。

“硬删”会发生什么

假如你写了这样的程序：

List<int> numbers = new List<int> { 1, 2, 3, 4, 5, 6 };
foreach (int x in numbers)
{
    if (x % 2 == 0)
        numbers.Remove(x);
}

看起来很合理：遍历所有数字，删掉偶数。但第二轮循环就报错——“集合在枚举时被修改了”。

有时候你可能想“硬刚”这个警告，试试能不能蒙混过关。可即使没报错，结果也会很迷。比如你可能会“跳过”某些元素，或者没删干净。

3. 那到底怎么做才对？

方法1：反向for循环

问题在于，删元素时后面的元素会往前挪，如果你从头往后遍历，索引很容易乱，导致漏删。最靠谱的做法是从后往前。

List<string> names = new List<string> { "安东", "鲍里斯", "维卡", "格里沙" };

for (int i = names.Count - 1; i >= 0; i--)
{
    if (names[i].StartsWith("V"))
        names.RemoveAt(i);
}

这样每次删完，后面的元素都往前挪，但还没处理的索引不会受影响。结果不会漏删。

方法2：过滤，创建新列表

有时候更简单（而且通常更快）的做法是遍历一遍，把要保留的元素收集起来，最后用新列表替换原来的。

var names = new List<string> { "安东", "鲍里斯", "维卡", "格里沙" };
names = names.Where(name => !name.StartsWith("V")).ToList();
// 最后只剩下"安东"和"格里沙"

这种方法适合集合不大，或者你不在乎原对象引用的情况。

方法3：用集合自带的专用方法

如果你用的是经典的List<T>，按条件删除有个很方便的方法：

names.RemoveAll(name => name.StartsWith("V"));

内部会帮你处理好一切，代码又短又清楚。

方法4：先收集要删的

有些集合不能“边遍历边改”（比如Dictionary、HashSet，或者你自己写的类）。这种情况可以用“先标记再删”的套路：

1. 先遍历一遍，把要删的元素收集到新列表里。
2. 再遍历这个新列表，从原集合里删掉对应元素。

Dictionary<int, string> dict = new Dictionary<int, string> { [1] = "one", [2] = "two", [3] = "three" };
var toDelete = new List<int>();
foreach (var kvp in dict)
{
    if (kvp.Key % 2 == 0)
        toDelete.Add(kvp.Key);
}
foreach (var key in toDelete)
    dict.Remove(key);

4. 有用的小细节

新手常见错误和误区

最常见的错误之一——以为遍历时删集合“应该能行”，毕竟有些别的语言（比如Python）经常能这么干。但在C#里，这绝对禁止，就是为了你的安全：报个明明白白的错，总比埋个阴险的bug谁都查不出来强。

还有个常见坑——用for循环正着加索引删。这会导致删完后面的元素都往前挪，结果有些元素会被跳过。只要按索引删，一定要从后往前。

故事的道理

“按条件删集合元素”这个需求在C#里太常见了，但绝不能在foreach循环里直接删——这是语言设计的底线，就是为了保证你的数据安全、避免莫名其妙的bug。
记住这个原则，能让你少熬好多夜。

永远正确的做法

• 千万别在foreach循环里直接删集合元素。不然就会报运行时错误。
• 对于列表（List<T>）和数组，要么用从后往前的for循环，要么用RemoveAll方法或LINQ过滤。
• 对于字典、集合等复杂结构——先收集要删的元素，再遍历这个列表从原集合里删。
• 不确定时，想想：删元素时集合会怎么变？迭代器会不会乱？只要有点不确定，说明方法不对。