『卧槽』意外发现了 Hashtable 的 foreach 用法 BUG

这段时间，公司项目中 遇到一个问题，最后查出： 是 Hashtable 的用法导致的。

 

        private static void AutoCleanCache()
        {
            try
            {
                lock (m_HashCache.SyncRoot)
                {
                    List<object> listKey = new List<object>();
                    List<Tuple<string, DateTime, byte[]>> list = new List<Tuple<string, DateTime, byte[]>>();
                    if (m_HashCache.Count >= MAXCACHE)
                    {
                        //foreach (KeyValuePair<object, object> pair in m_HashCache)  //不能这样使用 Hashtable —— 可能会导致: 指定的转换无效 的异常
                        foreach (object key in m_HashCache.Keys)
                        {
                            object value = m_HashCache[key];
                            var svg = key as string;
                            var bytes = value as Tuple<DateTime, byte[]>;
                            if (string.IsNullOrWhiteSpace(svg) || (bytes == null || bytes.Item2 == null || bytes.Item2.Length <= 0)) listKey.Add(key);

                            list.Add(new Tuple<string, DateTime, byte[]>(svg, (bytes == null ? DateTime.MinValue : bytes.Item1), (bytes == null ? null : bytes.Item2)));
                        }
                    }

                    List<Tuple<string, DateTime, byte[]>> list2 = Enumerable.ToList(list.OrderBy(x => x.Item2).Take(list.Count / 2));
                    foreach (var item in list2) listKey.Add(item.Item1);
                    foreach (var key in listKey) m_HashCache.Remove(key);

                    listKey.Clear();
                    list.Clear();
                    list2.Clear();
                }
            }
            catch (Exception ex)
            {
                logger.Warn(ex);
            }
        }

 

代码很丑 —— 各位不要介意。

我们只看重点代码：

//foreach (KeyValuePair<object, object> pair in m_HashCache) //不能这样使用 Hashtable —— 可能会导致: 指定的转换无效 的异常
foreach (object key in m_HashCache.Keys)

 

结论：

Hashtable 不要  foreach  KeyValuePair<object, object>

 

 

这下好了， 我这边 一堆底层代码 都遭殃了 ~