问题描述
我用C#做了个界面，包括一个text_box和一个rich_textbox，还有一个按钮建立两个线程：一个线程启动cmd执行text_box中的命令，如：ping1.1.1.1第二个线程提取cmd的输出流，实时显示到rich_textbox中点击按钮开始运行但是ping1.1.1.1后出现的信息是一行一行出现的，全部显示完之前界面无法响应别的操作。比如我还有个标签页切换控件（功能和这个例子无关），此时就无法响应。请大家帮忙看一下是什么原因这是全部的类函数：下面的是具体代码：publicpartialclassNS_FEM_Form:Form{publicdelegatevoidUpdateTxt(StreamReaderreader);publicUpdateTxtupdateTxt;StreamReaderreader;//////////////////////////////////////////////////////////////////////////////////////publicNS_FEM_Form(){InitializeComponent();}privatevoidbutton_run_Click(objectsender,EventArgse){ThreadobjThread1=newThread(newThreadStart(delegate{Thread_NS_FEM();}));objThread1.Start();ThreadobjThread2=newThread(Thread_display);objThread2.Start();}//这个函数不相关，请忽略privatevoidbutton_update_Click(objectsender,EventArgse){}privatevoidThread_NS_FEM(){Processproc=newProcess();proc.StartInfo.FileName="cmd.exe";proc.StartInfo.CreateNoWindow=true;proc.StartInfo.UseShellExecute=false;proc.StartInfo.RedirectStandardOutput=true;proc.StartInfo.RedirectStandardError=true;proc.StartInfo.RedirectStandardInput=true;proc.Start();proc.StandardInput.WriteLine(this.textBox_work_dir.Text);proc.StandardInput.WriteLine("exit");reader=proc.StandardOutput;}privatevoidThread_display(){Thread.Sleep(1000);this.BeginInvoke(updateTxt,reader);}privatevoidNS_FEM_Form_Load(objectsender,EventArgse){updateTxt=newUpdateTxt(update_textbox_method);}privatevoidupdate_textbox_method(StreamReaderreader){stringline=reader.ReadLine();while(!reader.EndOfStream){this.richTextBox_log.AppendText(line+"r");this.richTextBox_log.SelectionStart=this.richTextBox_log.Text.Length;this.richTextBox_log.ScrollToCaret();line=reader.ReadLine();}}}
解决方案
解决方案二：
补充一下：this.textBox_work_dir.Text就是输入ping1.1.1.1的text_boxthis.richTextBox_log就是用来显示的rich_textbox
解决方案三：
你的Thread_display应该删掉，应该使用System.Threading.Timer定时器。
解决方案四：
类似地，对于reader的读取，应该采取异步子线程方式，只要有信息就触发回调行为。而不是搞什么阻塞整个窗体的死循环操作。
解决方案五：
“死循环、阻塞（包括Sleep语句）”这都是编程大忌。有些人动不动就以为“我搞了一个死循环监控”挺时髦的。实际上如果你有一定的产品开发经验，就应该知道那种程序有多么地弱智。真正的监控操作是事件驱动的，而不是“死循环+阻塞”的。
解决方案六：
引用4楼sp1234的回复:
“死循环、阻塞（包括Sleep语句）”这都是编程大忌。有些人动不动就以为“我搞了一个死循环监控”挺时髦的。实际上如果你有一定的产品开发经验，就应该知道那种程序有多么地弱智。真正的监控操作是事件驱动的，而不是“死循环+阻塞”的。
我也觉得那个死循环是有问题的，但是我这个例子是在别人基础上改的，自己是新手还不太会改进。关于你说的信息触发回调的方式，能不能帮我推荐个小例子？或者写个伪代码也行，实在麻烦给我提供几个关键词我去查也好。谢谢
解决方案七：
引用5楼tan462song7的回复:
Quote: 引用4楼sp1234的回复:
“死循环、阻塞（包括Sleep语句）”这都是编程大忌。有些人动不动就以为“我搞了一个死循环监控”挺时髦的。实际上如果你有一定的产品开发经验，就应该知道那种程序有多么地弱智。真正的监控操作是事件驱动的，而不是“死循环+阻塞”的。我也觉得那个死循环是有问题的，但是我这个例子是在别人基础上改的，自己是新手还不太会改进。关于你说的信息触发回调的方式，能不能帮我推荐个小例子？或者写个伪代码也行，实在麻烦给我提供几个关键词我去查也好。谢谢
用sleep是因为我发现没有延时会出错，我猜测可能因为Thread_NS_FEM线程没执行到低时，reader变量还是null，再提取就出错，所以加了个延时。
解决方案八：
update_textbox_method方法里有个死循环，而这个方法是用this.BeginInvoke来调用的，也就是在主线程下调用的，所以你的主线程就死了。
解决方案九：
对于多线程操作，其实就是在非UI线程上进行比较耗时的操作，在需要更新UI时调用Invoke方法，通知UI线程进行更新。当然，Invoke方法尽量快速结束，也就是你的update_textbox_method方法尽量消耗少的时间，这样界面会比较流畅，不会出现阻塞现象。所以，在此需要修改update_textbox_method方法，将此方法放入一个线程中，privatevoidupdate_textbox_method(StreamReaderreader){stringline=reader.ReadLine();while(!reader.EndOfStream){//收到信息，再通知UI线程//this.BeginInvoke...//{//this.richTextBox_log.AppendText(line+"r");//this.richTextBox_log.SelectionStart=this.richTextBox_log.Text.Length;//this.richTextBox_log.ScrollToCaret();//}line=reader.ReadLine();}}
解决方案十：
UI编程时候，在子线程中尽量不要直接操作UI的控件，会阻塞UI的主线程，可以在线程中使用委托方式来响应UI控件。