枚举集合时添加元素引发重复添加问题排查

你猜的没错！这确实和foreach循环的枚举机制直接相关。foreach是依赖集合的**枚举器（Enumerator）**来工作的，当你启动遍历的瞬间，枚举器就会基于集合的当前状态生成一个“快照”，绑定到当时的集合结构上。

对于大多数标准.NET集合（比如List<T>）来说，在枚举过程中修改集合（添加/删除元素）要么直接抛出InvalidOperationException，要么即使没触发异常，枚举器也不会同步更新集合的最新状态——也就是说，你在循环里刚添加的新元素，根本不会被当前的foreach遍历到。

如果你的代码逻辑是“遍历每个元素，不匹配就走else添加新元素”，那自然会重复添加；哪怕是遍历完所有元素再判断添加，也可能因为枚举器的“快照”特性，导致判断逻辑出现偏差，最终重复插入。

这里给你几个实用的解决办法：

1. 先检查存在性，再决定添加（最推荐） #

不用遍历整个集合，直接用LINQ方法快速判断元素是否存在，既简洁又避开了遍历中修改集合的问题：

public void AddProduct(Product product)
{
    // 先确认集合中没有相同ID的产品
    if (!products.Any(p => p._id == product._id))
    {
        products.Add(product);
    }
}

2. 遍历集合副本，修改原集合 #

如果业务逻辑必须遍历元素做额外处理，可以先把集合转成副本（比如用ToList()），遍历副本时修改原集合，这样枚举器不会被干扰：

public void AddProduct(Product product)
{
    bool productExists = false;
    // 遍历集合的副本，不影响原集合的修改
    foreach (var p in products.ToList())
    {
        if (p._id == product._id)
        {
            productExists = true;
            // 这里可以添加找到元素后的处理逻辑
            break;
        }
    }
    if (!productExists)
    {
        products.Add(product);
    }
}

3. 用for循环代替foreach（适合索引型集合） #

如果你的集合支持索引访问（比如List<T>），可以改用for循环，它直接基于索引获取元素，能实时访问集合的最新状态：

public void AddProduct(Product product)
{
    bool productExists = false;
    for (int i = 0; i < products.Count; i++)
    {
        if (products[i]._id == product._id)
        {
            productExists = true;
            break;
        }
    }
    if (!productExists)
    {
        products.Add(product);
    }
}

另外还要提醒：如果你的代码是在foreach循环的else分支里直接添加元素（比如每遇到一个不匹配的元素就插入一次），那本身逻辑就有问题——这会导致只要有一个元素不匹配就触发添加，自然会生成大量重复项。正确逻辑应该是遍历完所有元素确认不存在后，只执行一次添加操作。

内容的提问来源于stack exchange，提问作者J.Doe