List<int> _ValueLis = new List<int>(); private void AddInt(int i) { _ValueLis.Add(i); } public void ThreadTest() { for (int _i = 0, _iCnt = 5000; _i < _iCnt; _i++) { Thread _thd = new Thread( () => { AddInt(_i); }); _thd.Start(); } }
本来设计是代码运行完后,_ValueLis中保存的是1到5000的值,可实际确不是。
问题1:_ValueLis中会存在很多不连续的重复值。
原因在于AddInt(_i);引用的i是for的参数。在线程真正要开始运行的时候,i的值可能已经改变了,造成传入函数AddInt的值是改变后的值。
解决办法,运行“闭包”定义个临时变量做为函数AddInt的参数。
List<int> _ValueLis = new List<int>(); private void AddInt(int i) { _ValueLis.Add(i); } public void ThreadTest() { for (int _i = 0, _iCnt = 5000; _i < _iCnt; _i++) { int _iInner = _i; Thread _thd = new Thread( () => { AddInt(_iInner); }); _thd.Start(); } }
问题2:_ValueLis的个数不是5000个。
这是由于多个线程同时调用_ValueLis.Add(i);引起的。说明List::Add不是线程安全方法。可以在调用前加锁。