C＃中使用多线程的Lambda表达式

Question

我试图理解为什么这个程序不起作用

预期输出：按随机顺序排列0-19 我跑的时候会得到：有些数字重复，有时会打印20个。

请帮忙。我在DoSomething（）中尝试使用lock（obj），但它没有帮助。

程序

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;

namespace ConsoleApplication2
{
    public delegate void callbackDelegate(int x);
    class Program
    {
        void processCallback(int x)
        {
            Console.WriteLine("IN callback: The value I got is " + x);
        }

        static void Main(string[] args)
        {
            Program p = new Program();
            p.processinThreads();
            Console.ReadKey();
        }

        public void DoSomething(int x, callbackDelegate callback)
        {
            Thread.Sleep(1000);
            //Console.WriteLine("I just woke up now " + x);
            callback(x);
        }

        public void processinThreads()
        {
            for (int i = 0; i < 20; i++)
            {
                Thread t = 
new Thread(new ThreadStart(()=>DoSomething(i, processCallback)));
                t.Start();
            }
        }
    }
}

Answer 1

public void processinThreads()
{
    for (int i = 0; i < 20; i++)
    {
        int local = i;
        Thread t = new Thread(new ThreadStart(()=>DoSomething(local, processCallback)));
        t.Start();
    }
}

你的问题与关闭lambda有关。

Answer 2

你应该只使用TPL，它比手动线程管理更容易和推荐：

Parallel.For(0, 20, x => {
    Thread.Sleep(1000);
    Console.WriteLine("IN callback: The value I got is " + x);
});

如果您不希望使用TPL Task，这也会阻塞直到循环结束，但我肯定会建议避免使用线程。

Answer 3

正如Jakub已经告诉过你的那样，你需要将i复制到另一个本地变量local中。在您的代码中，代理人可以直接访问i本身，而不是i的副本，因此他们会打印出i的当前值，这可能比您开始时的值更大线程。这叫做封闭。