17

我从 C# 启动一个进程，如下所示：

public bool Execute()
{
    ProcessStartInfo startInfo = new ProcessStartInfo();

    startInfo.Arguments =  "the command";
    startInfo.FileName = "C:\\MyApp.exe";

    startInfo.UseShellExecute = false;
    startInfo.RedirectStandardOutput = true;
    startInfo.RedirectStandardError = true;

    Log.LogMessage("{0} {1}", startInfo.FileName, startInfo.Arguments);

    using (Process myProcess = Process.Start(startInfo))
    {
        StringBuilder output = new StringBuilder();
        myProcess.OutputDataReceived += delegate(object sender, DataReceivedEventArgs e)
        {
            Log.LogMessage(Thread.CurrentThread.ManagedThreadId.ToString() + e.Data);
        };
        myProcess.ErrorDataReceived += delegate(object sender, DataReceivedEventArgs e)
        {
            Log.LogError(Thread.CurrentThread.ManagedThreadId.ToString() +  " " + e.Data);            
        };

        myProcess.BeginErrorReadLine();
        myProcess.BeginOutputReadLine();

        myProcess.WaitForExit();

    }

    return false;
}

但这有一个问题......如果有问题的应用程序按此顺序写入标准输出和标准错误：

std out: msg 1
std err: msg 2
std out: msg 3

然后我从日志中看到的输出是：

msg 2
msg 1
msg 3

这似乎是因为事件处理程序是在另一个线程中执行的。所以我的问题是如何保持写入 std err 和 std out 的过程顺序？

我想过使用时间戳，但由于线程的抢占性，我认为这不会起作用。

更新：确认在数据上使用时间戳是没有用的。

最终更新：接受的答案解决了这个问题 - 但是它确实有一个缺点，当流合并时，无法知道写入哪个流。因此，如果您需要写入 stderr == failure 而不是应用程序退出代码的逻辑，您可能仍然会被搞砸。

4

2 回答 2

14

据我了解，您希望保留 stdout/stderr 消息的顺序。我没有看到任何体面的方式来使用 C# 托管进程来做到这一点（反射 - 是的，讨厌的子类化黑客攻击 - 是的）。看起来它几乎是硬编码的。

此功能不依赖于线程本身。如果您想保持订单，STDOUT并且STDERROR必须使用相同的句柄（缓冲区）。如果他们使用相同的缓冲区，它将被同步。

这是 Process.cs 的一个片段：

 if (startInfo.RedirectStandardOutput) {
    CreatePipe(out standardOutputReadPipeHandle, 
               out startupInfo.hStdOutput, 
               false);
    } else {
    startupInfo.hStdOutput = new SafeFileHandle(
         NativeMethods.GetStdHandle(
                         NativeMethods.STD_OUTPUT_HANDLE), 
                         false);
}

if (startInfo.RedirectStandardError) {
    CreatePipe(out standardErrorReadPipeHandle, 
               out startupInfo.hStdError, 
               false);
    } else {
    startupInfo.hStdError = new SafeFileHandle(
         NativeMethods.GetStdHandle(
                         NativeMethods.STD_ERROR_HANDLE),
                         false);
}

如您所见，将有两个缓冲区，如果我们有两个缓冲区，我们已经丢失了订单信息。

基本上，您需要创建自己的 Process() 类来处理这种情况。伤心？是的。好消息是它并不难，看起来很简单。这是取自 StackOverflow 的代码，不是 C#，但足以理解算法：

function StartProcessWithRedirectedOutput(const ACommandLine: string; const AOutputFile: string;
  AShowWindow: boolean = True; AWaitForFinish: boolean = False): Integer;
var
  CommandLine: string;
  StartupInfo: TStartupInfo;
  ProcessInformation: TProcessInformation;
  StdOutFileHandle: THandle;
begin
  Result := 0;

  StdOutFileHandle := CreateFile(PChar(AOutputFile), GENERIC_WRITE, FILE_SHARE_READ, nil, CREATE_ALWAYS,
    FILE_ATTRIBUTE_NORMAL, 0);
  Win32Check(StdOutFileHandle <> INVALID_HANDLE_VALUE);
  try
    Win32Check(SetHandleInformation(StdOutFileHandle, HANDLE_FLAG_INHERIT, 1));
    FillChar(StartupInfo, SizeOf(TStartupInfo), 0);
    FillChar(ProcessInformation, SizeOf(TProcessInformation), 0);

    StartupInfo.cb := SizeOf(TStartupInfo);
    StartupInfo.dwFlags := StartupInfo.dwFlags or STARTF_USESTDHANDLES;
    StartupInfo.hStdInput := GetStdHandle(STD_INPUT_HANDLE);
    StartupInfo.hStdOutput := StdOutFileHandle;
    StartupInfo.hStdError := StdOutFileHandle;

    if not(AShowWindow) then
    begin
      StartupInfo.dwFlags := StartupInfo.dwFlags or STARTF_USESHOWWINDOW;
      StartupInfo.wShowWindow := SW_HIDE;
    end;

    CommandLine := ACommandLine;
    UniqueString(CommandLine);

    Win32Check(CreateProcess(nil, PChar(CommandLine), nil, nil, True,
      CREATE_NEW_PROCESS_GROUP + NORMAL_PRIORITY_CLASS, nil, nil, StartupInfo, ProcessInformation));

    try
      Result := ProcessInformation.dwProcessId;

      if AWaitForFinish then
        WaitForSingleObject(ProcessInformation.hProcess, INFINITE);

    finally
      CloseHandle(ProcessInformation.hProcess);
      CloseHandle(ProcessInformation.hThread);
    end;

  finally
    CloseHandle(StdOutFileHandle);
  end;
end;

资料来源：如何从 CreateProcess 执行的命令重定向大量输出？

您想使用 CreatePipe 而不是文件。从管道中，您可以像这样异步读取：

standardOutput = new StreamReader(new FileStream(
                       standardOutputReadPipeHandle, 
                       FileAccess.Read, 
                       4096, 
                       false),
                 enc, 
                 true, 
                 4096);

和 BeginReadOutput()

  if (output == null) {
        Stream s = standardOutput.BaseStream;
        output = new AsyncStreamReader(this, s, 
          new UserCallBack(this.OutputReadNotifyUser), 
             standardOutput.CurrentEncoding);
    }
    output.BeginReadLine();
于 2013-12-25T22:52:22.647 回答
9

虽然我很欣赏 Erti-Chris 的回答（那是什么，Pascal？），但我认为其他人可能更喜欢托管语言的答案。此外，对于那些说“你不应该这样做”的批评者，因为 STDOUT 和 STDERR 不能保证保持顺序：是的，我理解，但有时我们必须与程序（我们没有编写）进行互操作期望我们这样做，正确的语义该死。

这是 C# 中的一个版本。Process它不是通过调用来绕过托管API CreateProcess，而是使用另一种方法将 STDERR 重定向到Windows shell 中的 STDOUT 流。因为UseShellExecute = true实际上并不使用cmd.exeshell（惊喜！），所以您通常不能使用 shell 重定向。解决方法是自己启动cmd.exeshell，手动输入我们真正的 shell 程序和参数。

请注意，以下解决方案假定您的args数组已正确转义。我喜欢使用内核调用的蛮力解决方案GetShortPathName，但您应该知道它并不总是适合使用（例如，如果您不在 NTFS 上）。此外，您确实确实想要执行异步读取 STDOUT 缓冲区的额外步骤（如下所示），因为如果您不这样做，您的程序可能会死锁。

using System;
using System.Diagnostics;
using System.Text;
using System.Threading;

public static string runCommand(string cpath, string[] args)
{
    using (var p = new Process())
    {
        // notice that we're using the Windows shell here and the unix-y 2>&1
        p.StartInfo.FileName = @"c:\windows\system32\cmd.exe";
        p.StartInfo.Arguments = "/c \"" + cpath + " " + String.Join(" ", args) + "\" 2>&1";
        p.StartInfo.UseShellExecute = false;
        p.StartInfo.RedirectStandardOutput = true;
        p.StartInfo.RedirectStandardError = true;

        var output = new StringBuilder();

        using (var outputWaitHandle = new AutoResetEvent(false))
        {
            p.OutputDataReceived += (sender, e) =>
            {
                // attach event handler
                if (e.Data == null)
                {
                    outputWaitHandle.Set();
                }
                else
                {
                    output.AppendLine(e.Data);
                }
            };

            // start process
            p.Start();

            // begin async read
            p.BeginOutputReadLine();

            // wait for process to terminate
            p.WaitForExit();

            // wait on handle
            outputWaitHandle.WaitOne();

            // check exit code
            if (p.ExitCode == 0)
            {
                return output.ToString();
            }
            else
            {
                throw new Exception("Something bad happened");
            }
        }
    }
}
于 2016-08-10T19:51:58.463 回答