Есть переменная типа CWinThread;
Завершаю программу способом "по флагу остановки"
То есть в процедуре потока есть фича вида

Код

while(!my_flag) { /* Выполняем какие-то действия. Действия могут выполняться до 1 секунды. */ }

Ну а в мое классе есть фича вида

Код

BOOL MyDialog::DestroyWindow() { my_flag=TRUE; return CDialog::DestroyWindow(); }

Cуть проблемы в том, что период опроса флага достаточно большой. И если получается так, что мой основной поток уже зарылся, а тот еще продолжает работать, то выдается ошибка о неосвобождении памяти. Если же в основном потоке после присвоения переменной my_flag поставить Sleep(1010), то ошибка исчезает. Но это некрасиво, надо точно знать, сколько времени на завершение потока понадобится.
Поэтому хотелось бы знать функцию, которая вернет состояние потока, и ждать, пока поток не завершится самостоятельно, а уже потом убивать главное окно.
Пробовал ждать с помощью синхронизации - при запуске рабочего потока входить в критическую секцию, и выходить при завершении потока. А в главном потоке в DestroyWindow после установки флага завершения потока входить в эту же критическую секцию, тем самым усыпляя поток до момента выхода из критической секции второго потока. Работает, но как придется, то есть все равно какие-то доли секунды уходят на накладные расходы, за которые успевает закрыться основной поток. Здесь тоже помогало использование Sleep(10), но сами понимаете, что это некрасивый выход. Короче, надо как-то узнавать состояние потока. Причем уверен, что это просто. Но не знаю, какой функцией пользоваться.

GetExitCodeThread() или можно ждать на HANDLE - WaitForSingleObject()

Если CWinThread с автоудалением, то тогда CWinThread * может быть недействительным.

JoyEx
GetExitCodeThread() почему-то не работает, функция всегда STILL_ACTIVE.
Вот мой код:

Код

/*Wrtd - указатель на поток, полученный при AfxBeginThread()*/     DWORD TEMP;     BOOL b;     HANDLE hnd=HANDLE(*this->Wtrd);     do     {         b=::GetExitCodeThread(hnd,&TEMP);     }     while(TEMP==STILL_ACTIVE);

А Конструкция вида

Код

WaitForSingleObject(hnd,10000);

Помогает, спасибо!!!
Но я не понимаю, какое событие она отлавливает - почему срабатывает именно по выходу из потока, то есть именно так, как мне нужно.

Это сообщение отредактировал(а) Dreamer_0x01 - 18.8.2005, 11:17

Цитата(Dreamer_0x01 @ 18.8.2005, 11:04)

JoyEx GetExitCodeThread() почему-то не работает, функция всегда STILL_ACTIVE.

ну да, это же ведь Get - ExitCode - Thread пока поток жив, а когда он завершится, то тогда и получишь то, что вернет его управл. ф-ция
И потом, надо дублировать HANDLE потому, что его может закрыть сам CWinThread или CWinThread::m_bAutoDelete = FALSE

Цитата(Dreamer_0x01 @ 18.8.2005, 11:04)

А Конструкция вида код C++ 1: WaitForSingleObject(hnd,10000); Помогает, спасибо!!! Но я не понимаю, какое событие она отлавливает - почему срабатывает именно по выходу из потока, то есть именно так, как мне нужно.

В MSDN пишут:
The WaitForSingleObject function can wait for the following objects:

• Change notification
• Console input
• Event
• Job
• Memory resource notification
• Mutex
• Process
• Semaphore
• Thread
• Waitable timer

Из HANDLE Thread WaitForSingleObject() узнает жив ли еще поток, не событие это вовсе

Это сообщение отредактировал(а) JoyEx - 18.8.2005, 15:04

Цитата

ну да, это же ведь Get - ExitCode - Thread  пока поток жив, а когда он завершится, то тогда и получишь то, что вернет его управл. ф-ция

В том то и дело, что у меня поток уже завершился, а в TEMP все равно STILL_ACTIVE сидит...

Dreamer_0x01
Проверь, что вернула ф-ция + еще неплохо бы инициализировать TEMP

У тебя наверное первый раз ф-ция записала в TEMP STILL_ACTIVE, а потом туда ничего не писАла, а ты не проверял что (наверное, ЛОЖЬ) она возвращала.

Это сообщение отредактировал(а) JoyEx - 18.8.2005, 16:19

JoyEx

Нет, смотрел в отладчике, функция всегда 1 возвращала.

А вот инициализация TEMP помогла!!! Видать, она этот кусок кода как-то соптимизировала по-своему. Теперь если перед вызовом функции обнулять TEMP, то все работает как надо.

Но все равно мне больше понравился метод с WaitForSingleObject. Так вроде бы даже грамотнее, не только проще.

Dreamer_0x01

Цитата

А вот инициализация TEMP помогла!!!

Ты в "семерке" работаешь?

Coocky

Цитата

Ты в "семерке" работаешь?

Да

Dreamer_0x01
Теперь понятно, откуда такое отношение к инициализации.
Надеешься на студию
Я еще с шестерки ВСЕГДА сам инициализирую переменные.Столько проблем решается..

Coocky
А можешь пояснить, почему?

Dreamer_0x01

Цитата

А можешь пояснить, почему?

А ты не инициализируй, и пройдись в отладке через переменную..

Ну это-то понятно, что там мусор будет.
Но если она используется в функции, в которой все равно изменится ее значение - то зачем предварительная инициализация нужна?

Dreamer_0x01

Цитата

Но если она используется в функции, в которой все равно изменится ее значение - то зачем предварительная инициализация нужна?

Ну дык откуда ты знаешь, что она не пользуется значением, передаваемым в функцию где-то внутри этой функции?
То что она возвращает значение по ссылке, еще не чего не значит..
Может внутри есть такое..

Код

switch (TEMP) { case ... : ...... ; case ... : ...... ; case ... : ...... ; case ... : ...... ; default :...............; }

А если ТЕМР мусор, то что передал , то и получишь
Во всяком случае поверь моему опыту (хоть и маленькому)

Цитата

Во всяком случае поверь моему опыту (хоть и маленькому)

Ладно те пребедняться

Цитата

Может внутри есть такое.. Код  switch (TEMP) { case ... : ...... ; case ... : ...... ; case ... : ...... ; case ... : ...... ; default :...............; } А если ТЕМР мусор, то что передал , то и получишь

Хорошо,но как тогда узнать, какими значениями можно инициализировать TEMP, а какими нельзя? Ведь в хелпе этого не сказано... А в исходниках искать подобные вещи как-то не привлекает.