Reputation: 3577

Stopping a Thread

Here is the situation. I have a thread that gets started with the click of a button. It then works for approx 2 min (give or take quite a bit) and then finishes, updating the UI and all along the way. It is quite nice. Now I have decided to add a "Cancel" button to the mix. My question would be what is the best approach to stop a thread from the outside? What methods are best to avoid completely? What resources can I look into myself to get some more knowledge on the topic.

One other piece of info. The program was not written by me, I just "inherited" it, and am now trying to expand it's functionality in small ways to make it more user friendly.

EDIT: Working with VS2008 and .NET 3.5

Upvotes: 2

Answers (7)

Achille

Reputation: 111

I often use System.ComponentModel.BackgroundWorker in similar cases. On the UI side, call its CancelAsync() method; on the worker thread side, periodically check the CancellationPending property.

Upvotes: 5

thewpfguy

Reputation: 824

Rather than going for a shared boolean variable between UI and worker class, a more clearer approach would be to use reset events (AutoResetEvents/ManualResetEvent).

However it still depends on what kind of processing you are doing in the thread. If it is say for example a loop which processes files, the reset event threading constructs can be used. If it's a single operation (e.g copying a huge file) then you have no way of cancelling that, and you also do not know what the thread is currently executing.

It would also help if you know what the user wants by clicking cancel - drop everything that thread is doing or take the application to previous state or finish the current item being processed and return immediately.

Upvotes: 2

Henk Holterman

Reputation: 273844

what is the best approach to stop a thread from the outside?

The Thread has to cooperate. You need a shared flag (boolean) and the Thread-code has to check this periodically.

What methods are best to avoid completely?

Thread.Abort().

Upvotes: 6

Kimi

Reputation: 14129

If you are using .NET 4 try Tasks and their built in cancelation mechanism.

Upvotes: 1

Will Dean

Reputation: 39530

One 'best' way to do this type of stuff in .NET is to use BackgroundWorker.

BackgroundWorker will cleanly handle the progress updates (which you may currently have working only by luck, or by a lot of tedious Invoke() calls), and also provides a clean cancellation mechanism.

BackgroundWorker uses a pool thread rather than a dedicated one, which is also a better approach than some others.

Stopping a thread from outside it is almost always the wrong approach, though to begin with it usually feels like it's what you want to do.

If you're using .NET 4.0, there is a whole new task and cancellation mechanism available as well, but it sounds to me that a study of BackgroundWorker would be a good first point for you.

Upvotes: 4

Mike

Reputation: 519

In fact if you want to safely cancel some background operation the best approach is to create private boolean field _isCancelled which will be shared between GUI thread and background thread. If user presses cancel you just set _isCancelled to false. And in the background thread regulary check if _isCancelled is turned to true and return the method.

Upvotes: 12

Svish

Reputation: 158351

You should have a flag somewhere that the worker thread checks once in a while. And when the flag says the thread should cancel, it should stop working and exit gracefully.

Upvotes: 2

Stopping a Thread

Answers (7)

Related Questions