.NET 4.6.2 WinForms异步场景下OrderedDictionary枚举异常及线程安全问题

咱们先来拆解一下你遇到的问题：你加了静态锁但还是出现Collection was modified异常，核心原因是你的锁只保护了ExecuteQueryAsync里的遍历过程，却没管住调用方对传入的paramList的并发修改。

为什么原锁没用？ #

你用的static object syncLock锁的是遍历代码块，但paramList是调用方创建并传入的——当你await ExecuteQueryAsync时，当前线程会被释放，调用方的代码可能继续在其他线程（比如UI线程）修改这个paramList，而后台线程还在遍历它。这时候锁根本管不到调用方的修改操作，自然会触发异常。

可行的解决方案 #

方案1：确保传入后不再修改原集合

最直接的办法是，调用方在创建并填充paramList后，绝对不要再对它进行任何修改操作（Add/Remove/Clear等），直到ExecuteQueryAsync执行完毕：

var paramList = new OrderedDictionary();
paramList.Add("RuleID", btRule.RuleID);
paramList.Add("TypeSeq", 0);
paramList.Add("Type", btRule.Action);
// 传入后彻底不要再碰这个paramList
bool result = await ExecuteQueryAsync(sql, paramList, connection);
// 如果后续需要新的参数集合，重新创建一个实例

这个方案最简单，但要求你能完全控制调用方对paramList的操作。

方案2：创建集合的不可变副本（推荐）

如果没法保证调用方不修改原集合，那就在ExecuteQueryAsync内部先创建一个paramList的副本，之后遍历副本而不是原集合——这样即使原集合被修改，也不会影响遍历过程：

public static async Task<bool> ExecuteQueryAsync(string sql, OrderedDictionary paramList, SqlConnection connection) {
    try {
        OrderedDictionary safeParamList = null;
        if (paramList != null) {
            // 锁原集合，保证复制过程中不会被修改
            lock (paramList) {
                safeParamList = new OrderedDictionary(paramList.Count);
                foreach (DictionaryEntry entry in paramList) {
                    safeParamList.Add(entry.Key, entry.Value);
                }
            }
        }

        using (SqlCommand command = new SqlCommand(sql, connection)) {
            command.CommandType = CommandType.Text;
            // 遍历安全副本，无需额外锁
            if (safeParamList != null) {
                foreach (DictionaryEntry param in safeParamList) {
                    command.Parameters.AddWithValue(param.Key.ToString(), param.Value);
                }
            }
            command.CommandTimeout = 0;
            await command.ExecuteNonQueryAsync();
        }
        return true;
    } catch (Exception ex) {
        // 你的异常处理逻辑
    }
}

这里的关键是：

• 复制原集合时要锁原集合，防止复制过程中原集合被修改
• 后续遍历的是独立的副本，和原集合完全隔离

方案3：改用线程安全集合（如果允许放弃顺序）

如果你的业务逻辑不严格要求参数的顺序，可以改用ConcurrentDictionary<string, object>（.NET 4.0+内置的线程安全字典），它支持并发的读和写操作，不会触发遍历异常。但注意ConcurrentDictionary是无序的，如果你必须保持OrderedDictionary的顺序特性，这个方案不适用。

总结 #

你的核心问题不是遍历过程的线程安全，而是传入的集合在遍历期间被外部线程修改。最稳妥的做法是创建集合副本，彻底隔离遍历操作和外部修改。

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