使用 ThreadStart 我的 UI 卡住了

Question

我是线程新手 我的用户界面在流程完成之前没有响应。谁能告诉我这里出了什么问题。

精简代码：

  static string replacedname = "", replacedwith;
            private void startProcess(string FPath, string RNo)
            {
                if (FPath != "" && RNo != "")
                {
                    ....

                    UnZip(FPath, Path.GetDirectoryName(FPath) + "\\" + replacedwith);

                    DirectoryInfo dir = new DirectoryInfo(Path.GetDirectoryName(FPath) + "\\" + replacedwith);
                    foreach (FileInfo File1 in dir.GetFiles("*.zip"))
                    {
                        UnZip(File1.FullName, Path.GetDirectoryName(File1.FullName));
                        File.Delete(File1.FullName);
                    }

                }

                replacedname = "";
                MessageBox.Show("Completed");

            }
            private void button1_Click(object sender, EventArgs e)
            {

                ThreadStart threadStart = delegate() { startProcess(textBox1.Text, textBox2.Text); };

                threadStart.BeginInvoke(null, null);


            }
            private void UnZip(string zipToExtract, string unzipDirectoryl) // Extract zip/lot file in same directory
            {

                if (this.progressBar1.InvokeRequired)
                {
                    progressBar1.Invoke(new Action(delegate() { UnZip(zipToExtract, unzipDirectoryl); }));

                }
                else if (this.label3.InvokeRequired)
                {
                    label3.Invoke(new Action(delegate() { UnZip(zipToExtract, unzipDirectoryl); }));
                }
                else
                {                        ....
                            progressBar1.Value += 1;
                            progressBar1.Refresh();
                            label3.Text = ((progressBar1.Value * 100) / (progressBar1.Maximum)).ToString();
                            label3.Refresh();
                        ...
                }
            }


            private void status(string msg, Color selcColor)
            {
                if (this.richTextBox1.InvokeRequired)
                {
                    richTextBox1.Invoke(new Action(delegate() { status(msg, selcColor); }));
                }
                else
                {
                    richTextBox1.SelectionStart = richTextBox1.Text.Length;
                    var oldcolor = richTextBox1.SelectionColor;

                    richTextBox1.SelectionColor = selcColor;
                    richTextBox1.AppendText(msg + "\n");
                    richTextBox1.SelectionColor = oldcolor;
                    richTextBox1.Refresh();
                    richTextBox1.ScrollToCaret();
                }
            }

Answer 1

替换你的代码

 private void button1_Click(object sender, EventArgs e)
        {
            ThreadStart threadStart = delegate() { startProcess(textBox1.Text, textBox2.Text); };
            threadStart.BeginInvoke(null, null);
        }

与

 private void button1_Click(object sender, EventArgs e)
        {
            ThreadStart threadStart = delegate() { startProcess(textBox1.Text, textBox2.Text); };
            Thread thread = new Thread(threadStart);
            thread.Start();
        }

这将使用您分配的委托启动一个新线程。
有关 MSDN 参考，请参阅 here。

编辑

在您的UnZip-方法中，您正在做一些奇怪的事情

private void UnZip(string zipToExtract, string unzipDirectoryl){

            if (this.progressBar1.InvokeRequired)
            {
                progressBar1.Invoke(new Action(delegate() { UnZip(zipToExtract, unzipDirectoryl); }));

            }
            else if (this.label3.InvokeRequired) // why else if on another control???
            {
                label3.Invoke(new Action(delegate() { UnZip(zipToExtract, unzipDirectoryl); }));
            }
            else
            {                        ....
                        progressBar1.Value += 1;
                        progressBar1.Refresh(); //why use Refresh()?
                        label3.Text = ((progressBar1.Value * 100) / (progressBar1.Maximum)).ToString();
                        label3.Refresh();
                    ...
            }
        }

您不能调用progressbar1.InvokeRequired，如果它返回错误，请在标签控件上使用else if！ 您必须为每个控件使用单独的 if/else。

if (this.progressBar1.InvokeRequired)
     progressBar1.Invoke(new Action(delegate() { progressbar1.Value++; }));
else 
     progressbar1.Value++; //pseudo-code 

Answer 2

您应该在 WinForms 中使用 BackgroundWorker 控件进行异步操作。使用详情和示例请参考thisMSDN 文章。