web-dev-qa-db-ja.com

C#:すべてのスレッドが完了するのを待つ

私が書いているコードの一般的なパターンに遭遇しました。そこでは、グループ内のすべてのスレッドが完了するのをタイムアウトで待つ必要があります。タイムアウトはallスレッドの完了に必要な時間であると想定されているため、各スレッドで単純にthread.Join(timeout)を実行しても、タイムアウトはtimeout * numThreadsになるため機能しません。

今、私は次のようなことをしています:

var threadFinishEvents = new List<EventWaitHandle>();

foreach (DataObject data in dataList)
{
    // Create local variables for the thread delegate
    var threadFinish = new EventWaitHandle(false, EventResetMode.ManualReset);
    threadFinishEvents.Add(threadFinish);

    var localData = (DataObject) data.Clone();
    var thread = new Thread(
        delegate()
        {
            DoThreadStuff(localData);
            threadFinish.Set();
        }
    );
    thread.Start();
}

Mutex.WaitAll(threadFinishEvents.ToArray(), timeout);

しかし、この種のことにはもっと単純なイディオムがあるはずです。

61
JSBձոգչ

まだJoinを使用する方が簡単だと思います。予想される完了時間(Now + timeoutとして)を記録し、ループで

if(!thread.Join(End-now))
    throw new NotFinishedInTime();
26

.NET 4.0では、 System.Threading.Tasks の方がずっと簡単です。これが私にとって確実に機能するスピン待機ループです。すべてのタスクが完了するまでメインスレッドをブロックします。 Task.WaitAll もありますが、それは私にとっていつもうまくいったわけではありません。

        for (int i = 0; i < N; i++)
        {
            tasks[i] = Task.Factory.StartNew(() =>
            {               
                 DoThreadStuff(localData);
            });
        }
        while (tasks.Any(t => !t.IsCompleted)) { } //spin wait
22
T. Webster

質問がぶつかったので、先に進み、解決策を投稿します。

using (var finished = new CountdownEvent(1)) 
{ 
  for (DataObject data in dataList) 
  {   
    finished.AddCount();
    var localData = (DataObject)data.Clone(); 
    var thread = new Thread( 
        delegate() 
        {
          try
          {
            DoThreadStuff(localData); 
            threadFinish.Set();
          }
          finally
          {
            finished.Signal();
          }
        } 
    ); 
    thread.Start(); 
  }  
  finished.Signal(); 
  finished.Wait(YOUR_TIMEOUT); 
} 
9
Brian Gideon

私の頭上で、なぜThread.Join(timeout)だけで、合計タイムアウトから参加にかかった時間を削除しないのですか?

// pseudo-c#:

TimeSpan timeout = timeoutPerThread * threads.Count();

foreach (Thread thread in threads)
{
    DateTime start = DateTime.Now;

    if (!thread.Join(timeout))
        throw new TimeoutException();

    timeout -= (DateTime.Now - start);
}

編集:コードは擬似的になりました。 +4で修正した回答がまったく同じで、あまり詳細でない場合に、なぜ-2を修正するのかを理解しないでください。

8

これは質問に答えません(タイムアウトなし)が、コレクションのすべてのスレッドを待機する非常に簡単な拡張メソッドを作成しました。

using System.Collections.Generic;
using System.Threading;
namespace Extensions
{
    public static class ThreadExtension
    {
        public static void WaitAll(this IEnumerable<Thread> threads)
        {
            if(threads!=null)
            {
                foreach(Thread thread in threads)
                { thread.Join(); }
            }
        }
    }
}

それからあなたは単に電話する:

List<Thread> threads=new List<Thread>();
//Add your threads to this collection
threads.WaitAll();
7
Vincent

これはオプションではないかもしれませんが、Parallel Extension for .NETを使用できる場合は、生のスレッドの代わりにTasksを使用し、Task.WaitAll()を使用してそれらが完了するのを待ちます。 。

6
Jon Norton

私はこれを行う方法を理解しようとしていましたが、Googleから答えを得ることができませんでした。私はこれが古いスレッドであることを知っていますが、ここに私の解決策がありました:

次のクラスを使用します。

class ThreadWaiter
    {
        private int _numThreads = 0;
        private int _spinTime;

        public ThreadWaiter(int SpinTime)
        {
            this._spinTime = SpinTime;
        }

        public void AddThreads(int numThreads)
        {
            _numThreads += numThreads;
        }

        public void RemoveThread()
        {
            if (_numThreads > 0)
            {
                _numThreads--;
            }
        }

        public void Wait()
        {
            while (_numThreads != 0)
            {
                System.Threading.Thread.Sleep(_spinTime);
            }
        }
    }
  1. スレッドを実行する前に、Addthreads(int numThreads)を呼び出します。
  2. それぞれが完了した後、RemoveThread()を呼び出します。
  3. 続行する前に、すべてのスレッドが完了するまで待機するポイントでWait()を使用します
1
Dylan

私は本C#4.0:The Complete Reference of Herbert Schildtを読みました。作成者はjoinを使用して解決策を提供します。

class MyThread
    {
        public int Count;
        public Thread Thrd;
        public MyThread(string name)
        {
            Count = 0;
            Thrd = new Thread(this.Run);
            Thrd.Name = name;
            Thrd.Start();
        }
        // Entry point of thread.
        void Run()
        {
            Console.WriteLine(Thrd.Name + " starting.");
            do
            {
                Thread.Sleep(500);
                Console.WriteLine("In " + Thrd.Name +
                ", Count is " + Count);
                Count++;
            } while (Count < 10);
            Console.WriteLine(Thrd.Name + " terminating.");
        }
    }
    // Use Join() to wait for threads to end.
    class JoinThreads
    {
        static void Main()
        {
            Console.WriteLine("Main thread starting.");
            // Construct three threads.
            MyThread mt1 = new MyThread("Child #1");
            MyThread mt2 = new MyThread("Child #2");
            MyThread mt3 = new MyThread("Child #3");
            mt1.Thrd.Join();
            Console.WriteLine("Child #1 joined.");
            mt2.Thrd.Join();
            Console.WriteLine("Child #2 joined.");
            mt3.Thrd.Join();
            Console.WriteLine("Child #3 joined.");
            Console.WriteLine("Main thread ending.");
            Console.ReadKey();
        }
    }
1
Bùi Công Giao

可能な解決策:

var tasks = dataList
    .Select(data => Task.Factory.StartNew(arg => DoThreadStuff(data), TaskContinuationOptions.LongRunning | TaskContinuationOptions.PreferFairness))
    .ToArray();

var timeout = TimeSpan.FromMinutes(1);
Task.WaitAll(tasks, timeout);

DataListがアイテムのリストであり、各アイテムが個別のスレッドで処理される必要があると仮定します。

0
Alex Aza