C# BackgroundWorker Отмена с вспомогательной функцией

Question

Обычно, когда я хочу, чтобы отменить BackgroundWorker в C# я буду делать что-то вроде этого:

 while (backgroundWorker1.IsBusy) 
    { 
     backgroundWorker1.CancelAsync(); 
     autoResetEvent1.WaitOne(BGW_CANCEL_TIMEOUT); 
    } 

в одном приложении есть ряд backgroundWorkers. Действительно ли полезно использовать вспомогательную функцию для отмены фоновых рисунков, например?

CancelBackgroundWorker(BackgroundWorker bgw, AutoResetEvent are) 
{ 
    while (bgw.IsBusy) 
    { 
     bgw.CancelAsync(); 
     are.WaitOne(BGW_CANCEL_TIMEOUT); 
    } 
} 

Меня беспокоит то, что вместо передачи объектов в вопросе в функции, копии сделаны, тем самым поражение цели иметь функцию. Основная цель использования этой функции - уменьшить кодовое пространство и сделать код приложения более удобочитаемым/поддерживаемым.

Answer 1

Нет, это не нормально. Это вызывает тупик, когда BGW имеет обработчик события RunWorkerCompleted. Этот обработчик не может работать до тех пор, пока основной поток не перейдет в режим ожидания и не войдет в цикл сообщений. Свойство IsBusy останется True до тех пор, пока обработчик этого события не завершится.

У вас есть тайм-аут по вызову WaitOne, поэтому, по крайней мере, ваша программа не будет полностью висеть. Но когда WaitOne() возвращается, BGW не еще не завершен, обработчик события RWC еще не запущен. Единственная практическая альтернатива для обработчика событий RWC - делать все, что необходимо, когда завершение аннулирования завершено.

Answer 2

Ваш код является неправильным.

Вызов CancelAsync просто устанавливает CancellationPending flag на BackgroundWorker на true. Ожидается, что ваш код в событии DowWork будет периодически проверять этот флаг и останавливаться, если флаг равен true.

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