Ich habe hier ein Problem mit meinem Thread. Starten und laufen und so tut er wunderbar, nur wenn z.B. ein Button gedrückt wird, soll der Thread sich beenden und das Dialogfenster schließen.

Hab ich folgendermaßen implementiert:

//Header

void Starten();
static UINT AuswertungStarten (LPVOID pParam);

//CPP

//Starten()

//hier kommt ne ganze Reihe von Code und in der Schleife ist folgende bedingung:

if ( abbruch ) //wird mit true gefüllt wenn button gedrückt wird
{
     AfxEndThread(1,TRUE);
}

//AuswertungStarten()
UINT CClassDlg::AuswertungStarten(LPVOID pParam)
{
	CClassDlg* pDlg = (CClassDlgg*) pParam;
	pDlg->Starten();
	return 0; 
}

//Cancel Button

this->abbruch = TRUE;
DWORD ExitCode;
GetExitCodeThread(AuswertungStarten,&ExitCode);
if ( ExitCode == 1 )
    OnCancel();

Starten tue ich den Thread wie folgt:

AfxBeginThread(AuswertungStarten,this);

Komischerweise steht direkt vor dem starten des Threads, nach starten des Threads und nach beedigung des Threads immer der gleiche Wert in ExitCode.

Ich hab keine Ahnung woran das liegt.

aber dann müsste er zugehen, wenn ich das zweite mal druf klicke

das tut er ja auch net

1. Speichere irgendwo den gewonnen Zeiger von AfxBeginThread, wohl am besten in eine Membervariable.

//Cancel Button

this->abbruch = TRUE;
while(WaitForSingleObject(m_pThread->m_hThread, INFINITE) != WAIT_OBJECT_0);
DWORD ExitCode;
GetExitCodeThread(m_pThread->m_hThread,&ExitCode);
if(ExitCode == 1)
    OnCancel();

2. Warte mit WaitForSingleObject und dem Handle des Threads darauf, bis der Thread auch beendet wurde.

3. Wie machst du das eigentlich mit dem abbruch? Im Cancel Button Code hast du ein this->abbruch aber im Thread nur ein abbruch. Ich hoffe du weisst, was du da tust *g*

Grüssli

hab nur das this vergessen *zwinka*

aber das hört sich gut an

ich proebiere das gleich aus

danke schonmal

Edit: .. machst Du schon ...war gestern bei Robbie

DWORD dwExitCode = 0;
GetExitCodeThread(m_pThread->m_hThread, &dwExitCode);

Grüssli

dann beende ich den thread

Gerade noch einen Fehler entdeckt. Mich hat das GetExitCodeThread schon vorhin gestört, dass du dem einen Zeiger auf deine Funktion übergibst, doch ich habe damit noch nie gearbeitet und deshalb nicht weiterüberlegt. Nun habe ich allerdings endlich in der MSDN nachgeschaut und siehe da ... GetExitCodeThread verlangt ein Handle auf deinen Thread und einen Zeiger auf eine DWORD Variable. Ich habe es oben editiert und führe es hier nochmals auf:

DWORD dwExitCode = 0;
GetExitCodeThread(m_pThread->m_hThread, &dwExitCode);

Grüssli

jo sicher; stand da ja auch; aber da ich den zeiger bis eben noch nicht zugewiesen hatte, habe ich halt die funktion übergeben

dankeschön

m_pThread = AfxBeginThread(AuswertungStarten, (LPVOID)this, THREAD_PRIORITY_NORMAL, 0, CREATE_SUSPENDED);
if(m_pThread != NULl)
{
    m_pThread->m_bAutoDelete = FALSE;
    m_pThread->ResumeThread();
}
else
{
    // Thread konnte nicht gestartet werden.
}

Grüssli

hab ich jetzt alles auch so gemacht

nur: wie lösche ich den thread? einfach mit Delete(); ?

Edit: Neee, er wirft mir zwar keinen Fehler aus, aber dafür Memory Leaks *wein*