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

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

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

等

事实上，如果我在每个线程创建后放一个延迟，我会得到我期望得到什么。

有什么想法？

来源

2013-12-10 elady

我曾尝试你的代码（vs2012 C＃控制台应用程序默认设置，win8.0 64） - 我从来没有重复的数字。 - 我确实按照您所期望的使用多线程的方式将它们乱序排列，但不能重复。 –

难道你不想让你的ThreadJob方法成为一个实例方法吗？它似乎不是线程安全的 –

这是因为循环变量。 [关闭循环变量被认为是有害的]（http://blogs.msdn.com/b/ericlippert/archive/2009/11/12/closing-over-the-loop-variable-considered-harmful.aspx）。更改为for循环或使用本地foreach变量，如oakio所示。 – Patrick

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

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#

来源

2013-12-10 19:54:58 oakio

如果你发现一个重复的问题，你应该投票/标记以重复的方式关闭它，而不是发布一个答案。 – Servy

非常感谢！它解决了我的问题！ – elady

new Thread(Threadjob).Start(server);

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

来源

2013-12-10 20:04:00

它也解决了我的问题，谢谢！ – elady

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

顺便说一句，我找到了另一种解决方案here，如何将参数传递给线程的主要功能，所以我定我的循环，它工作正常现在

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

来源

2013-12-11 08:07:12 elady

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

回答

相关问题