8

我正在创建一个子进程,并读取它的输出。当子进程创建输出 () 时,我的代码工作正常cmd /c echo Hello World,但是如果进程不创建输出 (),ReadFile 将挂起cmd /c echo Hello World > output.txt。我只在进程终止后阅读。

我做错了什么可怕的事情吗?无论如何用同步模式来做到这一点,还是我必须使用异步模式?所有这些都发生在一个单独的线程中,所以我认为异步模式不会给我带来任何好处,除非它是让它工作的唯一方法。非常感谢!

saAttr.nLength = sizeof(SECURITY_ATTRIBUTES); 
saAttr.bInheritHandle = TRUE; 
saAttr.lpSecurityDescriptor = NULL; 

CreatePipe(&g_hChildStd_OUT_Rd, &g_hChildStd_OUT_Wr, &saAttr, 0);
SetHandleInformation(g_hChildStd_OUT_Rd, HANDLE_FLAG_INHERIT, 0);

memset(&piProcInfo, 0, sizeof(PROCESS_INFORMATION));
memset(&siStartInfo, 0, sizeof(STARTUPINFO));
siStartInfo.cb = sizeof(STARTUPINFO); 
siStartInfo.hStdError = g_hChildStd_OUT_Wr;
siStartInfo.hStdOutput = g_hChildStd_OUT_Wr;
siStartInfo.dwFlags |= STARTF_USESTDHANDLES;
CreateProcess(NULL, commandWideString, NULL, NULL, TRUE, 0, NULL, NULL, &siStartInfo, &piProcInfo);

while(1)
{
    GetExitCodeProcess(piProcInfo.hProcess, &processExitCode);
    if(processExitCode != STILL_ACTIVE)
        break;
    else
        Sleep(1);
}

*output = (char *)calloc(32, sizeof(char));
processOutputSize = 0;
while(1)
{
    bSuccess = ReadFile( g_hChildStd_OUT_Rd, processOutputTemp, 32, &dwRead, NULL);
    if(!bSuccess || !dwRead)
        break;
    memcpy(*output + processOutputSize, processOutputTemp, dwRead);
    processOutputSize += dwRead;
    if(dwRead == 32)
        *output = (char *)realloc(*output, processOutputSize + 32);
    else
    {
        memset(*output + processOutputSize, 0, 1);
        break;
    }
}
CloseHandle(piProcInfo.hProcess);
CloseHandle(piProcInfo.hThread);
CloseHandle(g_hChildStd_OUT_Rd);
CloseHandle(g_hChildStd_OUT_Wr);
4

3 回答 3

17

正如@Marcus 在评论中建议的那样,您应该在读取之前关闭输出管道的写入端。

CloseHandle(g_hChildStd_OUT_Wr);

对我来说,这是真正的答案。

于 2016-08-03T09:11:59.797 回答
6

您将进程的输出重定向到管道,启动进程,等到它退出,然后读取输出。

问题是 Windows 只缓冲有限数量的数据。因此,您必须在进程仍在运行时读取管道,否则进程将被阻塞,因为它无法向管道写入更多数据。

于 2012-12-15T18:35:02.680 回答
3

您可以在这样的循环中使用PeekNamedPipe

for (;;)
{
    DWORD bytesAvail = 0;
    if (!PeekNamedPipe(stdoutPipeRead, NULL, 0, NULL, &bytesAvail, NULL)) {
        std::cout << "Failed to call PeekNamedPipe" << std::endl;
    }
    if (bytesAvail) {
        CHAR buf[BUFSIZE];
        DWORD n;
        BOOL success = ReadFile(stdoutPipeRead, buf, BUFSIZE, &n, NULL);
        if (!success || n == 0) {
            std::cout << "Failed to call ReadFile" << std::endl;
        }
        std::cout << std::string(buf, buf + n);
    }
}
于 2015-11-03T08:29:25.650 回答