C# for和foreach解析

来源：互联网发布：马六甲知乎编辑：程序博客网时间：2024/06/06 19:53

首先上一部分代码：

        private static List<AllShop> allShop = new List<AllShop>();//临时存放推送商场                public Service1()        {            InitializeComponent();        }        protected override void OnStart(string[] args)        {                        writestr("服务启动");            timer1.Enabled = true;            timer1.Interval = basetime;            get_Shops();            for (int i = 0; i < allShop.Count; i++)            {                //为每个shop开启新线程                ThreadStart threadStart = new ThreadStart(delegate ()                {                    try                    {                        pushDDXX(allShop[i]);                        writestr(allShop[i].getShopID() + "线程结束");                    }                    catch (Exception ex)                    {                        writestr("OnStart_"+ i + ":"+ex.Message.ToString());                    }                });                Thread thread = new Thread(threadStart);                thread.Start();            }        }

看出有什么问题了吗？

这是报错日志：

2017-04-13 09:46:30
OnStart_16:索引超出范围。必须为非负值并小于集合大小。

这里的

allShop.Count是等于16的。所以理论上不会出现上述问题，不可能索引超出范围。

这是更改后的：

        private static List<AllShop> allShop = new List<AllShop>();//临时存放推送商场                public Service1()        {            InitializeComponent();        }        protected override void OnStart(string[] args)        {                        writestr("服务启动");            timer1.Enabled = true;            timer1.Interval = basetime;            get_Shops();            foreach (var shop in allShop)            {                //为每个shop开启新线程                ThreadStart threadStart = new ThreadStart(delegate ()                {                    try                    {                        pushDDXX(shop);                        writestr(shop.getShopID() + "线程结束");                    }                    catch (Exception ex)                    {                        writestr("OnStart_"+ shop.getShopID() + ":"+ex.Message.ToString());                    }                });                Thread thread = new Thread(threadStart);                thread.Start();            }        }

后来经过断点调试，发现在多线程中，for循环的i+1后，前面代码里的i也会跟着改变。for循环停止的时候i=16，这时候不管在哪个线程，只要没有走完i都是16。外层循环结束的时候，里面的执行方法不一定结束了。
这个错误在于没有理解for循环和进程的关系，以为最后i=16的时候不会走进去，其实这是个误区，i=16的时候只是不再产生新的进程，但所有进程都使用一个全局的i，所以最终出现了writestr(allShop[i].getShopID() + "线程结束");导致错误，改法就是把变量作为参数传进去。

或者用ParameterizedThreadStart委托代替ThreadStart委托并在匿名委托方法体中用委托参数代替i用thread[i].Start(i)来启动线程

1 0