|
If I recall well, that isn't the whole story, The CWinThread pointer is deleted too.
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke
[My articles]
|
|
|
|
|
CPallini wrote: If I recall well, that isn't the whole story, The CWinThread pointer is deleted too.
You recall correctly! 
There's a delete this; in the destructor as well that is executed if m_bAutoDelete is set to TRUE.
I omitted it for brevity since I considered the name of the variable m_bAutoDelete rather self-explanatory. 
But the only thing the OP need is to read Joe's article over and over again until it's understood. Really.
"It's supposed to be hard, otherwise anybody could do it!" - selfquote
"High speed never compensates for wrong direction!" - unknown
|
|
|
|
|
Roger Stoltz wrote: But the only thing the OP need is to read Joe's article over and over again until it's understood. Really.
"Softly spoken magic spells".
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke
[My articles]
|
|
|
|
|
The issues with what you've done are:
1. You haven't shown us what you're doing inside the 'run' function. (can you please rename it to something other than run, I'm just terrified looking at the context - CWinThread::Run() ... you get the picture)
2. Your WaitFor... call does not 'wait', it just performs a check on a thread which might be running or might be dead, and then attempts to delete something which (again) may have been already dead (you haven't set the m_bAutoDelete to TRUE).
You may first try to run a thread successfully, and then try using Events with WaitFor... functions. Simulate processing with Sleep(10) within a loop (for learning purposes) and then shut it down gracefully by setting an Event. You need not 'delete' the CWinThread object, like I said. Give that excellent article another read, and you will be all set. Post a reply, if you're stuck somewhere.
It is a crappy thing, but it's life -^ Carlo Pallini
|
|
|
|
|
1. Run is simple displaying function. which has nothing to do with CWinThread::Run().
Now it looks like below...
CWinThread * myWorkerThread;
myWorkerThread = AfxBeginThread(run, this);
myWorkerThread->m_bAutoDelete = TRUE;
UINT run(LPVOID pParam)
{
..........
............
// display the data
}
In ~Destructor()
{
DWORD result =WaitForSingleObject(myWorkerThread->m_hThread,0);
if(result == WAIT_OBJECT_0)
delete myWorkerThread;
}
2. In WaitFor... I have given time 0 as I require only to check the status of the thread.
|
|
|
|
|
mikert_2008 wrote:
CWinThread * myWorkerThread;
myWorkerThread = AfxBeginThread(run, this);
myWorkerThread->m_bAutoDelete = TRUE;
In all the confusion from the discussion earlier today you're partly forgiven, but consider the following for spawning the worker thread:
CWinThread* pThread;
pThread = AfxBeginThread( run, this, 0, 0, CREATE_SUSPENDED, NULL );
if( pThread )
{
pThread->m_bAutodelete = FALSE;
pThread->ResumeThread();
}
What kind of event stops the worker thread from running?
The way I interpret your destructor is that the thread does its job and then terminates by itself.
What does the thread do? Your comment "display the data" worries me since you should not touch the GUI from a worker thread in order to avoid deadlocks.
Probably you shouldn't be using multithreading at all for the problem you're trying to solve, since you seem to be doing thing sequentially and not in parallel.
Your destructor doesn't really make sense. There's also a race condition that in most cases would generate an exception when you're trying to free memory that has already been freed since you haven't set m_bAutoDelete to FALSE. You're mixing manual and automatic deletion of the CWinThread object.
You should also make sure all thread spawned by your program are terminated before allowing the program to exit. It's considered Best Practice because when you exit your program you don't have the power to do anything about it any longer.
The most harsh way the code that waits for the thread to exit could be like this:
if( pThread )
{
if( ::WaitForSingleObject( pThread->m_hThread, INFINITE ) == WAIT_OBJECT_0 )
{
delete pThread;
pThread = NULL;
}
}
You need to get back to the article and read it again as the above suggests that you haven't read it carefully enough and apparently have not understood it properly.
Believe me, I've suggested Joe's article to fellow programmers since he wrote it almost a decade ago. In my opinion it's the best article that can be found on the internet describing multithreading, especially together with MFC framework.
"It's supposed to be hard, otherwise anybody could do it!" - selfquote
"High speed never compensates for wrong direction!" - unknown
|
|
|
|
|
mikert_2008 wrote: This I will put in the destructor of class (which starts the thread)...
You're starting a thread from within the destructor? 
"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
|
|
|
|
|
You need two CEvent objects, one representing the request from the primary thread to stop, and the other representing the acknowledgment from the secondary thread that it has stopped. When the primary thread responds to a stop request (e.g., Alt+F4, X, Cancel), it signals the first event and then waits for the second event. In the secondary thread, call WaitForSingleObject() on that event with 0 as the wait time. If it comes back as signaled, signal the other event.
"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 There.
I am not able to use C# Managed DLL into my VC++ Application (DLL).
Below are the steps I am doing but with no success.
1. Copied C# DLL into the folder where all my lib files reside.
2. Included #using :GT:JPEGMaker.dll:LT: statement into stdafx.h file
3. After this I got an error message regarding CLR Support, so I enabled the same in project settings.
4. After this I got a linking error...
:GT: Greater Then Sign
:LT: Less Then Sign
Please provide details or Do I need to convert C# DLL into Native DLL?
Thanks
PanB
|
|
|
|
|
PankajB wrote: Please provide details or Do I need to convert C# DLL into Native DLL?
No, you need to re-post this in the correct forum[^] and then provide some detail, like what linker error you got...
Java, Basic, who cares - it's all a bunch of tree-hugging hippy cr*p
|
|
|
|
|
Ohh Thanks. Yes thats a correct group to talk in...
|
|
|
|
|
I want to use Hindi in my multilingual application. So can anyone tell me that "Which one is the best font for Hindi language"?
thank you.
|
|
|
|
|
Please let me google that for you [^].
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke
[My articles]
|
|
|
|
|
I have done that already.
|
|
|
|
|
Fixedsys 
|
|
|
|
|
There is no one 'best font' for any language. If you are writing an Unicode application, the Devnagari script would be displayed automatically (by default) with Mangal. You need not worry about fonts, if you are working on an Unicode aware application.
It is a crappy thing, but it's life -^ Carlo Pallini
|
|
|
|
|
When VC++ "LineTo" statement is called very often (say every 100 ms), it is using more resource and it is not getting released. If my application is running for longer period (say 2 hours), high usage of resource problem causing more RAM usage and memory is not getting released and growing continuously.
Sample code is given below (Please send your personal email id, so that I can email complete code)
//------------------------------------------------------------------
//Callback method creation (100 ms)
m_nEventID = timeSetEvent(100,0,GeneratePeriodicCall,DWORD(this),TIME_PERIODIC);
//callback method
void CALLBACK GeneratePeriodicCall(UINT uID, UINT UMsg, DWORD dwUser, DWORD dw1, DWORD dw2)
{
CTestAppView* pView = (CTestAppView*) dwUser;
//kill thread if active
if(g_hDrawChartThread != INVALID_HANDLE_VALUE)
{
DWORD dwExitCode =NULL;
DWORD dw = NULL;
GetExitCodeThread(g_hDrawChartThread, &dwExitCode);
if(dwExitCode == STILL_ACTIVE)
{
::CloseHandle(g_hDrawChartThread);
g_hDrawChartThread = NULL;
}
else
{
g_hDrawChartThread = NULL;
}
}
//create thread
g_hDrawChartThread = CreateThread(NULL,0,DrawChart,pView,0,0);
}
//Thread call
DWORD __stdcall DrawChart(LPVOID lParam)
{
CTestAppView* pView = (CTestAppView*)lParam;
CClientDC dc(pView);
int nSaveDC = dc.SaveDC();
long x = 0;
long y = 0;
for(long i = 0 ; i < 100000 ;i++)
{
dc.MoveTo(x,y);
//THIS LINE IS CAUSING SERIOUS RESOURCE LEAK
dc.LineTo(x + 1, y + 1);
x = x + 1;
y = y + 1;
}
dc.RestoreDC(nSaveDC);
return 0;
}
//------------------------------------------------------------------
Anyone can clarify me why "LineTo()" is not releasing resources & utilising more RAM. This RAM memory is not getting released until unless I close my VC++ application.
Any solution/hint, welcome.
Thanks in advance,
Madhu
|
|
|
|
|
I don't know exactly why you encounter a memory leak, but one think that is sure is that you are doing it the wrong way: you should never access your user interface from a separate thread (which is what you are doing here). This is not the way to do it. Instead, in a separate thread, you can add data in a std::vector (for instance) which is protected against multi-threading access and then you send a message to the UI to refresh itself. The UI will then (in the main thread) extract information from the vector and draw the additional points.
I don't know if this will fix your problem but this is something you'll have to fix or you'll encounter problems in the future.
|
|
|
|
|
I don't think LineTo() causes high usage of resource&RAM. You are exactly doing it in wrong way.
Firstly, you need to understand the callback function of a timer in VC can be re-called while previous call is not completed yet. Basically, each callback will create a thread.
Secondly, I don't understand why you create a thread in the callback function, which will lead to run many threads at the same time.
So i think running many threads at the same time causes your problem.
If you want to update you UI, you can set a timer or a thread in your UI class which gets data and draw them on UI periodically. Meanwhile, there is another thead to prepare those drawing data.
Welcome to www.softwaretree.net!
This website is generated completely by static html pages transforming technology.
You can find many excellent audio/video tools there!
|
|
|
|
|
adepumadhu1 wrote: g_hDrawChartThread = CreateThread(NULL,0,DrawChart,pView,0,0);
Are you calling this function 10 times per second?
"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
|
|
|
|
|
The program is to keep records and perform statistical for a class of students.
the class may have up to 40 students. They have 5 quizzes, and all the students are identified by a 4 digit student number
and the program will print the students scores and calculate and print the statistics for each quiz
Student Quiz 1 Quiz 2 Quiz 3 Quiz 4 Quiz 5
1234 78 83 87 91 86
2134 67 77 84 82 79
3124 77 89 93 87 71
High Score 78 89 93 91 86
Low Score 67 77 84 82 71
Average 73.4 83.0 88.2 86.6 78.6
but this is what i have so far
#include<stdio.h>
#define ROW 4
#define COLS 5
int main(void)
{
// Local Declarations
int table [ROW] [COLS] =
{
{1234,52,7,100,78,34}'
{2134,90,36,90,77,30},
{3124,100,45,20,90,70},
{4532,11,17,81,32,77}
};
int line [ROW * COLS];
// Statments
for (int row = 0; row <ROWS;row++)
for (int column =0; column < cols, column++)
line[row * cols + column] = table [row] [column];
for (int row = 0; row <ROWS * cols; row++)
printf("%02d", line [row]);
return 0;
} // main
please help
modified on Thursday, April 30, 2009 1:56 AM
|
|
|
|
|
I don't want to reveal my age, but the last time I did a homework was 10 years ago.
You need to google first, if you have "It's urgent please" mentioned in your question.
 _AnShUmAn_
|
|
|
|
|
Okay
and thanks for nothing
|
|
|
|
|
biggiant22000 wrote: please help
With what?
|
|
|
|
|
biggiant22000 wrote: #define COLS 5
There are 6 "columns" in the table array.
biggiant22000 wrote: int line [ROW * COLS];
This variable is not necessary.
biggiant22000 wrote: {1234,52,7,100,78,34}'
...
for (int row = 0; row < ROWS;row++)
for (int column =0; column < cols, column++)
line[row * cols + column] = table [row] [column];
How do these even compile?
With a little bit of coercion , your code could produce something like:
Student Quiz 1 Quiz 2 Quiz 3 Quiz 4 Quiz 5
------- ------ ------ ------ ------ ------
1234 52 7 100 78 34
2134 90 36 90 77 30
3124 100 45 20 90 70
4532 11 17 81 32 77
High Score 100 45 100 90 77
"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
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
