后台工作人员冻结 WPF

Question

关注BackgroundWorker 有什么问题？ 如果我尝试设置一些断点..bw_ProgressChanged 的值会更新，但如果我正在运行所有，我的 WPF 是“冻结”

    public MainWindow()
    {
        InitializeComponent();
        bw.WorkerReportsProgress = true;
        bw.WorkerSupportsCancellation = true;
    }
    private readonly BackgroundWorker bw = new BackgroundWorker();
    private void btnStart_Click(object sender, RoutedEventArgs e)
    {
        if (!bw.IsBusy)
        {
            bw.DoWork += bw_DoWork;
            bw.RunWorkerCompleted += bw_RunWorkerCompleted;
            bw.ProgressChanged += bw_ProgressChanged;
            bw.RunWorkerAsync();
        }
    }
    private void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
    {
        this.lblProgress.Content= e.ProgressPercentage.ToString()+ "%";
        this.pb.Value = e.ProgressPercentage;
    }

    private void bw_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        if (e.Cancelled)
            lblProgress.Content = "Cancel !";

        else if (e.Error!=null)
            lblProgress.Content= "Error: " + e.Error.Message;

        else
            lblProgress.Content = "Finish !";
    }

    private void bw_DoWork(object sender, DoWorkEventArgs e)
    {
        BackgroundWorker bw = sender as BackgroundWorker;
        StreamReader sr = new StreamReader("C:\\File 1.txt");
        StreamWriter sw = new StreamWriter("C:\\Out-File 1.txt");

            var fi = new FileInfo("C:\\File 1.txt");
            long total = (char)fi.Length;
            int  i = 0;
            string result;
            while (!sr.EndOfStream)
            {
                if (!bw.CancellationPending)
                {
                    result = sr.ReadLine();
                    sw.WriteLine(result);
                    i = i + (char)result.Length;
                    bw.ReportProgress((int)((decimal)i / (decimal)total * (decimal)100));
                }
                else
                {e.Cancel = true; break;}
            } sw.Close(); sr.Close();
    }

    private void btnCancel_Click(object sender, RoutedEventArgs e)
    {
        if (bw.WorkerSupportsCancellation)
            bw.CancelAsync();
    }
}

Answer 1

我知道这不是关于后台工作者问题的真正答案，但也许是时候使用名为 async/await 的“新”C# 5.0 功能了？我对您的代码进行了一些修改并对其进行了测试，并且所有工作都没有错误和冻结：

    private bool cancelled;

    private bool started;

    public MainWindow()
    {
        InitializeComponent();
    }

    private async void btnStart_Click(object sender, RoutedEventArgs e)
    {
        if (started) return;
        cancelled = false;
        started = true;
        try
        {
            await DoAsync();
        }
        catch (Exception exception)
        {
            lblProgress.Content = "Error: " + exception.Message;
            return;
        }
        finally
        {
            started = false;
        }
        lblProgress.Content = cancelled ? "Cancel !" : "Finish !";
    }

    private async Task DoAsync()
    {
        using (var sr = new StreamReader("C:\\File 1.txt"))
        {
            using (var sw = new StreamWriter("C:\\Out-File 1.txt"))
            {
                var fi = new FileInfo("C:\\File 1.txt");
                long total = (char) fi.Length;
                int i = 0;
                string result;
                while (!sr.EndOfStream)
                {
                    result = await sr.ReadLineAsync();
                    await sw.WriteLineAsync(result);
                    i = i + (char) result.Length;
                    ProgressChanged((int) ((decimal) i/(decimal) total*(decimal) 100));
                    if (cancelled) return;
                }
            }
        }
    }

    private void ProgressChanged(int progress)
    {
        this.lblProgress.Content = progress.ToString() + "%";
        this.pb.Value = progress;
    }

    private void btnCancel_Click(object sender, RoutedEventArgs e)
    {
        cancelled = true;
    }

另外我对百分比计算的建议是检查它是否在每次迭代中发生变化，然后进行界面更新。

Answer 2

问题在于您的后台工作人员的工作部分（即从文件中读取一行并将该行写入另一个文件）是一个非常简短的操作，在这个非常简短的操作之后您更新用户界面。

这意味着您非常频繁地更新用户界面，并且用户界面线程非常忙于进行这些更新。因此它没有机会做你想做的其他事情。

如果您将更新次数限制为 100，您应该会发现用户界面没有锁定。

BackgroundWorker bw = sender as BackgroundWorker;
StreamReader sr = new StreamReader("C:\\File 1.txt");
StreamWriter sw = new StreamWriter("C:\\Out-File 1.txt");

var fi = new FileInfo("C:\\File 1.txt");
long total = (char)fi.Length;
int  i = 0;
string result;
long updateIncrement = total / 100;
long nextUpdate = 0;
while (!sr.EndOfStream)
{
    if (!bw.CancellationPending)
    {
        result = sr.ReadLine();
        sw.WriteLine(result);
        i = i + (char)result.Length;
        if ( i > nextUpdate )
        {
            nextUpdate += updateIncrement;
            bw.ReportProgress((int)((decimal)i / (decimal)total * (decimal)100));
        }
    }
    else
    {
        e.Cancel = true; 
        break;
    }
} 
sw.Close(); 
sr.Close();

顺便说一句，为什么要将文件长度转换为 char 然后将其分配给 long？ 如果文件大小大于 0xFFFF，您将遇到问题。

Answer 3

我猜您在工作线程中遇到了 NullReferenceException，因为您没有检查 result != null，当您到达流的末尾时会发生这种情况。

编辑：我在这里可能是错的 - 我不知道 sr.EndOfStream 在你读完最后一行之后是否为真。此外，即使您遇到异常，runWorkerCompleted 应该也会执行。也就是说，您可能在这里泄漏了流；您应该将读取的流包装在 using 块中。

Answer 4

用户界面中的冻结通常是由 GUI 线程繁忙引起的。为什么它很忙，我无法从代码中看出。也许你有一些其他的 UI 代码在进程中间的某个地方触发，它循环并且永远不会退出。这将导致进度更新和/或 RunWorkerCompleted 方法永远不会执行（冻结）。

我也会从找出何时冻结开始。也许这可以给你一些线索。在进度报告前后以及文件打开和关闭前后插入Debug.WriteLine-lines。看看调试输出是什么样子的。与 UI 更新相反，调试写入可以从任何线程触发，因此非常适合调试线程问题。