BackgroundWorkerを使用したフォームのフリーズ

Question

私が必要とするすべてのプロセスが操作されるWinFormで作業しています。今では、BackgroundWorkerをProgressBarと統合し、キャンセルボタンを自分のコードに組み込もうとしています。別の方法ではなく、自分のコードの周りにローカルに置いて欲しい。これをテストするために、進行中のバー（まだアクティブではない）とfor-loopを停止するボタンで新しいフォームが作成されます。ただし、コードは機能していません（プログレスバーはまだ含まれていません）。フォームはすぐにフリーズし（画像参照）、キャンセルボタンをテストすることはできません。ただしforループは実行され、"Done: " + l.ToString()が表示されます。これをどうすれば解決できますか？

void stopMeasurement(object sender, EventArgs e) 
{ 
    stopMeas = true;  
} 

public void testcancel() // Test method which is triggered manually 
{ 
    int l = 0; 

    MetingProgress metingProgress = new MetingProgress(); 
    metingProgress.btnCancelmeting.Click += new EventHandler(stopMeasurement); 

    BackgroundWorker worker = new BackgroundWorker(); 
    worker.WorkerSupportsCancellation = true; 
    worker.DoWork += (sender, args) => 
    {      
     for (int k = 0; k < 10; k++) 
     { 
      Thread.Sleep(1000); 
      l++; 

      if (worker.CancellationPending) 
       break; 
     } 

     MessageBox.Show("Done: " + l.ToString()); 

    }; 
    worker.RunWorkerAsync(); 

    while (worker.IsBusy) 
    { 
     if (stopMeas) 
      worker.CancelAsync(); 
    } 

    metingProgress.Dispose(); 
    MessageBox.Show("All done"); 

} 

Answer 1

フォームは、ループはまだメインスレッドで実行している間、あなたはを持っているので、これはすぐに

をフリーズ！したがって、フォームは反応しません。これは、buisy waitingと呼ばれています。 CancelAsyncメソッドを呼び出すことはできません。であるあなたは基本的に行っている何

void stopMeasurement(object sender, EventArgs e) 
{ 
    stopMeas = true; 
    worker.CancelAsync(); 

} 

：あなたは、第二のキャンセルトークンを作成し

一つの解決策は、whileループを削除し、ボタンイベントコードにキャンセル電話をかけることができます。

worker.DoWork += (sender, args) => 
{      
    for (int k = 0; k < 10; k++) 
    { 
     Thread.Sleep(1000); 
     l++; 

     if (stopMeas) 
      break; 
    } 

    string mes = stopMeas ? "Done: " + l.ToString() : "Task aborted!"; 
    MessageBox.Show(mes); 

}; 

EDIT：この行：

metingProgress.Dispose(); 

がObjectDisposed例外につながる可能性があるので、別の可能性は、バックグラウンド操作をキャンセルするだけstopMeasを使用することができます。バックグラウンドプロセスがまだ実行中で、プログレスバーを更新しようとしていて、すでにフォームを破棄している場合。この行を削除してガベージコレクタに残す必要があります。

Answer 2

このコードは、あなたの問題である：あなたの労働者が完了するまで

while (worker.IsBusy) 
{ 
    if (stopMeas) 
     worker.CancelAsync(); 
} 

あなたのGUIスレッドは、そのループです。 あなたは、あなたのワーカー・インスタンスをEventHandler内から到達可能にして、そこからworker.CancelAsync（）を呼び出す必要があります。この外

---

、私は個人的に2つの段階でコードを改善する：

1. はMetingProgressクラスに全体のBackgroundWorkerを移動して、そのコンストラクタは、実際の作業の実施のためのデリゲートを取らせ

   。

2. TAP（タスク非同期パターン）を使用します。つまり、async/progressとCancellationTokenでタスクを待機します。