C#调用原因任务永远不会完成

时间:2014-11-22 08:33:18

标签: c# multithreading

这是我的代码:

public void ReadCodes(){
        Task compute = Task.Run(() =>
        {
             //foo code
             foreach (var line in lines)
             {
                 //some other codes
                 SetText("doing something");
             }
        });
        compute.Wait();
        //some other foo
}
private void SetText(string text)
{
        if (this.lblUsedFiles.InvokeRequired)
        {
                SetTextCallback d = new SetTextCallback(SetText);
                this.Invoke(d, new object[] { text });
        }
        else
        {
                this.lblUsedFiles.Text = text;
        }
}

当我删除SetText("doing something")调用ReadCodes()函数时,它会挂起。我的代码出了什么问题?我想更新Task.Run()内的用户界面。

1 个答案:

答案 0 :(得分:5)

您的ReadCodes方法(可能在UI线程上执行)会生成一个新任务,并阻塞等待它 - UI线程在完成之前无法执行任何操作。

但是,您的任务(在线程池上运行)调用Invoke以在UI线程上运行操作,同时阻塞直到完成。由于两个操作都被阻塞等待,因此两者都无法继续,从而导致死锁。

要解决此问题,最简单的解决方案通常是将Invoke替换为BeginInvoke,从而允许您的任务继续(并完成),而无需等待UI更新。任务完成后,将释放UI线程,并处理挂起的UI更新。

编辑:响应更新的问题以包含循环:我建议您使用异步模式(在C#5中引入)。这将完全消除UI线程阻塞,并允许您在处理每一行时立即执行UI更新。

public async Task ReadCodes()
{
    foreach (var line in lines)
    {
        string text = await Task.Run(() =>
        {
            //foo code
            // return text;
        });

        this.lblUsedFiles.Text = text;
    }
    //some other foo
}

修改 2 :我已根据Peter Duniho's comment更新了我的代码。由于可能在UI线程上调用ReadCodes,因此您可以在等待每个任务时直接在其中更新UI(无需Invoke)。我假设//some other codes中执行的后台处理可以集成到后续迭代//foo code中。