|
Hello Everyone,,
Sorry saw all yr response now.Thanks a lot for the reply.
I am very much new to the Thread Concept. thought of learning the basics of threads.
can anyone suggest me some nice Article to start with.
Thanking you,
Suresh
|
|
|
|
|
There are plenty of articles around, but before all that, do you have a book on MFC programming? If not, please buy one. My book recommendations for MFC are here[^]
Essay recommendation:
Worker Threads[^]
UI Threads[^]
There are plenty of articles at CP as well, just do a search.
It is a crappy thing, but it's life -^ Carlo Pallini
|
|
|
|
|
Message Closed
modified 14-Oct-15 22:14pm.
|
|
|
|
|
You can forget Terminate*(TerminateThread/TerminateProcess).
MSDN:
TerminateThread is a dangerous function. Use it only in the most extreme cases.
Why 1.00/5 ???
Of one Essence is the human race
thus has Creation put the base
One Limb impacted is sufficient
For all Others to feel the Mace
(Saadi )
modified on Tuesday, April 14, 2009 5:54 AM
|
|
|
|
|
That's probably one of the worst way to do that. You should never call TerminateThread (unless in very specific conditions). In general, there are much better ways to achieve what you want.
|
|
|
|
|
Hi Parag Patel,
Thanks for the reply.
I tried with this below code but still the thread is running its not terminated,
can u please tell how to kill the thread.
void CThreadDemoDlg::OnEnd()
{
TerminateThread(hr1,1);
TerminateThread(hr2,1);
}
|
|
|
|
|
Having looked at your code in your first post, I can tell you that you MUST NOT use TerminateThread(). It's saddening to see you've picked the worst suggestion that was given to you.
It is a crappy thing, but it's life -^ Carlo Pallini
|
|
|
|
|
Rajesh R Subramanian wrote: It's saddening to see you've picked the worst suggestion that was given to you.
That's usually the case because it looks like the easiest solution
|
|
|
|
|
Maybe you didn't see the replies to his post but it is strongly recommanded not to use TerminateThread.
A better approach would be to pass a pointer to your dialog class to the thread function, and within the thread cast it back to your dialog and call a public function on it. Inside that function you put the code you already provided except that in your loop, you also check if a flag is set or not (this flag is a simple bool which is a member of the dialog). If the flag is set, you simply stop the loop and the function terminates.
When you click on the end button, you can simply set this flag so that the loop stops.
|
|
|
|
|
Hello Cédric Moonen,,
Sorry saw yr response now.Thanks a lot for the reply.
I am very much new to the Thread Concept. thought of learning the basics of threads.
can anyone suggest me some nice basic Article to start with.
Thanking you,
Suresh
|
|
|
|
|
This[^] article is one of the best I know of.
|
|
|
|
|
It looks like we both gave him the same article to read, at the same time (20 mins back, as of now).
It is a crappy thing, but it's life -^ Carlo Pallini
|
|
|
|
|
It seems that somebody is voting 1 to all of our messages. Can you guess who it is ?
|
|
|
|
|
I know, but I'm ignoring that.
I've been balancing the votes. Forget it, the votes will be balanced by the regulars.
It is a crappy thing, but it's life -^ Carlo Pallini
|
|
|
|
|
Can you paste your code snap here?
You can also use flag mechanism, its good solution.
HANDLE hr1,hr2;
bool stopTh;
void CThreadDemoDlg::OnStart()
{
stopTh = false;
hr1 = CreateThread(NULL,0,(unsigned long (__stdcall *)(void *))WorkerThreadOne,this,0,0);
hr2 = CreateThread(NULL,0,(unsigned long (__stdcall *)(void *))WorkerThreadTwo,this,0,0);
}
void CThreadDemoDlg::OnEnd()
{
stopTh = true;
}
UINT WorkerThreadOne(LPVOID Param)
{
fstream OutFile;
OutFile.open("FileOne.txt",ios::out);
for(int i=0;i<10000;i++)
{
if(stopTh == true)
break;
OutFile << i << " ";
}
OutFile.close();
return true;
}
Why 1.0/5.0?
Parag Patel
modified on Thursday, April 16, 2009 9:40 AM
|
|
|
|
|
ParagPatel wrote: if(stopTh == true)
Unless stopTh is volatile , the compiler will optimize out this check.
"Old age is like a bank account. You withdraw later in life what you have deposited along the way." - Unknown
"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons
|
|
|
|
|
Thanks ,
Yes.. stopTH must be globle or thread function must use through passed class object.
Parag Patel
Sr. Software Eng, Varaha Systems
|
|
|
|
|
ParagPatel wrote: Thanks Simmons,
ParagPatel wrote: Yes.. stopTH must be globle
Being global has nothing to do with it. If the compiler detects that nothing in the loop is changing that variable, it will optimize out the check. So even if the secondary thread changes that variable, it will go unseen.
"Old age is like a bank account. You withdraw later in life what you have deposited along the way." - Unknown
"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons
|
|
|
|
|
Changing it through the class object = changing it from a different thread (the main thread of the app). Which means it will be optimized away.
It is a crappy thing, but it's life -^ Carlo Pallini
|
|
|
|
|
Did you see bottom of CreateThread on the MSDN it explains it for you.
Of one Essence is the human race
thus has Creation put the base
One Limb impacted is sufficient
For all Others to feel the Mace
(Saadi )
|
|
|
|
|
Suresh H wrote: for(int i=0;i<10000;i++)
You need to add another condition to this loop, something like:
CEvent m_event;
for (int i = 0; i < 10000; i++)
{
if (WaitForSingleObject(m_event.m_hObject, 0) == WAIT_OBJECT_0)
break;
}
"Old age is like a bank account. You withdraw later in life what you have deposited along the way." - Unknown
"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons
|
|
|
|
|
Hi all,
I have to work on a small project in VC++ using "Visual Studio.net 2003".
I have to use data base in my project.
Do I need to install anything to use ODBC or is it built in.
If yes, please tell met the softwares that are to be installed to use database.
Regards,
Sunil Kumar
|
|
|
|
|
See ODBC[^]
You need to google first, if you have "It's urgent please" mentioned in your question.
_AnShUmAn_
|
|
|
|
|
Hi all,
I am trying to write a lightweight Windows utility with C & Win32 APIs only.
I have setup a timer and would like to display some information on the window title bar (not dialog box) at a set time interval.
I tried to use the SetWindowText function but it doesn't seem to work.
Any help or pointers would be appreciated! Thanks.
M.
|
|
|
|
|
Did you use of SetWindowText on WM_TIMER?
Of one Essence is the human race
thus has Creation put the base
One Limb impacted is sufficient
For all Others to feel the Mace
(Saadi )
|
|
|
|