Question

我想做的事情很简单，
我扫描一个字符串列表，然后，我将每个字符串传递给一个新的线程进行打印。

using System;
using System.Collections.Generic;
using System.Threading;

namespace MultithreadingSynchronization
{
    class Program
    {
        static void Main(string[] args)
        {
            List<string> stringList = new List<string> { "server1", "server2", "server3", "server4", "server5", "server6", "server7", "server8", "server9"};

            foreach (string server in stringList)
            {
                ThreadStart work = delegate { Threadjob(server); };
                new Thread(work).Start();
                //Thread.Sleep(10); // 10 ms delay of main thread
            }
        }

        public static void Threadjob(object server)
        {
            Console.WriteLine(server);
        }
    }
}

由于某些原因，有些线程收到错误的值，因此，输出会显示一些重复的字符串，并且还会遗漏一些字符串。
我期待这个输出（顺序并不重要）：

server1
server2
server3
server4
server5
server6
server7
server8
server9

但是，有时我会得到这个：

server3
server2
server5
server5
server7
server4
server8
server9
server9

有时我得到这个：

server2
server2
server4
server3
server6
server7
server7
server8
server9

等

的确，如果我在每个线程创建后都延迟了，我得到了我期望得到的。

有什么想法吗？

Answer 1

您应该制作变量的本地副本。试试这个：

    foreach (string server in stringList)
    {
        string local = server;
        ThreadStart work = delegate { Threadjob(local); };
        new Thread(work).Start();
        //Thread.Sleep(10); // 10 ms delay of main thread
    }

此处有更多信息：Captured variable in a loop in C#

Answer 2

new Thread(Threadjob).Start(server);

完成！但是，使用任务而不是线程（或者至少是ThreadPool）可能更合适。

Answer 3

好的，我想我明白了。
Froeach循环保持一个指针，并在每次迭代时更改它。有时，线程已创建，但仍未运行。同时，循环在下一次迭代中更改指针，当线程开始其作业时，它获取当前指针的值（后续字符串）。

顺便说一下，我找到了另一个解决方案here，如何将参数传递给线程的main函数，所以我修复了我的循环，现在它正常工作

foreach (string server in stringList)
{
    Thread thread1 = new Thread(new ParameterizedThreadStart(Threadjob));
    thread1.Start(server); 
}

将每个列表项逐个传递给新的线程

3 个答案: