C# threading Метод Имя ожидаемой ошибки

Question

Я пытаюсь создать простую процедуру потоковой передачи (если это моя первая попытка потоковой передачи), и все, что я хочу сделать, это для каждой строки в строке [] просто пройти через void и выполнить простую сканирование папки. Тем не менее, я ПОЛУЧАТЬ Имя метода ожидалось, и я не уверен, почему

string[] FileListing = {@"C:\","E:\"}; 
     foreach (string fl in FileListing) 
     { 
      ProjectDirectoryProcessing pjp = new ProjectDirectoryProcessing(); 
      //error here 
      Thread oThread = new Thread(new ThreadStart(pjp.ProjectProcessor(fl))); 
      oThread.Start(); 
     } 

public class ProjectDirectoryProcessing 
{ 
    public void ProjectProcessor(string rootDirectory) 
    { 
     Stopwatch stopwatch = new Stopwatch(); 
     stopwatch.Start(); 
     HashSet<string> DirectoryHolding = new HashSet<string>(); 
     //do some work 
     //foreach loop 
    }; 
    } 

Answer 1

Вам необходимо пройти в качестве делегата, вы можете сделать это с лямбда:

Thread oThread = 
     new Thread(() => pjp.ProjectProcessor(fl)); 

Но тогда вы бы захватив переменную петли, так что она

foreach (string fl in FileListing) 
    { 
     string copy = fl; 

     ProjectDirectoryProcessing pjp = new ProjectDirectoryProcessing(); 
     //error here 
     Thread oThread = 
      new Thread(() => pjp.ProjectProcessor(copy)); 
     oThread.Start(); 
    } 

Дополнительная

Весьма дорого сделать нить каждый раз, вероятно, вам лучше использовать:

Parallel.ForEach(FileListing, fl => 
     {    
     ProjectDirectoryProcessing pjp = new ProjectDirectoryProcessing();   
     pjp.ProjectProcessor(fl); 

     }); 
    // when here: all threads are done. 

Отметьте, что pjp теперь создан на темы.

Answer 2

Вы должны пройти делегат конструктору потока. Самый простой способ сделать это здесь, чтобы использовать лямбда-выражение:

string copy = fl; 
Thread oThread = new Thread(() => pjp.ProjectProcessor(copy)); 

Обратите внимание, что вам нужно сделать копию переменной цикла из-за способа, что переменные цикла захватываются. (См Eric Lippert's blog post для более подробной информации.)

Альтернативно, как вы создаете новый ProjectDirectoryProcessing экземпляр на каждой итерации, можно передать строку в конструктор вместо:

foreach (string fl in FileListing) 
{ 
    // Note change in name to be clearer (IMO) 
    ProjectDirectoryProcessor pjp = new ProjectDirectoryProcessor(fl); 
    Thread oThread = new Thread(pjp.Execute); 
    oThread.Start(); 
} 

...

public class ProjectDirectoryProcessor 
{ 
    private readonly string rootDirectory; 

    public ProjectDirectoryProcessor(string rootDirectory) 
    { 
     this.rootDirectory = rootDirectory; 
    } 

    public void Execute() 
    { 
     Stopwatch stopwatch = Stopwatch.StartNew(); 
     HashSet<string> DirectoryHolding = new HashSet<string>(); 
     // do some work 
     //foreach loop 
    } 
} 

Answer 3

Thread oThread = new Thread(new ThreadStart(pjp.ProjectProcessor(fl))); 

Вы, по существу, проходя void в качестве аргумента конструктора ThreadStart.

Answer 4

Должна ли быть нитью? Задача лучше подходит для этого. Перестановка из:

Thread oThread = new Thread(new ThreadStart(pjp.ProjectProcessor(fl))); 
oThread.Start(); 

с:

Task.Factory.StartNew(pjp.ProjectProcessor(fl));