如何用LINQ在C#字典中移除多个同值键？解决迭代器失效问题

兄弟，你这个痛点我太懂了——直接遍历Dictionary的时候删除元素确实会触发迭代器失效，那种无限循环的写法不仅看着头疼，性能也实在拉胯。别担心，用LINQ或者更合理的集合操作就能完美解决，给你几个靠谱的方案：

先看你的原实现（方便对比） #

Dictionary<string, string> Channels = //...; 
while (true) { 
    var bFound = false; 
    foreach(var c in Channels) { 
        if(c.Value == version) { 
            Channels.Remove(c.Key); 
            bFound = true;
            // ... 其他操作？
        }
    }
    if(!bFound) break;
}

方案1：先收集待移除的键，再批量删除（推荐修改原字典的场景） #

这种方式既避免了迭代器失效，又只需要遍历两次集合（一次收集键，一次删除），性能和可读性都拉满：

// 第一步：用LINQ筛选出所有值等于version的键，转成List避免迭代器问题
var keysToRemove = Channels.Where(kv => kv.Value == version)
                           .Select(kv => kv.Key)
                           .ToList();

// 第二步：遍历键列表删除元素，还能顺便执行你的其他操作
foreach (var key in keysToRemove)
{
    Channels.Remove(key);
    // 这里可以放你原来的额外逻辑，比如日志、回调等
}

方案2：创建新字典（适合允许替换原字典的场景） #

如果你的业务场景允许直接替换原Dictionary实例，这种LINQ风格的写法最简洁，代码一眼就能看懂：

// 直接生成一个排除了目标元素的新字典，替换原引用
Channels = Channels.Where(kv => kv.Value != version)
                   .ToDictionary(kv => kv.Key, kv => kv.Value);

注意：如果原字典还有其他地方在引用，替换实例前要确认不会影响其他逻辑哦。

为什么原写法不够好？ #

你的无限循环写法最坏情况下会遍历N次字典（N是待删除元素的数量），不仅性能低效，而且逻辑绕，后续维护的人看到这种代码很容易懵圈。上面的两种方案都是O(n)时间复杂度，代码清晰易懂，维护成本低。

内容的提问来源于stack exchange，提问作者Andy