Фоновый работник не рендерит окно бегло - PullRequest
Новая
       18

Фоновый работник не рендерит окно бегло

0 голосов
/ 18 ноября 2011

Я только что попал в WPF и в настоящее время испытываю удачу с фоновым работником, поэтому я решил, что просто открою любой файл с помощью FileOpenDialog, переберу все байты внутри файла и сообщу об общем прогрессе через worker.ReportProgress в процентах ... увы, это работает примерно как ~ 20 раз, а затем он действительно застревает и внезапно останавливается на 100%.

Вот мой код: 

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;
using Microsoft.Win32;
using System.IO;
using System.Threading;
using System.ComponentModel;

namespace BitStream
{
public partial class MainWindow : Window
{
    private int bytes = 0;
    private long length = 0;

    public MainWindow()
    {
        InitializeComponent();
    }

    private void selectFile_Click(object sender, RoutedEventArgs e)
    {
        BackgroundWorker bw = new BackgroundWorker();
        OpenFileDialog ofd = new OpenFileDialog();
        if ((bool)ofd.ShowDialog())
        {
            FileInfo fi = new FileInfo(ofd.FileName);
            this.length = fi.Length;
            bw.DoWork += bw_DoWork;
            bw.RunWorkerCompleted += bw_RunWorkerCompleted;
            bw.ProgressChanged += bw_ProgressChanged;
            bw.WorkerReportsProgress = true;
            Stream str = ofd.OpenFile();

            bw.RunWorkerAsync(str);
        }
    }

    private void bw_DoWork(object sender, DoWorkEventArgs e)
    {
        Stream str = (Stream)e.Argument;
        int singleByte = 0;
        this.Dispatcher.Invoke(
                new Action(() =>
                {
                    int currentProgress = 0;
                    while ((singleByte = str.ReadByte()) != -1)
                    {

                        label1.Content = singleByte;
                        bytes++;

                        currentProgress = Convert.ToInt32(((double)bytes) / length * 100);
                        if (currentProgress > progress)
                        {
                            progress = currentProgress;
                            ((BackgroundWorker)sender).ReportProgress(progress);
                            Thread.Sleep(100);
                        }
                    }
                }

            ), System.Windows.Threading.DispatcherPriority.Render);
    }

    private void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
    {
        label2.Content = e.ProgressPercentage + "% completed";
    }

    private void bw_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
    }
}

}

Метки 1 и 2 предназначены для отображения текущего байта и текущего прогресса в%.

Не стесняйтесь критиковать и все остальные аспекты моего кода, я только начал работать с WPF сегодня.

Отредактированный метод DoWork: 

 private void bw_DoWork(object sender, DoWorkEventArgs e)
    {
        Stream str = (Stream)e.Argument;
        int singleByte = 0;

        int currentProgress = 0;
        while ((singleByte = str.ReadByte()) != -1)
        {
            this.Dispatcher.Invoke(
                new Action(() =>
                {
                    label1.Content = singleByte;
                }), System.Windows.Threading.DispatcherPriority.Render);
            bytes++;

            currentProgress = Convert.ToInt32(((double)bytes) / length * 100);
            if (currentProgress > progress)
            {
                progress = currentProgress;
                this.Dispatcher.Invoke(
                new Action(() =>
                {
                    ((BackgroundWorker)sender).ReportProgress(progress);


                }), System.Windows.Threading.DispatcherPriority.Render);
                Thread.Sleep(500);        
            }
        }
    }

Спасибо

Денис

Ответы [ 2 ]

1 голос
/ 18 ноября 2011

Первая мысль: вы не распределяете возврат из openfiledialog, поэтому чем больше вы запускаете его, тем больше ресурсов вы выбрасываете ... Я бы бросил имя файла на работника и затем позволил бы ему управлять ресурсом, но 

using(Stream s = ofd.OpenFileDalog())
{
   get length and such
}
// run up woker pass filename.

в вашем коде вызова решит проблему, так как я предполагаю, что вы используете length для сортировки индикатора выполнения.

1 голос
/ 18 ноября 2011

Итак, если вы действительно хотите сделать кросс-потокный вызов для каждого байта (что я бы не рекомендовал), код будет выглядеть примерно так: 

private void bw_DoWork(object sender, DoWorkEventArgs e) 
{ 
    Stream str = (Stream)e.Argument; 
    int singleByte = 0; 
    int currentProgress = 0; 
    while ((singleByte = str.ReadByte()) != -1) 
    { 

       bytes++; 
       this.Dispatcher.Invoke( 
            new Action(() => 
            { 
                    label1.Content = singleByte; 
            } 

        ), System.Windows.Threading.DispatcherPriority.Render); 

        currentProgress = Convert.ToInt32(((double)bytes) / length * 100); 
        if (currentProgress > progress) 
        { 
            progress = currentProgress; 
            ((BackgroundWorker)sender).ReportProgress(progress); 
            Thread.Sleep(100); 
        } 
    } 
}

Идея в том, что вы можете манипулировать толькоDispatcherObject в потоке, в котором он был создан.

...