重定向标准错误的C#问题

时间:2009-11-15 22:24:08

标签: c# multithreading

我的应用程序中的主窗体启动一个新线程,然后打开另一个窗体,用作进度窗口。 该线程管理两个控制台应用程序之间的一些数据,并将StandardError中的信息发送到进度窗口。我使用DataReceivedEventHandler异步读取标准错误。

如果我让所有事情都顺利进行,那么它的效果非常好,但是当用户按下我的表单上的“取消”按钮时会出现问题。会发生什么,即使在我停止进程后,ErrorDataReceived函数仍会继续触发!有时取消会成功,但有时我会遇到死锁情况(我认为这是正确的词)。

以下是我的代码的一些片段,以便您可以看到发生了什么。它等待“p2.WaitForExit();”和“Invoke(new updateProgressDelegate(this.updateProgress),e.Data);” (visual studio按照这些线条放置一个绿色箭头,表示它们将成为下一个执行的箭头)

// start 2 processes (p & p2) and pipe data from one to the other
// this runs in thread t
                    p.Start();
                    p2.Start();
                    byte[] buf = new byte[BUFSIZE];
                    int read = 0;
                    p2.ErrorDataReceived += new DataReceivedEventHandler(p2_ErrorDataReceived);
                    p2.BeginErrorReadLine();

                    try
                    {
                        read = p.StandardOutput.BaseStream.Read(buf, 0, BUFSIZE);
                        while (read > 0 && read <= BUFSIZE)
                        {
                            if (canceled==false)
                                p2.StandardInput.BaseStream.Write(buf, 0, read);
                            if (canceled==false)
                                read = p.StandardOutput.BaseStream.Read(buf, 0, BUFSIZE);
                            else
                            {
                                return;
                            }
                        }
                    }


// this function is called when a user presses the "cancel" button on a form.
private void cancel_encode()
        {

            if (p2 != null)
            {
                if (p2.HasExited == false)
                    p2.Kill();
                if (p2.HasExited == false)
                    p2.WaitForExit();
            }
            if (p != null)
            {
                if (p.HasExited == false)
                    p.Kill();
                if (p.HasExited == false)
                    p.WaitForExit();
            }
            if (t.IsAlive)
            {
                if (!t.Join(2000))
                    t.Abort();
            }
        }


// this function sends the error data to my progress window
void p2_ErrorDataReceived(object sender, DataReceivedEventArgs e)
        {
            if (e.Data != null && canceled == false)
                Invoke(new updateProgressDelegate(this.updateProgress), e.Data);
        }

1 个答案:

答案 0 :(得分:1)

两件事

在“p2_ErrorDataReceived”中,调用Invoke()。这可能会导致部分僵局。您可能希望将其更改为BeginInvoke()。

Process.WaitForExit()的文档建议在调用Kill()之后,应该调用带有整数的WaitForExit重载,如果返回true,则在没有参数的情况下再次调用它以确保所有异步处理都有完成:

 p2.Kill();
 if(p2.WaitForExit(waitTime)) //waitTime is the number of milliseconds to wait
 {p2.WaitForExit();}  //waits indefinitely

 //likewise for p

http://msdn.microsoft.com/en-us/library/ty0d8k56.aspx