ProgressBar in multi-threaded application

[leppie]

gajatko wrote:

progress.BeginInvoke

Do not use BeginInvoke, as that will queue the invocation. Invoke should be sufficient.

xacc.ide - now with TabsToSpaces support
IronScheme - 1.0 alpha 4a out now (29 May 2008)

[realJSOP]

You're making it harder than it has to be. Use a BackgroundWorker object to perform the calculation, and call the UpdateProgress method at the desired interval.

private BackgroundWorker myWorker = BackgroundWorker();

//--------------------------------------------------------------------------------
/// /// Initializes the background worker thread.
///
private void InitEncryptWorker()
{
myWorker.WorkerReportsProgress = true;
myWorker.WorkerSupportsCancellation = true;
myWorker.DoWork += new System.ComponentModel.DoWorkEventHandler(myWorker_DoWork);
myWorker.RunWorkerCompleted += new System.ComponentModel.RunWorkerCompletedEventHandler(myWorker_RunWorkerCompleted);
myWorker.ProgressChanged += new System.ComponentModel.ProgressChangedEventHandler(myWorker_ProgressChanged);
}

//--------------------------------------------------------------------------------
public void myWorker_DoWork(Object sender, DoWorkEventArgs e)
{
BackgroundWorker thisWorker = sender as BackgroundWorker;

int progress = 0;
for (int i = 0; i < 1000; i++)
{
    if (!thisWork.CancellationPending)
    {
        // do some work here

        // and then report progress
        thisWorker.ReportProgress(progress);
    }
}

}

//--------------------------------------------------------------------------------
private void myWorker_ProgressChanged(object sender,ProgressChangedEventArgs e)
{
if (e.ProgressPercentage != this.progressBar.Value)
{
this.progressBar.Value = e.ProgressPercentage;
}
}

//--------------------------------------------------------------------------------
private void myWorker_RunWorkerCompleted(object sender,RunWorkerCompletedEventArgs e)
{
}

"Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
-----
"...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001

[Lutoslaw]

This way you use only one thread for calculations and utilize 25% of quad core processors

Greetings - Gajatko Portable.NET is part of DotGNU, a project to build a complete Free Software replacement for .NET - a system that truly belongs to the developers.

[Lutoslaw]

Application hangs when using Invoke instead of BeginInvoke. From debugger, it stops on call to the Invoke method.

Greetings - Gajatko Portable.NET is part of DotGNU, a project to build a complete Free Software replacement for .NET - a system that truly belongs to the developers.

[Daniel Grunwald]

Parallel.For does not run the calculation in the background. It just splits the work to multiple threads, but the UI is blocked until the calculation has finished. If you want multiple threads to calculate in the background, use BackgroundWorker+Parallel.For.

[Lutoslaw]

Frankly speaking I use my own implementation of Parallel to preserve .Net 2.0 compatibility but it works as you said, too. I will check if it works and post soon.

Greetings - Gajatko Portable.NET is part of DotGNU, a project to build a complete Free Software replacement for .NET - a system that truly belongs to the developers.

[realJSOP]

C'mon, be a programmer. If your calculations take so long that it's bothersome, put some Thread.Sleep(10) calls in the loop somewhere.

"Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
-----
"...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001

[Mark Salsbery]

:-D

Mark Salsbery Microsoft MVP - Visual C++ :java:

[Lutoslaw]

John Simmons / outlaw programmer wrote:

C'mon, be a programmer

Yeach and then post your code on a Coding Horrors forum before someone else does. :laugh:

Greetings - Gajatko Portable.NET is part of DotGNU, a project to build a complete Free Software replacement for .NET - a system that truly belongs to the developers.

[Lutoslaw]

BackgroundWorker+Parallel.For combination doesn't work.

progress.Value = e.ProgressPercentage;

throws an exception (Cross-thread operation not valid: Control 'progress' blah blah blah) and

progress.Invoke(new Action(delegate {
progress.Value = e.ProgressPercentage;
}));

hangs the program. Any ideas? I though that this situation is so common that there is at least one satisfactory solution for it... :(

Greetings - Gajatko Portable.NET is part of DotGNU, a project to build a complete Free Software replacement for .NET - a system that truly belongs to the developers.

[Daniel Grunwald]

Why is it hanging? Take a look at what the threads are doing. Why is the main thread blocked? What is it waiting for? Maybe you have a dead lock? (progress.Invoke waits for the main thread to finish any active event handlers, maybe the main thread is somehow still waiting for your calculation?)

[realJSOP]

Yeah, I'll do that. In the meantime, you'll still be dickin' around with your code, and I will have finished this particular task moved on to something else. On the other hand, maybe using 25% of the CPU ain't so bad, after all. Of course, we can't determine the trade-offs because you haven't given us all the requirements.

"Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
-----
"...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001