Background worker tasks sequentially?

[sjoshi]

I'm able to use the Bacground Worker class to execute task and pain the
UI. However I want to be able to execute n tasks in order, say n1, then
n2 if there were no errors during n1, n3 so on.

I'm wondering what would be the best way to do this , do I wait in a
do...while loop for each to finish and then start another ?

thanks
Sunit

[Nicholas Paldino [.NET/C# MVP]]

Sunit,

That's basically what you will have to do. It seems like you have three
tasks, but you want all three to run sequentially, just not on the UI
thread. You can still use the BackgroundWorke r class for this, you just
have to do the first task with it, then the second, then the third, and so
on, and so on.

Hope this helps.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"sjoshi" <sj****@ingr.co mwrote in message
news:11******** *************@t 46g2000cwa.goog legroups.com...

I'm able to use the Bacground Worker class to execute task and pain the
UI. However I want to be able to execute n tasks in order, say n1, then
n2 if there were no errors during n1, n3 so on.

I'm wondering what would be the best way to do this , do I wait in a
do...while loop for each to finish and then start another ?

thanks
Sunit

[Brian Gideon]

Sunit,

In addition to what Nicholas said there is an event on the
BackgroundWorke r class that is raised when the task is complete. You
should be able to use that to your advantage.

Brian

sjoshi wrote:

I'm able to use the Bacground Worker class to execute task and pain the
UI. However I want to be able to execute n tasks in order, say n1, then
n2 if there were no errors during n1, n3 so on.

I'm wondering what would be the best way to do this , do I wait in a
do...while loop for each to finish and then start another ?

thanks
Sunit

[sjoshi]

Nicholas Paldino [.NET/C# MVP] wrote:

Sunit,

That's basically what you will have to do. It seems like you have three
tasks, but you want all three to run sequentially, just not on the UI
thread. You can still use the BackgroundWorke r class for this, you just
have to do the first task with it, then the second, then the third, and so
on, and so on.

Hope this helps.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"sjoshi" <sj****@ingr.co mwrote in message
news:11******** *************@t 46g2000cwa.goog legroups.com...

I'm able to use the Bacground Worker class to execute task and pain the
UI. However I want to be able to execute n tasks in order, say n1, then
n2 if there were no errors during n1, n3 so on.

I'm wondering what would be the best way to do this , do I wait in a
do...while loop for each to finish and then start another ?

thanks
Sunit

Ok thanks I need to run a background task for each row ina dset. So I
tried this:

foreach (DAO.DAODatabas e.DatabaseRow backupEntry in
_srvBackupDSet. Database)
{
InitAndStartBac kgroundWorker(b ackupEntry);
while (!_lastRunFinis hed)
{
Thread.Sleep(50 0);
}
if (!_lastRunOk)
break;
}
private void InitAndStartBac kgroundWorker(D AO.DAODatabase. DatabaseRow
backupRow)
{
worker = new BackgroundWorke r();
worker.WorkerRe portsProgress = true;

//Set background worker
worker.DoWork += new DoWorkEventHand ler(worker_DoWo rk);
worker.Progress Changed += new
ProgressChanged EventHandler(wo rker_ProgressCh anged);
worker.RunWorke rCompleted += new
RunWorkerComple tedEventHandler (worker_RunWork erCompleted);

//Start Backup
worker.RunWorke rAsync(backupRo w);
}

void worker_DoWork(o bject sender, DoWorkEventArgs e)
{
_lastRunFinishe d = false;
_backupMgr.Perc entCompleted += new
PercentEventHan dler(_backupMgr _PercentComplet ed);
BackgroundWorke r theWorker = sender as BackgroundWorke r;
DoBackup((DAO.D AODatabase.Data baseRow)e.Argum ent);
}

void worker_RunWorke rCompleted(obje ct sender,
RunWorkerComple tedEventArgs e)
{
if (e.Error != null)
{
_lastRunOk = false ;
UpdateGridForEr ror(e.Error);
}
else
_lastRunOk = true;
_lastRunFinishe d = true;
}
void worker_Progress Changed(object sender, ProgressChanged EventArgs e)
{
//code to UpdateGrid progress bar
}

However now the UI freezes and the 1st job itself never completes; i.e
the run worker_RunWorke rCompleted event is never raised.

It works fine if I just pass a single row.

thanks
Sunit

[Nicholas Paldino [.NET/C# MVP]]

It looks like you are waiting on a loop until the processing is done.
You should be letting the events that the BackgroundThrea d fires tell you
when you are done instead of waiting for the variables to be set (which you
are doing the wrong way, since you are not locking access to them).
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"sjoshi" <sj****@ingr.co mwrote in message
news:11******** *************@f 1g2000cwa.googl egroups.com...

>
Nicholas Paldino [.NET/C# MVP] wrote:

>Sunit,

That's basically what you will have to do. It seems like you have
three
tasks, but you want all three to run sequentially, just not on the UI
thread. You can still use the BackgroundWorke r class for this, you just
have to do the first task with it, then the second, then the third, and
so
on, and so on.

Hope this helps.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"sjoshi" <sj****@ingr.co mwrote in message
news:11******* **************@ t46g2000cwa.goo glegroups.com.. .

I'm able to use the Bacground Worker class to execute task and pain the
UI. However I want to be able to execute n tasks in order, say n1, then
n2 if there were no errors during n1, n3 so on.

I'm wondering what would be the best way to do this , do I wait in a
do...while loop for each to finish and then start another ?

thanks
Sunit

Ok thanks I need to run a background task for each row ina dset. So I
tried this:

foreach (DAO.DAODatabas e.DatabaseRow backupEntry in
_srvBackupDSet. Database)
{
InitAndStartBac kgroundWorker(b ackupEntry);
while (!_lastRunFinis hed)
{
Thread.Sleep(50 0);
}
if (!_lastRunOk)
break;
}
private void InitAndStartBac kgroundWorker(D AO.DAODatabase. DatabaseRow
backupRow)
{
worker = new BackgroundWorke r();
worker.WorkerRe portsProgress = true;

//Set background worker
worker.DoWork += new DoWorkEventHand ler(worker_DoWo rk);
worker.Progress Changed += new
ProgressChanged EventHandler(wo rker_ProgressCh anged);
worker.RunWorke rCompleted += new
RunWorkerComple tedEventHandler (worker_RunWork erCompleted);

//Start Backup
worker.RunWorke rAsync(backupRo w);
}

void worker_DoWork(o bject sender, DoWorkEventArgs e)
{
_lastRunFinishe d = false;
_backupMgr.Perc entCompleted += new
PercentEventHan dler(_backupMgr _PercentComplet ed);
BackgroundWorke r theWorker = sender as BackgroundWorke r;
DoBackup((DAO.D AODatabase.Data baseRow)e.Argum ent);
}

void worker_RunWorke rCompleted(obje ct sender,
RunWorkerComple tedEventArgs e)
{
if (e.Error != null)
{
_lastRunOk = false ;
UpdateGridForEr ror(e.Error);
}
else
_lastRunOk = true;
_lastRunFinishe d = true;
}
void worker_Progress Changed(object sender, ProgressChanged EventArgs e)
{
//code to UpdateGrid progress bar
}

However now the UI freezes and the 1st job itself never completes; i.e
the run worker_RunWorke rCompleted event is never raised.

It works fine if I just pass a single row.

thanks
Sunit

[sjoshi]

Nicholas Paldino [.NET/C# MVP] wrote:

It looks like you are waiting on a loop until the processing is done.
You should be letting the events that the BackgroundThrea d fires tell you
when you are done instead of waiting for the variables to be set (which you
are doing the wrong way, since you are not locking access to them).
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"sjoshi" <sj****@ingr.co mwrote in message
news:11******** *************@f 1g2000cwa.googl egroups.com...

Nicholas Paldino [.NET/C# MVP] wrote:

Sunit,

That's basically what you will have to do. It seems like you have
three
tasks, but you want all three to run sequentially, just not on the UI
thread. You can still use the BackgroundWorke r class for this, you just
have to do the first task with it, then the second, then the third, and
so
on, and so on.

Hope this helps.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"sjoshi" <sj****@ingr.co mwrote in message
news:11******** *************@t 46g2000cwa.goog legroups.com...
I'm able to use the Bacground Worker class to execute task and pain the
UI. However I want to be able to execute n tasks in order, say n1, then
n2 if there were no errors during n1, n3 so on.

I'm wondering what would be the best way to do this , do I wait in a
do...while loop for each to finish and then start another ?

thanks
Sunit

Ok thanks I need to run a background task for each row ina dset. So I
tried this:

foreach (DAO.DAODatabas e.DatabaseRow backupEntry in
_srvBackupDSet. Database)
{
InitAndStartBac kgroundWorker(b ackupEntry);
while (!_lastRunFinis hed)
{
Thread.Sleep(50 0);
}
if (!_lastRunOk)
break;
}
private void InitAndStartBac kgroundWorker(D AO.DAODatabase. DatabaseRow
backupRow)
{
worker = new BackgroundWorke r();
worker.WorkerRe portsProgress = true;

//Set background worker
worker.DoWork += new DoWorkEventHand ler(worker_DoWo rk);
worker.Progress Changed += new
ProgressChanged EventHandler(wo rker_ProgressCh anged);
worker.RunWorke rCompleted += new
RunWorkerComple tedEventHandler (worker_RunWork erCompleted);

//Start Backup
worker.RunWorke rAsync(backupRo w);
}

void worker_DoWork(o bject sender, DoWorkEventArgs e)
{
_lastRunFinishe d = false;
_backupMgr.Perc entCompleted += new
PercentEventHan dler(_backupMgr _PercentComplet ed);
BackgroundWorke r theWorker = sender as BackgroundWorke r;
DoBackup((DAO.D AODatabase.Data baseRow)e.Argum ent);
}

void worker_RunWorke rCompleted(obje ct sender,
RunWorkerComple tedEventArgs e)
{
if (e.Error != null)
{
_lastRunOk = false ;
UpdateGridForEr ror(e.Error);
}
else
_lastRunOk = true;
_lastRunFinishe d = true;
}
void worker_Progress Changed(object sender, ProgressChanged EventArgs e)
{
//code to UpdateGrid progress bar
}

However now the UI freezes and the 1st job itself never completes; i.e
the run worker_RunWorke rCompleted event is never raised.

It works fine if I just pass a single row.

thanks
Sunit

Yes I forgot about the locking...will that be the reason for freeeze ?
I do set the variables in the RunWorkerComple ted event so that I can
know when to start another one and as such I can create a new
BackgroundWorke r in the InitAndStart method. Is this the correct way to
start next task ?
Can you please let me know what I'm doing wrong here (pseudo code...)

thanks
Sunit

[Peter Duniho]

"sjoshi" <sj****@ingr.co mwrote in message
news:11******** *************@t 46g2000cwa.goog legroups.com...

I'm able to use the Bacground Worker class to execute task and pain the
UI. However I want to be able to execute n tasks in order, say n1, then
n2 if there were no errors during n1, n3 so on.

I'm wondering what would be the best way to do this , do I wait in a
do...while loop for each to finish and then start another ?

If you want tasks processed sequentially, you need a queue of some sort.

One way to do this would be to take advantage of the suggestions offered so
far. You would add the tasks to the queue, and then use the event Brian
mentions to know when to start a new task thread.

Alternatively, you could just keep a single background thread running that
pulls tasks from the queue as long as the queue isn't empty. Once the queue
is empty, the background thread could wait on an EventWaitHandle that the
main thread would set any time it adds something to the queue.

In either approach, you can include logic to check the results of the
previous task before starting the next one and just abort the whole thing if
an error occurs.

For sure, the thread synchronization you've tried in the code you posted
already is wrong. It appears that you're polling (which is why the main UI
freezes up...you never leave the loop until all tasks are done), and you're
checking an unsynchronized flag (which isn't thread-safe).

Pete