Problema listando proceso en win32

[Uknow]

Hola, buenas, saludines. : )

Tengo un problema que me esta tirando los pelos d la cabeza, con este codigo que no logro que funcione como debe.

Estoy intentando verificar si un proceso esta corriendo desde interfaz grafica, pero por alguna razon no me funciona, el codigo esta asi:

Código (cpp) [Seleccionar] Expandir

#include <windows.h>
#include <Tlhelp32.h>

/*  Declare Windows procedure  */
LRESULT CALLBACK WindowProcedure (HWND, UINT, WPARAM, LPARAM);

/*  Make the class name into a global variable  */
char szClassName[ ] = "WindowsApp";
int Buscar_Proceso();
void Crear_Ventana(HWND hwnd);
HINSTANCE instancia;
HWND Button2;

   
    int WINAPI WinMain (HINSTANCE hThisInstance,
                    HINSTANCE hPrevInstance,
                    LPSTR lpszArgument,
                    int nFunsterStil)

{
    HWND hwnd;               /* This is the handle for our window */
    MSG messages;            /* Here messages to the application are saved */
    WNDCLASSEX wincl;        /* Data structure for the windowclass */
    instancia = hThisInstance;   
    /* The Window structure */
    wincl.hInstance = hThisInstance;
    wincl.lpszClassName = szClassName;
    wincl.lpfnWndProc = WindowProcedure;      /* This function is called by windows */
   // wincl.style = CS_DBLCLKS;                 /* Catch double-clicks */
    wincl.cbSize = sizeof (WNDCLASSEX);

    /* Use default icon and mouse-pointer */
    wincl.hIcon = LoadIcon (NULL, IDI_APPLICATION);
    wincl.hIconSm = LoadIcon (NULL, IDI_APPLICATION);
    wincl.hCursor = LoadCursor (NULL, IDC_ARROW);
   // wincl.lpszMenuName = NULL;                 /* No menu */
    wincl.cbClsExtra = 0;                      /* No extra bytes after the window class */
    wincl.cbWndExtra = 0;                      /* structure or the window instance */
    /* Use Windows's default color as the background of the window */
    wincl.hbrBackground = (HBRUSH) COLOR_BACKGROUND;

    /* Register the window class, and if it fails quit the program */
    if (!RegisterClassEx (&wincl))
        return 0;

    /* The class is registered, let's create the program*/
    hwnd = CreateWindowEx (
           0,                   /* Extended possibilites for variation */
           szClassName,         /* Classname */
           "Windows App",       /* Title Text */
           WS_OVERLAPPEDWINDOW, /* default window */
           CW_USEDEFAULT,       /* Windows decides the position */
           CW_USEDEFAULT,       /* where the window ends up on the screen */
           544,                 /* The programs width */
           375,                 /* and height in pixels */
           HWND_DESKTOP,        /* The window is a child-window to desktop */
           NULL,                /* No menu */
           hThisInstance,       /* Program Instance handler */
           NULL                 /* No Window Creation data */
           );

    /* Make the window visible on the screen */
   
    ShowWindow (hwnd, nFunsterStil);

    /* Run the message loop. It will run until GetMessage() returns 0 */
    while ( GetMessage (&messages, NULL, 0, 0) == 1)
    {
        /* Translate virtual-key messages into character messages */
       TranslateMessage(&messages);
        /* Send message to WindowProcedure */
        DispatchMessage(&messages);
    }

    /* The program return-value is 0 - The value that PostQuitMessage() gave */
    return messages.wParam;
}


/*  This function is called by the Windows function DispatchMessage()  */

LRESULT CALLBACK WindowProcedure (HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    switch (message)                  /* handle the messages */
    {
       
          case WM_CREATE:
           Crear_Ventana(hwnd);
           break;
         
            case WM_COMMAND:
                 
            if((HWND)lParam == Button2 )
            {
                  if(Buscar_Proceso() == 0)
                   MessageBox(0, "El proceso esta corriendo", "Proceso", 0);       
                       
            }
            break;

                     
        case WM_DESTROY:
            PostQuitMessage (0);       /* send a WM_QUIT to the message queue */
           
         
             break;
             
        default:                      /* for messages that we don't deal with */
            return DefWindowProc (hwnd, message, wParam, lParam);
    }

    return 0;
}

void Crear_Ventana(HWND hwnd)
{
     //Boton 2
Button2 = CreateWindowEx(0,"BUTTON", "Buscar P", WS_CHILD|WS_VISIBLE|WS_TABSTOP,
                         100, 300, 100, 20, hwnd, 0, instancia, NULL);
}

int Buscar_Proceso()
{
    HANDLE CProc;   
    PROCESSENTRY32 Proceso;    //Necesario para realizar la captura de procesos
    CProc=CreateToolhelp32Snapshot(TH32CS_SNAPPROCESS, 0);   //Con esto creamos una captura de la lista de procesos y la guardamos en CProc
Process32First(CProc,&Proceso);    //Leemos el primer proceso

while(Process32Next(CProc,&Proceso))   //Mientras haya procesos para leer leemos el siguiente
{
                                           
if(!strcmp(Proceso.szExeFile,"explorer.exe") )
return 0;
}
   
    return 1;
}


Sin duda ha de ser una chorrada pero no lo veo
help! D:

Lo que mas rabia me da es que si lo hago desde la consola si sale!!

Código (cpp) [Seleccionar] Expandir


#include <windows.h>
#include <Tlhelp32.h>
#include <stdio.h>

HANDLE CProc;   //Para crear la captura de la lista de procesos
int Buscar_Proceso();

int main()
{
    if(Buscar_Proceso() == 0)
      MessageBox(0, "el proceso esta corriendo", "proceso", 0);

     getchar();

return 0;    //Retornamos 0 diciendo que salio bien
}

int Buscar_Proceso()
{
    PROCESSENTRY32 Proceso;    //Necesario para realizar la captura de procesos
    CProc=CreateToolhelp32Snapshot(TH32CS_SNAPPROCESS, 0);   //Con esto creamos una captura de la lista de procesos y la guardamos en CProc
Process32First(CProc,&Proceso);    //Leemos el primer proceso

while(Process32Next(CProc,&Proceso))   //Mientras haya procesos para leer leemos el siguiente
{
                                           
if(!strcmp(Proceso.szExeFile,"explorer.exe") )
return 0;
}
   
    return 1;
}
   
   
Alguien puede ofrecerme una manito con esto?

Gracias de antemano!!

[naderST]

Bueno a simple vista no veo nada extraño pero no puedo probar porque ahora no estoy en windows pero verifica que se cumpla esta condicion:

Código (c) [Seleccionar] Expandir


(HWND)lParam == Button2


Ojo te lo digo asumiendo de que se cree bien la ventana y el botón...

EDIT:

Bueno ya lo corrí en Windows y el problema es el siguiente en tu función Buscar_proceso():

dwSize
The size of the structure, in bytes. Before calling the Process32First function, set this member to sizeof(PROCESSENTRY32). If you do not initialize dwSize, Process32First fails.

Es decir:

Código (cpp) [Seleccionar] Expandir

Proceso.dwSize = sizeof(PROCESSENTRY32);

[Uknow]

Bueno a simple vista no veo nada extraño pero no puedo probar porque ahora no estoy en windows pero verifica que se cumpla esta condicion:

Código (c) [Seleccionar] Expandir


(HWND)lParam == Button2


Ojo te lo digo asumiendo de que se cree bien la ventana y el botón...

EDIT:

Bueno ya lo corrí en Windows y el problema es el siguiente en tu función Buscar_proceso():

dwSize
The size of the structure, in bytes. Before calling the Process32First function, set this member to sizeof(PROCESSENTRY32). If you do not initialize dwSize, Process32First fails.

Es decir:

Código (cpp) [Seleccionar] Expandir

Proceso.dwSize = sizeof(PROCESSENTRY32);

::-O
Muchisimas gracias!!!!

Ya veo . . . pero me causa curiosidad porque cuando uso el console subsystem no tengo declarar el tamaño de la estructura ???
Que raro. .

Pero bueno, gracias. (:

saludos!

[naderST]

De nada, la verdad es muy raro pero este tipo de cosas suelen pasar en C/C++, de todas maneras siempre que estés usando el API de Windows asegúrate de documentarte muy bien en la MSDN.