Question

我有一个Windows窗体程序（** VS 2010 .NET 4 **），可以检测目录的递归文件夹和子文件夹并优化文件。我是通过任务库来做的，并且有一个进度条，显示项目和标签在进度条附近显示当前文件的进度。我有一个SetText（字符串文本）方法和委托（委托void SetTextCallback（字符串文本）;）用于此目的。当我想在进程结束时显示一个消息框时，我认为它是死锁的。但是当我没有在按钮6中使用task.Wait();时，它就可以了，UI也不会挂起。这是我的代码：

    public partial class Form1 : Form
    {


int MaxFileCounter = 0;
    static int FileCounter = 0;

delegate void SetTextCallback(string text);
delegate void SetProgressCallback(int i);

private void button6_Click(object sender, EventArgs e)
{
    Task task = Task.Factory.StartNew(() =>
    {
        editFiles(txtFilePath.Text);
    });

    task.Wait();

    MessageBox.Show("finished");
}

private void editFiles(string directoryPath)
{
    try
    {
        //DirectoryInfo dirInfo = new DirectoryInfo(txtFilePath.Text);
        DirectoryInfo dirInfo = new DirectoryInfo(directoryPath);
        //string[] extensionArray = { ".jpg", ".png", ".gif" ,".jpeg"};
        string[] extensionArray = txtExtensionList.Text.Split(';');
        HashSet<string> allowedExtensions = new HashSet<string>(extensionArray, StringComparer.OrdinalIgnoreCase);

        FileInfo[] files = Array.FindAll(dirInfo.GetFiles(), f => allowedExtensions.Contains(f.Extension));
        writeFiles(files);

        foreach (DirectoryInfo dir in dirInfo.GetDirectories())
        {
            editFiles(dir.FullName);
        }
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }
}

private void writeFiles(FileInfo[] files)
{
    try
    {
        foreach (FileInfo fileinfo in files)
        {
            MemoryStream mo;

            using (Image image = Image.FromFile(fileinfo.FullName))
            {

                SetText(fileinfo.FullName);

                FileCounter++;

                SetProgress(FileCounter * 100 / MaxFileCounter);

                mo = (MemoryStream)optimizeImage(image, int.Parse(txtPercent.Text));
            }

            byte[] bt = new byte[mo.Length];
            mo.Read(bt, 0, bt.Length);
            mo.Flush();
            mo.Close();

            string fullpath = fileinfo.FullName;

            File.WriteAllBytes(fullpath, bt);

        }
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }
}

private void SetText(string text)
{
    // InvokeRequired required compares the thread ID of the
    // calling thread to the thread ID of the creating thread.
    // If these threads are different, it returns true.
    if (lblFileName.InvokeRequired)
    {
        SetTextCallback d = new SetTextCallback(SetText);
        this.Invoke(d, new object[] { text });
    }
    else
    {
        this.lblFileName.Text = text;
    }
}
private void SetProgress(int i)
{
    if (progressBar1.InvokeRequired)
    {
        SetProgressCallback p = new SetProgressCallback(SetProgress);
        this.Invoke(p, new object[] { i });
    }
    else
    {
        this.progressBar1.Value = i;
    }
}
}

我该如何处理？

Answer 1

    this.Invoke(d, new object[] { text });

使用任务的目的是不要等待它。如果您仍然这样做，那么您将挂起UI线程。它变得紧张，不再响应来自Windows的通知。包括您自己的代码生成的代码，如Invoke（）调用。它向UI线程发送消息以寻找要调用的委托。

因此Invoke（）调用无法完成，UI线程挂起。 Task.Wait（）调用无法完成，因为任务挂起在Invoke（）调用上。一个致命的拥抱，一个标准的线程错误称为死锁。

请注意BeginInvoke（）如何解决此问题，它不会等待完成。解决僵局，并没有为您提供实时更新，因为您仍然有一个对全世界都没有的UI线程。您必须删除Wait（）调用。您可以通过使用TaskScheduler.FromCurrentSynchronizationContext（）添加任务延续来获取要运行的MessageBox.Show（）调用。你需要担心当用户关闭窗口时让这个任务停止，让它继续运行不会（通常）结果很好。添加到C＃版本5的async / await关键字是解决此问题的更通用的解决方案。

在工作线程中调用时的任务死锁

1 个答案: