многопоточность WinForms

Question

Как заставить работать вывод в Label/TextBox в многопоточном режиме?

Происходит считывание и обработка директории с файлами, чью динамику показывает Progressbar. Хочу чтобы параллельно с заполнением этой полоски прогресс бара выводилась процентовка относительно количества обработанных файлов. Однако, процент выводится только в самом конце (100%) никаких промежуточных процентов не выводится.

delegate void Print(string str);
private void Form1_Load(object sender, EventArgs e) {
    Print p1 = (string str) => { label1.Text += str; };       
    var fb =new FolderBrowserDialog();
    fb.ShowDialog();

    var dir=new DirectoryInfo(fb.SelectedPath);
    var files = dir.GetFileSystemInfos();

    progressBar1.Maximum = files.Length;
    foreach(var f in files) {
        hand(f.FullName);
        progressBar1.Value++;
        //1)
        label1.Text = Convert.ToDouble(progressBar1.Value/ progressBar1.Maximum)*100+"%";
        //2)
        //Task.Run(() => { DoSomething(); });
        //3)
        //label1.Invoke(p1, new object[] { (progressBar1.Value / progressBar1.Maximum) * 100 + "%" });
    }

}

public void DoSomething() {
    if (label1.InvokeRequired) {
        label1.Invoke(new Action(DoSomething));
    }
    else {
        label1.Text = Convert.ToDouble(progressBar1.Value / progressBar1.Maximum) * 100 + "%";
    }
}

По частоте появления этот вопрос самый задаваемый. Смотрим наглядный пример. — Bulson, Apr 05 '18 at 07:36

score 1 · Answer 1 · edited Apr 05 '18 at 07:30

Для этих целей можно использовать класс BackgroundWorker, его метод ReportProgress а также события. Пример использования ниже.

public partial class Form1 : Form
{
    BackgroundWorker bgw = new BackgroundWorker();       
    public Form1()
    {
        InitializeComponent();
        label1.Text = "";
        label2.Text = "";
    }

    private void button1_Click_1(object sender, EventArgs e)
    {
        bgw.DoWork += new DoWorkEventHandler(bgw_DoWork);
        bgw.ProgressChanged += new ProgressChangedEventHandler(bgw_ProgressChanged);
        bgw.RunWorkerCompleted += new RunWorkerCompletedEventHandler(bgw_RunWorkerCompleted);
        bgw.WorkerReportsProgress = true;
        bgw.RunWorkerAsync();
    }

    void bgw_DoWork(object sender, DoWorkEventArgs e)
    {
        int total = 57; //some number (this is your variable to change)!!

        for (int i = 0; i <= total; i++) //some number (total)
        {
            System.Threading.Thread.Sleep(100);
            int percents = (i * 100) / total;
            bgw.ReportProgress(percents, i);
            //2 arguments:
            //1. procenteges (from 0 t0 100) - i do a calcumation 
            //2. some current value!
        }
    }

    void bgw_ProgressChanged(object sender, ProgressChangedEventArgs e)
    {
        progressBar1.Value = e.ProgressPercentage;
        label1.Text = String.Format("Progress: {0} %", e.ProgressPercentage);
        label2.Text = String.Format("Total items transfered: {0}", e.UserState);
    }

    void bgw_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
         //do the code when bgv completes its work
    }
}

UPDATE: Как верно предложил @Сергей, можно реализовать обновление на Task'ах:

private bool complete = false;
private int fileProgress = 0;
private int totalProgress = 100;

private async void button1_Click(object sender, EventArgs e)
{
    await Task.WhenAll(Task.Factory.StartNew(()=> {
        while (!complete)
        {
            if (fileProgress != 0 && totalProgress != 0)
            {
                //Here you signal the UI thread to execute the action:
                progressBar1.Invoke((Action)(() =>
                {
                    //This is done by the UI thread:
                    var a = (((double)fileProgress / (double)totalProgress) * 100);
                    progressBar1.Value = Convert.ToInt32(a);
                    label1.Text = a.ToString();
                }));
            }
        }
    }), 
    Task.Factory.StartNew(()=> {
        for (int i = 0; i < totalProgress; i++)
        {
           fileProgress++;
           Thread.Sleep(3000);
        }
        complete = true;
    } 
    ));
    MessageBox.Show("Done");  //here we're on the UI thread.
}

если студия и фреймворк не древнии - лучше уж async/await и Task использовать. — srvr4vr, Apr 05 '18 at 03:35
В комментариях к этому вопросу я дал ссылку на пример. Посмотрите его, пожалуйста, т.к. на дворе уже 2018 год, а вы всё еще, судя по вашему коду, находитесь где-то в 2012. — Bulson, Apr 05 '18 at 07:44
@Bulson и что такого нового я там должен увидеть из 2018 ? Такие же таски как и у меня в ответе. Работают так же корректно, как и мой код. Принципиально нового из 2018 я там ничего не нашел. — Exodium, Apr 05 '18 at 08:18
BackgroundWorker древнее г**** мамонта, для простого создания задачи в .Net 4.6 был добавлен метод Task.Run() вместо Task.Factory.StartNew(). — Bulson, Apr 05 '18 at 08:24
@Bulson мне автор вопроса не указал его версию .NET, мб у него проект версии более ранней, чем 4.6. В таком случае ваши комменты не актуальны. Мой код будет работать под более широким спектром версий.NET — Exodium, Apr 05 '18 at 08:37

score 0 · Answer 2 · answered Apr 05 '18 at 05:42

Я пользуюсь для этого таким способом. Не знаю насколько это правильно. Допустим у нас есть некий класс, в котором что-то происходит (фоновый процесс о ходе которого необходимо уведомлять).

public class Rename
{
   ...
   public event Action<int> ProcessChanged;

   public void ABC()
   {
       ...
       ProcessChanged(i);
       ...
   }    
   ...    
}

В основном классе создаем объект подписываемся на событие

Rename.ProcessChanged += FileOper_ProccesChanged;

private void FileOper_ProccesChanged(int progress)
{
    Action action = () =>
    {
       PGBar_Music.Value = progress;
    };
    if (InvokeRequired)
          Invoke(action);
    else
          action();
}

многопоточность WinForms

2 Answers2