Help with thread.start...

VMI <vo******@yahoo .com> wrote:

In the following code, how can I display the MessageBox once MyThread has
finished executing? updateFiles method copies 5 humongous files to the HDD
but for some reason, the MessageBox is displayed before updateFles has
finished executing.

Of course - precisely because you're doing it in a different thread!

Why not just put the message box at the end of the updateFiles method?

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

Because this method is conditioned by an If statement. And there are several
lines of code that run after the Threads regardless of what thread is
executed. The code's like this:
Thread MyThread = null;
if (execute)
{
MyThread = new Thread(new ThreadStart (updateFiles));
MyThread.Start( );
}
else
{
MyThread = new Thread(new ThreadStart (updateFiles2)) ;
MyThread.Start( );
}

/* Lines of code that are run after files are copied */

"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** *@msnews.micros oft.com...

VMI <vo******@yahoo .com> wrote:

In the following code, how can I display the MessageBox once MyThread has
finished executing? updateFiles method copies 5 humongous files to the
HDD
but for some reason, the MessageBox is displayed before updateFles has
finished executing.

Of course - precisely because you're doing it in a different thread!

Why not just put the message box at the end of the updateFiles method?

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

You have to call MessageBox.Show ("Program has ended");
at the end of your updateFiles procedure.

Willy.

"VMI" <vo******@yahoo .com> wrote in message
news:%2******** ********@TK2MSF TNGP14.phx.gbl. ..

In the following code, how can I display the MessageBox once MyThread has
finished executing? updateFiles method copies 5 humongous files to the HDD
but for some reason, the MessageBox is displayed before updateFles has
finished executing.

This is the code:

MyThread = new Thread(new ThreadStart (updateFiles));
MyThread.Start( );
MessageBox.Show ("Program has ended");
private void updateFiles()
{
/* several File.Copy calls of five 400MB files */
}

Thanks.

VMI <vo******@yahoo .com> wrote:

Because this method is conditioned by an If statement. And there are several
lines of code that run after the Threads regardless of what thread is
executed. The code's like this:
Thread MyThread = null;
if (execute)
{
MyThread = new Thread(new ThreadStart (updateFiles));
MyThread.Start( );
}
else
{
MyThread = new Thread(new ThreadStart (updateFiles2)) ;
MyThread.Start( );
}

/* Lines of code that are run after files are copied */

It sounds like you actually want:

ThreadStart threadJob;

if (execute)
{
threadJob = new ThreadStart(upd ateFiles);
}
else
{
threadJob = new ThreadStart(upd ateFiles2);
}
new Thread(threadJo b).Start();

However, either way, those "lines of code that are run after files are
copied" will be executed pretty much immediately, while the files are
being copied. If you didn't want it to happen in the background, why
bother starting it in a different thread in the first place? You could
call Thread.Join, but there seems to be very little point in starting a
thread only to join it immediately.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

I did it with threads because, once the files start being copied, the
application window looks like it's frozen. So the user might think the
program froze when it's actually copying one of the really huge files. I
want the user to be able to move the window around while the files are being
copied.
"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** @msnews.microso ft.com...

VMI <vo******@yahoo .com> wrote:

Because this method is conditioned by an If statement. And there are
several
lines of code that run after the Threads regardless of what thread is
executed. The code's like this:
Thread MyThread = null;
if (execute)
{
MyThread = new Thread(new ThreadStart (updateFiles));
MyThread.Start( );
}
else
{
MyThread = new Thread(new ThreadStart (updateFiles2)) ;
MyThread.Start( );
}

/* Lines of code that are run after files are copied */

It sounds like you actually want:

ThreadStart threadJob;

if (execute)
{
threadJob = new ThreadStart(upd ateFiles);
}
else
{
threadJob = new ThreadStart(upd ateFiles2);
}
new Thread(threadJo b).Start();

However, either way, those "lines of code that are run after files are
copied" will be executed pretty much immediately, while the files are
being copied. If you didn't want it to happen in the background, why
bother starting it in a different thread in the first place? You could
call Thread.Join, but there seems to be very little point in starting a
thread only to join it immediately.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

VMI <vo******@yahoo .com> wrote:

I did it with threads because, once the files start being copied, the
application window looks like it's frozen. So the user might think the
program froze when it's actually copying one of the really huge files. I
want the user to be able to move the window around while the files are being
copied.

Right. So you do indeed need a new thread, but you can't just put code
after the Thread.Start call. You should make your copying methods tell
the UI thread when they're finished.

See http://www.pobox.com/~skeet/csharp/t...winforms.shtml for more
information.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

Hello VMI,

You should use event to notify the caller of the completion of process. I'm
going to assume that updateFiles procedure exists within the same class.
Here is the sample.

private delegate void ProcessComplete dEventHandler() ;
private event ProcessComplete dEventHandler ProcessComplete d;
private void button1_Click(o bject sender, System.EventArg s e)
{
// bind the event to OnProcessComple ted
// This should preferably done within constructor of the current class
this.ProcessCom pleted += new ProcessComplete dEventHandler(O nProcessComplet ed);

Thread t = new Thread(new ThreadStart(Lon gRunningProcess ));
t.Start();
}

private void LongRunningProc ess()
{
// Long Running Process here

// Process finished, now raise the event.
ProcessComplete d();
}

private void OnProcessComple ted()
{
MessageBox.Show ("Process Completed");
}

Hope this helps.

In the following code, how can I display the MessageBox once MyThread
has finished executing? updateFiles method copies 5 humongous files to
the HDD but for some reason, the MessageBox is displayed before
updateFles has finished executing.

This is the code:

MyThread = new Thread(new ThreadStart (updateFiles));
MyThread.Start( );
MessageBox.Show ("Program has ended");
private void updateFiles()
{
/* several File.Copy calls of five 400MB files */
}
Thanks.