Question

I’m trying to create a simple threading procedure (granted this is my first attempt at threading) and all I want to it to do is for each string in an string[] simply go through a void and perform simple folder crawling. However, I’m recieving Method Name expected and i’m unsure of why

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

You need to pass in a delegate, you can do that with a lambda:

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

But then you would be capturing a loop variable , so make it

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

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

Additional

It is quite expensive to make a Thread each time, you’re probably better of using:

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

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

Note that pjp is now created on the thread.