468,777 Members | 2,364 Online
Bytes | Developer Community
New Post

Home Posts Topics Members FAQ

Post your question to a community of 468,777 developers. It's quick & easy.

Thread.Sleep slowing down the whole application

Hello,

I have a program that runs a long process. I tried to add a thread
to check if the use hit a cancel button to abort the process.

Dim t As Threading.Thread
t = New Threading.Thread(AddressOf StartProcess)
t.IsBackground = True
t.Priority = ThreadPriority.Highest
t.Start()

Do

Thread.CurrentThread.Sleep(500)
Application.DoEvents()
if bCancel=true then
t.Abort()
exit do
end if

Loop
When I add 'StartProcess' into a thread, it's about twice as slow if it
just ran it without a thread. Could anyone tell me what I'm not doing
right? Thanks. I played around with this all day and I'm lost.
Thanks.

David

Nov 12 '06 #1
7 2344
da*******@gmail.com wrote in news:1163365534.236771.249840
@m7g2000cwm.googlegroups.com:
Hello,

I have a program that runs a long process. I tried to add a thread
to check if the use hit a cancel button to abort the process.

Dim t As Threading.Thread
t = New Threading.Thread(AddressOf StartProcess)
t.IsBackground = True
t.Priority = ThreadPriority.Highest
t.Start()

When I add 'StartProcess' into a thread, it's about twice as slow if it
just ran it without a thread. Could anyone tell me what I'm not doing
right? Thanks. I played around with this all day and I'm lost.
Thanks.

I noticed that you set the thread priority to highest - I think when it's
set to highest, it will not yield CPU time to other processes.

You should leave the thread priority to normal and try again.

Also, take a look at callback functions - depending on the situation,
they're abit better than firing a thread to run a long running function.

Nov 12 '06 #2
On Sun, 12 Nov 2006 21:55:01 GMT, Spam Catcher <sp**********@rogers.com>
wrote:
>Also, take a look at callback functions - depending on the situation,
they're abit better than firing a thread to run a long running function.
Could you give a small example?

--

Dennis K.
Nov 12 '06 #3
Dennis K. <no****@iglou.invalidwrote in
news:1v********************************@4ax.com:
On Sun, 12 Nov 2006 21:55:01 GMT, Spam Catcher <sp**********@rogers.com>
wrote:
>>Also, take a look at callback functions - depending on the situation,
they're abit better than firing a thread to run a long running function.

Could you give a small example?
Here you go:

http://msdn.microsoft.com/library/de...l=/library/en-
us/cpguide/html/cpovrasynchronousprogrammingoverview.asp

You'll probably want the example: "Executing a Callback Method When an
Asynchronous Call Completes" or "Polling for Asynchronous Call Completion".
Nov 13 '06 #4
Thank you very much for the information. I been playing around with
it all day. I have to wait until tomorrow to try this out but I think
this might work.

David
Spam Catcher wrote:
Dennis K. <no****@iglou.invalidwrote in
news:1v********************************@4ax.com:
On Sun, 12 Nov 2006 21:55:01 GMT, Spam Catcher <sp**********@rogers.com>
wrote:
>Also, take a look at callback functions - depending on the situation,
they're abit better than firing a thread to run a long running function.
Could you give a small example?

Here you go:

http://msdn.microsoft.com/library/de...l=/library/en-
us/cpguide/html/cpovrasynchronousprogrammingoverview.asp

You'll probably want the example: "Executing a Callback Method When an
Asynchronous Call Completes" or "Polling for Asynchronous Call Completion".
Nov 13 '06 #5

Dim t As Threading.Thread
t = New Threading.Thread(AddressOf StartProcess)
t.IsBackground = True
t.Priority = ThreadPriority.Highest
t.Start()
Good, you've created a new thread :)
>
Do

Thread.CurrentThread.Sleep(500)
Application.DoEvents()
if bCancel=true then
t.Abort()
exit do
end if

Loop
Ahh, you are sleeping the CurrentThread, which is your main thread, not the
thread you created. So, instead of sleeping there, you should sleep in
StartProcess. Secondly, it's rather bad to call "t.Abort". You will get no
end of problems especially if your thread is handling COM objects. The
accepted pattern for dealing with threads is to flag a "bCancel" inside the
thread that it checks periodically so it can terminate cleanly.

>
When I add 'StartProcess' into a thread, it's about twice as slow if it
just ran it without a thread. Could anyone tell me what I'm not doing
right? Thanks. I played around with this all day and I'm lost.
Thanks.

David

Nov 13 '06 #6
Thanks for the reply. Wouldn't "t.Sleep(1000)' just slow down the
whole thread? I tried it out and have similar results. What I want
to try to do is run a thread and have another thread refresh the form
and check the cancel button without slowing down the 'StartProcess'
thread.

When I remove the thread itself and just run 'StartProcess', the time
decreases in half.

Thanks again.

David
Robinson wrote:
Dim t As Threading.Thread
t = New Threading.Thread(AddressOf StartProcess)
t.IsBackground = True
t.Priority = ThreadPriority.Highest
t.Start()

Good, you've created a new thread :)

Do

Thread.CurrentThread.Sleep(500)
Application.DoEvents()
if bCancel=true then
t.Abort()
exit do
end if

Loop

Ahh, you are sleeping the CurrentThread, which is your main thread, not the
thread you created. So, instead of sleeping there, you should sleep in
StartProcess. Secondly, it's rather bad to call "t.Abort". You will get no
end of problems especially if your thread is handling COM objects. The
accepted pattern for dealing with threads is to flag a "bCancel" inside the
thread that it checks periodically so it can terminate cleanly.


When I add 'StartProcess' into a thread, it's about twice as slow if it
just ran it without a thread. Could anyone tell me what I'm not doing
right? Thanks. I played around with this all day and I'm lost.
Thanks.

David
Nov 13 '06 #7

<da*******@gmail.comwrote in message
news:11**********************@i42g2000cwa.googlegr oups.com...
Thanks for the reply. Wouldn't "t.Sleep(1000)' just slow down the
whole thread? I tried it out and have similar results. What I want
to try to do is run a thread and have another thread refresh the form
and check the cancel button without slowing down the 'StartProcess'
thread.

When I remove the thread itself and just run 'StartProcess', the time
decreases in half.

Thanks again.

David
Robinson wrote:
Dim t As Threading.Thread
t = New Threading.Thread(AddressOf StartProcess)
t.IsBackground = True
t.Priority = ThreadPriority.Highest
t.Start()

Good, you've created a new thread :)
>
Do

Thread.CurrentThread.Sleep(500)
Application.DoEvents()
if bCancel=true then
t.Abort()
exit do
end if

Loop

Ahh, you are sleeping the CurrentThread, which is your main thread, not
the
thread you created. So, instead of sleeping there, you should sleep in
StartProcess. Secondly, it's rather bad to call "t.Abort". You will get
no
end of problems especially if your thread is handling COM objects. The
accepted pattern for dealing with threads is to flag a "bCancel" inside
the
thread that it checks periodically so it can terminate cleanly.

>
When I add 'StartProcess' into a thread, it's about twice as slow if it
just ran it without a thread. Could anyone tell me what I'm not doing
right? Thanks. I played around with this all day and I'm lost.
Thanks.

David
Oh okay, I misunderstood. I thought you were wanting to sleep the worker
thread so it didn't take too much time away from the main GUI thread (which
is the more normal scenario). I suggest you re-archietect what you are
doing slightly. Let your worker thread run free inside a loop checking for
"cancelled = true", at which point it exits. You then can "fire and forget"
the worker thread.

When your main thread is finished, it can "invoke" a delegate on your form
to say it's finished, or a delegate to say it was cancelled or a delegate to
say it was failed (or you can put all 3 into the same method).

Something like this:

User Action
Create Thread
Start Thread (
[Thread loops doing it's business]
[If bCancelled Then Exit Thread]
Exit:
Invoke Completed Method on Main Form )

Completed:
Update UI

Nov 13 '06 #8

This discussion thread is closed

Replies have been disabled for this discussion.

Similar topics

4 posts views Thread by Matthew Groch | last post: by
22 posts views Thread by Brett | last post: by
5 posts views Thread by John A. Bailo | last post: by
11 posts views Thread by Jon Slaughter | last post: by
10 posts views Thread by Paul E Collins | last post: by
1 post views Thread by =?Utf-8?B?QU1lcmNlcg==?= | last post: by
3 posts views Thread by stumorgan | last post: by
reply views Thread by zhoujie | last post: by
By using this site, you agree to our Privacy Policy and Terms of Use.