坑1.Dictionary的Remove()与Add(),别插队啊喂
最近做的代码里有一个方法是用来控制并发的,因为下位机程序的一些Bug,可能会在极短时间内传来两条指令,并且两条指令的Guid还一致,当时的做法是做了一个验证方法,保持一个Dictionary和相应的锁,长度为100,如果有指令传进来之后,对Dictionary进行加锁,查找有无该指令的Id所对应的Key,如果有则不再进行处理,如果没有该Key则将该Key加入到Dictionary中,当Dictionary的Count大于100时,移除掉Dictionary的第0个元素。当时在本地测试并没有什么异常(其实是因为只运行了十来条指令,当时根本没考虑的Remove()和Add()的问题,只测试并发了),到了客户那运行了几天后发现并发依旧存在,审了一遍代码根本没审处啥问题,不行了,打日志吧,加上日志后又在客户那运行了几天(没办法,这指令频率相当低),发现了同样的问题,查看日志发现当Dictionary的Count数到100之后就不变了,写了一段测试代码
static clsLimitInsertDic dic = new clsLimitInsertDic(10);
static void Main(string[] args)
{
for (int i = 0; i < 20; i++)
{
Console.WriteLine("第{0}次",i);
dic.IsExists("" + i, "" + i + "index");
}
Console.ReadLine();
}
public bool IsExists(string key, string ext)
{
Console.WriteLine("进入了IsExists,key:{0}", key);
lock (_LockObj)
{
Console.WriteLine("进入了_lockObj锁,锁Hash:{0}", _LockObj.GetHashCode());
Console.WriteLine("DicCount:{1},Dic条目:{0}", JsonMapper.ToJson(dic), dic.Count);
if (dic.ContainsKey(key))
{
Console.WriteLine("返回结果:true");
return true;
}
dic.Add(key, ext);
if (dic.Count > Limit)
{
dic.Remove(dic.First().Key);
}
Console.WriteLine("返回结果:false");
return false;
}
}
运行结果如下
运行了20次,却只存了1~10条的数据,第0条的数据也没有了,断点之后发现,当第一次执行Remove()时,删除了0位置的数据,当再次执行Add()操作时,Item被添加到了原来0的位置这么循环下去,每次删除的都是刚新增的数据,难怪加不上。。。(我所说的位置不能确定内存位置,是在断点调试时对象监视器中元素的位置顺序)
分析应该是因为Dictionary是Hash的原因,但是提供的First()方法是为了个啥呢。。。。
将Dictionary<string,string>改为了 List<KeyValuePair<string, string>>解决了问题(不用Queue的原因是因为还存在从集合的某个位置移除元素的需求,用List比较方便)