TThread: как узнать что поток уничтожен?
Метод Terminated закрытый   

с помощью Wait*** функции

Оберните Tthread своим классом - и получите доступ к полю

Мне надо что-то вроде этого:


Это сообщение отредактировал(а) Alca - 18.3.2009, 17:05

Ну и создайте свой класс:


И уже проверяйте у себя так:

Anikmar, скорее всего из этого ничего не выйдет. Terminate() и Terminated служат для безболезненного завершения потока. Т.е. Terminate() сама по себе ничего не делает, кроме как взводит флаг Terminated. А Terminated используется в главной функции (Execute):


ЗЫ покрайней мере, об этом писалось в книге...

Все это частности. Самый дельный ответ дал как всегда Lazin. 
Тем более у TThread есть свой метод WaitFor() - ожидающий прекращения потока.

Куда его втулить?  

Это сообщение отредактировал(а) Alca - 19.3.2009, 10:26

Alca, по-моему класс TThread напрямую не используют, надо делать свой класс-наследник от TThread.

в хелпе к 2009 есть пример   

с этим никто не спорит.



так делать нельзя.

Я так и сделал.

Добавлено через 34 секунды


Добавлено через 2 минуты и 9 секунд

Чего?

я не в курсе как там это сделано в TThread
вот обычный паттерн

• - запускаем поток
• - ...
• - говорим потоку что пора-бы и завершиться
• - ждем пока завершится

сначало вызов Terminate():
затем WaitFor();

блин смотрите хелп!

Добавлено через 8 минут и 51 секунду

как всегда несколько проще

А если поток уже отработал, то:

разумеется. я же писал - так делать нельзя. тут уже надо определиться - либо рубим поток через Terminate(), либо ждем пока он сам завершится через WaitFor() - вместе эти два метода не используются.

Использование FreeOnTerminate = true несовместимо с любыми попытками узнать у объекта статус завершенности потока. Так как при завершении потока объект само уничтожится, и звать после этого от него любые методы уже нельзя - объект разрушен.


Это сообщение отредактировал(а) xvr - 19.3.2009, 13:42

а если в OnTerminate или в деструкторе потока сигналить ивентом??

Это сообщение отредактировал(а) Alca - 19.3.2009, 14:06

Это можно, но надо не забыть потом закрыть HANDLE этого event'а

Типа для каждого потока надо ивенты создавать?

Я не понимаю, чем мой подход вам не понравился? Я использовал это сто раз - все нормально, указатель обнуляется.

Конкретно доя потоков не применял, но формы у меня все самоуничтожаются и обнуляют указатели. Для потоков - аналогично все должно работать.

Как то через задницу это выглядит - отпустить поток на волю (FreeOnTermiate), а потом пытаться поймать его обратно   Озвучите, что вы с ним хотите сделать, пока это не очень понятно

Добавлено @ 00:50

Хочу так?

или так 

Короче, если поток есть, то его убить, а если он еще пашет сдох, то пропустить удаление.

Это сообщение отредактировал(а) Alca - 20.3.2009, 13:10

Я же предложил, оберните его в свой класс - и получите доступ к protected свойству - дополнительно можете спокойно изменить конструктор так, чтобы запомнить адрес указателя, и соответственно деструктор - чтобы его обнулить. 

Короче сделал счетчик потоков - при запуске увеличиваю, а при OnTerminate уменьшаю.
В главной форме, при нажатии кнопки "Стоп", проверяю значение счетчика на нуль.
Если нуль, то пропускаю удаление потоков, если больше, то удаляю.   
Как вам такая идейка? Может чего-то криво?

Добавлено через 1 минуту и 24 секунды

Чета у меня там была какая-то лажа, надо будет еще раз проверить....

Лучше все-таки индивидуально знать состояние каждого потока. Счетчик может быть не нуль, но при удалении всех четырех кто-то модет еще жив, а кто-то нет.

Согласен

Добавлено через 12 минут и 5 секунд

В OnTerminate разве потоки не начинают разрушаться?

Вот с этого и надо было начинать   
Предлагаю поменять подход - не ловить за хвост недобитые потоки, а говорить тем, кто не помер, что уже пора   
В главной форме заводится event (CreateEvent) с ручным сбросом. При создании потока ему передают HANDLE этого event'а, от этого HANDLE поток делает DuplicateHandle и в дальнейшем ждет на нем (через WaitFor*Object). Когда дождался - закрывает свою копию HANDLE'а и завершается.
Главная форма, когда надо всех прибить, просто делает SetEvent & CloseHandle. Затем создает новый event для будущего использования

Alca,  я посмотрел ваш код - там слегка подправить пару строчек и никаких навротов особых нет
Что я изменил:

Заголовочный файл


CXThread **BackVar;
Здесь хранится указатель на поток

bool FlagWork;
Этот флаг показывает, что основная функция потока работает. Не особо нужная вещь - просто показывает как можно ввести собственную некую индикацию состояния потока - например индикатор прогресса.

__fastcall CXThread(CXThread **BackVar, bool CreateSuspended);
В конструктор добавил параметр - адрес переменной указателя потока

AnsiString StrOut;
Украшательское поле - для разнообразия вывода.

Теперь по коду.
Код потока выглядит так:


Я немножко модифицировал вывод - теперь поток засыпает на случайное значение, а с верофтность 0,5% при каждом цикле завершает самостоятельно свою работу.

Основная проверочная форма теперь выглядит так:


Я добавил еще одно поле МЕМО и кнопку для заполнения. По этой кнопке пробегаю по списку потоков и вывожу их состояние: Работает, завершил работу, уничтожен.

Вариант "Завершил работу" будет выдан при отключенном флаге самоликвидации - поток уже отработал, но не уничтожен. Если самоликвидация включена - то по завершению функции потока он самоуничтожается и указатель на себя обнуляет.

P.S.
Что-то с русским языком при копировании кода в форум глюки - как справится?

Прикрепи исходники, пж.

Легко



Присоединённый файл ( Кол-во скачиваний: 11 )
 ______.rar 4,46 Kb

Типа так?


Это сообщение отредактировал(а) Alca - 26.3.2009, 00:41

Не совсем.

•  На поданном Event'е надо ждать, а не взводить его
•  С поданного Event'а нужно сделать копию
•  Копию надо удалять в деструкторе