在我的程序中不可能声明 INT 变量

Question

我尝试用 C++ 创建一个小项目。我的程序应该能够读取 Windows 中当前正在运行的进程，并每 5 分钟将有关进程的信息发送到我的私有 MySQL 数据库。在这段时间内，我可以阅读流程。

见下面的代码：

#include <stdio.h>
#include <iostream>
#include <windows.h>
#include <tlhelp32.h>

bool getAllProcesses(void);

int main(void){
    getAllProcesses();
}

bool getAllProcesses(){
    HANDLE WINAPI snapshot = CreateToolhelp32Snapshot( TH32CS_SNAPPROCESS, 0 );
    LPPROCESSENTRY32 pe32;

    Process32First(snapshot, pe32);
    while(Process32Next(snapshot, pe32)){
        std::cout << pe32->szExeFile << "\n";
    }
    std::cout << "End of list";
    CloseHandle( snapshot );
    return true;
}

上面的代码运行良好。

但如果我添加代码“int i;i=1;”像这样：

#include <stdio.h>
#include <iostream>
#include <windows.h>
#include <tlhelp32.h>

bool getAllProcesses(void);

int main(void){
    getAllProcesses();
}

bool getAllProcesses(){
    int i;
    i=0;
    HANDLE WINAPI snapshot = CreateToolhelp32Snapshot( TH32CS_SNAPPROCESS, 0 );
    LPPROCESSENTRY32 pe32;

    Process32First(snapshot, pe32);
    while(Process32Next(snapshot, pe32)){
        std::cout << pe32->szExeFile << "\n";
    }
    std::cout << "End of list";
    CloseHandle( snapshot );
    return true;
}

进行此更改后，程序将崩溃，并显示警告“程序停止工作”。

我试图找出问题所在，并确定了以下几点：

如果我使用函数 Process32First() OR Process32Next() 我不能在所有程序中声明相同的 int。

有什么问题？

Answer 1

这里有很多问题。我在 VS 2008 Win7 上试过这个。 “i”声明与问题无关，但可能只是稍微移动堆栈以隐藏真正的问题。在下面更正的代码中查看我的 cmets。

  bool getAllProcesses(){
        int i;
        i=0;
        HANDLE WINAPI snapshot = CreateToolhelp32Snapshot( TH32CS_SNAPPROCESS, 0 );
       /* replace LPPROCESSENTRY32 with PROCESSENTRY32. LP is just a define as
 a pointer to a structure, so you did not actually allocate any memory for the
 return of Process32First() to return the result into.
 So you were probably overwriting stuff in the stack. */

        PROCESSENTRY32 pe32;

       /* need to initialize the structure properly. Memset may be overkill,
         but better to see all zeros than garbage.  dwSize must be
         initialized as per the SDK documentation. */
        memset(&pe32,0,sizeof(pe32) );
        pe32.dwSize = sizeof(pe32);
        BOOL result;

        /* did not check the result from the call if TRUE/FALSE */
        result = Process32First(snapshot, &pe32);

        std::cout << "result ="  << result << "\n";
        while(Process32Next(snapshot, &pe32)){
            std::cout << pe32.szExeFile << "\n";
        }
        std::cout << "End of list";
        CloseHandle( snapshot );
        return true;
    }

Answer 2

添加这 2 行使程序无法运行只是巧合。程序根本不应该运行，因为有两个主要错误：

• pe32 是一个未初始化的指针

• PROCESSENTRY32 结构的

  dwSize 成员未按照[MS.Docs]: Process32First function 中所述进行初始化：

  调用应用程序必须将PROCESSENTRY32 的dwSize 成员设置为结构的大小（以字节为单位）。

第三个较小的错误是您忽略了 Process32First 返回的流程数据。

为了使事情正常工作，请替换以下代码行：

LPPROCESSENTRY32 pe32;

Process32First(snapshot, pe32);
while(Process32Next(snapshot, pe32)){
    std::cout << pe32->szExeFile << "\n";

，与：

PROCESSENTRY32 pe32;
pe32.dwSize = sizeof(PROCESSENTRY32);

if (Process32First(snapshot, &pe32))
{
    do {
        std::cout << pe32.szExeFile << "\n";
    }
    while (Process32Next(snapshot, &pe32));
}