Problemas con creación de hilo

[aguml]

A ver si me puedes ayudar, el caso es que solo hay dos hilos, el principal y el del debuger. El principal crea al hilo del debuger y posteriormente llama a métodos del hilo del debuger los cuales necesitan acceder a unas estructuras y variables que se crean al inicio del execute del hilo con lo que se da a veces el caso de que el hilo principal va más rápido que el secundario y accede a las variables sin haberse creado antes en el hilo secundario. He estado buscando la manera de solucionar eso y no me sirve waitfor porque espera a que acabe el hilo y en este caso el secundario contiene el bucle de eventos del debuger con lo que no acabará. La solución que encontré es usar TEvent y usarlo en la zona de código que quiero que se ejecute en el hilo secundario para que asigne la memoria a las variables y que una vez hecho eso ya continúe el hilo principal pero solo he encontrado esto en español: https://elvex.ugr.es/decsai/builder/...rdination.html
Y no tengo nada claro como hacerlo.

[[escafandra]]

Pero quizás tengas que preguntarte primero si en realidad te hacen falta dos hilos.


En caso afirmativo, una solución rápida es que las variables se creen en la clase TDebugger y se asignen en execute. Las variables que necesita leer el hilo principal siempre deben leerse con un método de TDebuger y a ser posible no deben ser asignadas mientras TDebugger se ejecuta. Puedes saber si la variable leída es válida dándole determinado valor en caso de no ser válido. Por ejemplo, puede ser un puntero inicializado a NULL en cuyo caso no trataras de leerlo, pero puedes hacer un bucle (con un sleep corto) hasta tener un valor correcto. También puedes controlarlas con otra variable que se pone a true cuando el valor es válido...

WaitForSingleObject puede ser usado con otros tipos de Handle que no tienen que ser el Thread, puedes usar un Mutex o un Event. Estos también te pueden servir para poner el hilo a la espera. Todo depende de tu diseño particular.


Saludos.

[aguml]

creo que TEvent es lo mejor pero no se como se usa. Tengo esto.
En el archivo TDebuger.h:

Código PHP:

TEvent *EventoVariablesArmadas; 


En el TDebuger.cpp:

Código PHP:

bool TDebugger::InitDebug(AnsiString Path, int mode, AnsiString WName, DWORD PID)
{
        ...
        ... 
    EventoVariablesArmadas = new TEvent(false);
    return retval;
} 


Código PHP:

void TDebugger::LoopDebug(void)
{
        HiloDepuracion = new(std::nothrow) THiloDebugger(false,Mode,PathFile,NameWindow,pid,this); // no me gusta no hay control de PathFile en este método.
        EventoVariablesArmadas->WaitFor(INFINITE);
        if(HiloDepuracion == NULL)
        {
        ...
                ...
} 


En el THiloDebuger.cpp:

Código PHP:

void __fastcall THiloDebugger::Execute()
{
    DEBUG_EVENT DebugEv;
        HANDLE CurrentExceptionThread;
        DWORD dwContinueStatus;
        int Error;
        TDll *ItemDLL;
        bool salir=false;

        ListaBPs = new TList;
        ListaDlls = new TList;
        ListaThreads = new TList;

        //Synchronize(OnInitLoopDebugThread); //Usaremos este evento para inizialización de variables en la clase usuario si se requiere.

        si.cb = sizeof(STARTUPINFO);
        GetStartupInfo(&si);

        // El proceso se tiene que crear en la misma función donde está el hilo del depurador sino no vale
        switch(Mode){
                case MODE_NORMAL:
                        CreateProcess(PathFile.c_str(),NULL, NULL, NULL, FALSE, DEBUG_PROCESS | DEBUG_ONLY_THIS_PROCESS, NULL, NULL, &si, &pi);
                        break;
                case MODE_ATACH_BY_WINDOW_NAME:
                        if(!AtachProcessByNameWindow(WindowName.c_str(),Error)){
                                return;
                        }else{
                                //Aqui Obtengo informacion del proceso
                                EnumerarModulos( pi.dwProcessId );
                                SystemEntryPoint=0;
                        }
                        break;
                case MODE_ATACH_BY_PID:
                        if(!AtachProcessById(pid,Error)){
                                return;
                        }else{
                                //Aqui Obtengo informacion del proceso
                                EnumerarModulos( pi.dwProcessId );
                                SystemEntryPoint=0;
                        }
                        break;
                default:
                        return;
        }

        InfoProcess.hProcess = pi.hProcess;
        InfoProcess.ProcessID = pi.dwProcessId;

        EventoVariablesArmadas->SetEvent();
               ...
               ...
        if(EventoVariablesArmadas != NULL)
            delete EventoVariablesArmadas;
} 


No se porque al llamar al constructor del TEvento me da NULL pero si en vez de declararlo en TDebugger.h lo hag en TDebugger.cpp ahi si me asigna bien pero claro, no puedo acceder desde THiloDebugger.

[[escafandra]]

SetEvent lo señaliza de forma que WaitForSingleObject(hEvent, INFINITE) deja de esperar. por ejemplo.
Si usas TEvent, WaitFor haría lo mismo.


Es una forma de hacer esperar hasta que algo señaliza el evento.


Saludos.

[aguml]

Si, eso ya lo vi pero el caso es que al ejecutar el constructor me da NULL si lo declaró en TDebugger.h y me da un valor diferente de NULL si lo hago en TDebugger.cpp. No entiendo porque pasa eso sí Tdebugger.h se incluye en TDebugger.cpp.

[aguml]

Me he dado cuenta que si la declaro como:

Código PHP:

static TEvent *EventoVariablesArmadas; 


en el archivo de cabecera ya si me crea el TEvent o eso parece porque no retorna NULL en el constructor pero al intentar operar desde el hilo sobre él alli si vale NULL. No entiendo que está pasando

[[escafandra]]

Es buena idea declararla static de la clase pero entonces no la inicializa el constructor de la misma. Deber incluir en la parte de código esa inicialización. Lee sobre los miembros estáticos de un a clase en CPP. Son miembros que no pertenecen al objeto, sino a la clase misma por lo que existen siempre aún sin instanciar ningún objeto. Se puede acceder a ellos a través del nombre de la clase o desde un objeto.

En debugger.h

Código PHP:

TEdbugger::class...
.....
public:
static TEvent  *Event;
..... 


En Debugger.cpp

Código PHP:

TEdbugger::TEdbugger.....
....Funciones.....

//Fuera de cualquier función:
TEvent*  TDebugger::Event = new TEvent(false);

...... 


Ten en cuenta que al ser un miembro de la clase, será el mismo para todos los objetos Debugger, estará compartido y accesible por todos (no está encapsulado).

Saludos.