我在 svn.exe 周围编写了一个快速而肮脏的包装器来检索一些内容并对其执行某些操作,但对于某些输入,它偶尔会重复挂起并且无法完成。例如,一个调用是 svn list:
svn list "http://myserver:84/svn/Documents/Instruments/" --xml --no-auth-cache --username myuser --password mypassword
当我从命令 shell 执行此操作时,此命令行运行良好,但它挂在我的应用程序中。我运行它的 C# 代码是:
string cmd = "svn.exe";
string arguments = "list \"http://myserver:84/svn/Documents/Instruments/\" --xml --no-auth-cache --username myuser --password mypassword";
int ms = 5000;
ProcessStartInfo psi = new ProcessStartInfo(cmd);
psi.Arguments = arguments;
psi.RedirectStandardOutput = true;
psi.WindowStyle = ProcessWindowStyle.Normal;
psi.UseShellExecute = false;
Process proc = Process.Start(psi);
StreamReader output = new StreamReader(proc.StandardOutput.BaseStream, Encoding.UTF8);
proc.WaitForExit(ms);
if (proc.HasExited)
{
return output.ReadToEnd();
}
这需要整整 5000 毫秒并且永远不会完成。延长时间并没有什么帮助。在单独的命令提示符中,它会立即运行,因此我很确定这与等待时间不足无关。然而,对于其他输入,这似乎工作得很好。
我还尝试在这里运行一个单独的cmd.exe(其中exe是svn.exe,args是原始arg字符串),但挂起仍然发生:
string cmd = "cmd";
string arguments = "/S /C \"" + exe + " " + args + "\"";
我在这里可能会搞砸什么,以及如何调试这个外部进程的东西?
EDIT:
我现在正着手解决这个问题。 Mucho 感谢乔恩·斯基特(Jon Skeet)的建议,这确实非常有效。不过,我对处理这个问题的方法还有另一个问题,因为我是多线程新手。我想要关于改进任何明显的缺陷或其他愚蠢的事情的建议。我最终创建了一个小类,其中包含 stdout 流、一个用于保存输出的 StringBuilder 以及一个用于告知何时完成的标志。然后我使用 ThreadPool.QueueUserWorkItem 并传入我的类的一个实例:
ProcessBufferHandler bufferHandler = new ProcessBufferHandler(proc.StandardOutput.BaseStream,
Encoding.UTF8);
ThreadPool.QueueUserWorkItem(ProcessStream, bufferHandler);
proc.WaitForExit(ms);
if (proc.HasExited)
{
bufferHandler.Stop();
return bufferHandler.ReadToEnd();
}
... 和 ...
private class ProcessBufferHandler
{
public Stream stream;
public StringBuilder sb;
public Encoding encoding;
public State state;
public enum State
{
Running,
Stopped
}
public ProcessBufferHandler(Stream stream, Encoding encoding)
{
this.stream = stream;
this.sb = new StringBuilder();
this.encoding = encoding;
state = State.Running;
}
public void ProcessBuffer()
{
sb.Append(new StreamReader(stream, encoding).ReadToEnd());
}
public string ReadToEnd()
{
return sb.ToString();
}
public void Stop()
{
state = State.Stopped;
}
}
这似乎有效,但我怀疑这是否是最好的方法。这合理吗?我能做些什么来改善它?