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不是线程安全方法。可以在调用前加锁。

 

 posted on 2014-11-03 00:00  Grart  阅读(1264)  评论(0编辑  收藏  举报