Club Builder

[Perico]

Hola,

estoy creando Threads dinamicamente, osea instancias de otro.

Como puedo saber cuando ha terminado la ejecución de un Thread si tengo puesto FreeOnTerminate();

Como puedo hacer por ejemplo que una vble se incremente justamente al terminar el thread?

Como puedo implementar algo parecido en caso de que no exista OnTerminate?


Gracias

[Crandel]

Que es exactamente lo que quieres hacer?

si incrementas una variable dentro de la clase cuando se esta por destruir la vas a perder Confused

Podrías incrementar una variable externa. Pero como sabrías que hilo se termino, solo sabrias cuantos fueron y no quien fue.

Puedes asignar el evento OnTerminate a un método de tu formulario y ahí capturas la finalización.

Abajo hay un link de mi pagina donde hablo de eventos, al final del artículo explica como asignar el evento a un método.

Suerte
_________________
Crandel

"hay 10 tipos de personas, las que ven en binario y las que no"

Angel Zeitoune - Artículo: Generación de eventos en C++Builder

[Perico]

Hola Crandel,

primero, gracias por responder.

El tema esta en que yo creo una clase
class MiHilo : public TThread

y despues voy creando instancias
nodo=new MiHilo(this);
nodo->FreeOnTerminate=true;
nodo->Resume();

pero lo que sucede es que dentro del hilo creado se va a ejecutar la función ping de IcmpClient, y como sabes el ping tiene un tiempo de respuesta, entonces el hilo ejecuta ->Ping();
Pero como quiero hacer ping cada 5 segundos pues lo tengo dentro de un Timer, dentro del OnTimer.

pero no se si el hilo a terminado cuando le toca hacer otra vez el ping

y si pongo

nodo->Terminate(); // o nodo->Free();
nodo=new MiHilo(this);
nodo->FreeOnTerminate=true;
nodo->Resume();

tampoco va, y eso que lo elimino antes.

Gracias

[Crandel]

[Perico]

Hola de nuevo,

si lo dejo como al principio osea como tu me dices, funciona bien, pero con el task manager de windows veo como la memoria usada por mi programa va aumentando (no mucho unos 8 kb por thread) y es como si no llegase a liberar la memoria usada para los threads.
He probado a crear un thread con un botón (pero sin ejecutar) y otro botón para hacer un delete y funciona bien, al pulsar el 1º botón, me crea el thread y la memoria usada aumenta 8k y al pulsar el 2º botón me libera los 8k de la memoria.
Pero he descubierto con esto que si pulso el 1º botón varias veces, me va quitando 8k de la memoria cada vez que lo pulso, osea si lo pulso 10 veces me quita 80 kb. Y al pulsar el 2º ´botón solo libera 8k (si vuelvo a pulsar el 2º botón me da error).

[Cesar C. Salazar Gonzalez]

Bien, asi como esta hablando, es un poco dificil, entender, el procedimiento de tu Hilo.

intenta generar el hilo de la siguiente manera

creamos el hilo

[Crandel]

Con el permiso de César,

creo que tu problema esta en que tus hilos no se estan liberando, dado que no concluyen el método execute.

la poner el método FreeOnTerminate a true se deberían liberar la memoria automáticamente sin necesidad de hacer ningún Free ni nada.

Prueba eliminando tu código del método execute y veras como vuelve a disminuir la memoria.

el porque si te la libera con Free, es porque fuerzas a destruir el objeto.

Por otro lado, no puedes hacer dos veces click sobre el botón de destruir porque solo almacenas el puntero de acceso al último hilo creado y una vez destruido, si lo intentas destruir vas a obtener un error.

por último, tu problema no tiene nada que ver con el CreateSuspended, aunque si sería importante que lo fijaras.

el código nodo->Terminate(); es para terminar tu método execute, vos no lo necesitas, dado que estableces el FreeOnTerminate a true para que se destruya automaticamente al finalizar el método execute.
_________________
Crandel

"hay 10 tipos de personas, las que ven en binario y las que no"

Angel Zeitoune - Artículo: Generación de eventos en C++Builder

[Invitado]

Muchas gracias Crandel y César,

He de pediros perdón a los dos ya que en realidad los threads si se liberan con freeOnTerminate=true, el problema estaba en que tengo un std::vector que va guardando los valores de respuesta de los ping y eso es lo que va creciendo con tiempos.resize(tiempos.size())+1;

siento mucho haberles hecho pensar en algo que estaba bien y yo lo volví un problema.

Un saludo