Closing over the Loop Variable in C#

独自空忆成欢 提交于 2019-12-28 16:06:26

问题


From this post, I was told that the following section of code suffered from "the egregious act of closing over the loop variable."

    foreach (Canidate canidate in allCanidates)
    {
        Thread newThread = new Thread(delegate()
        {
            BusyWait(canidate);
        });

        newThread.Start();
    }

I switched it to this:

foreach (Canidate canidate in allCanidates)
        {
            var can = canidate;
            Thread newThread = new Thread(delegate()
            {                 
                BusyWait(can);
            });

            newThread.Start();
        }

But my boss keeps insisting that it will suffer from the same issues. I used this link to try to solve this issue. Can someone help me to correctly solve the issue so that my code will execute correctly. this issue is all new to me, and I am not sure I full understand it.


回答1:


Closing over variables that later change values is what is causing the problem.

  • candidate changes every "lap" around the loop and will cause problems.
  • can is newly created every "lap" around the loop, never changes, and won't cause problems.


来源:https://stackoverflow.com/questions/9626051/closing-over-the-loop-variable-in-c-sharp

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!