BackgroundWorker блокирует главный поток

Driwars · Регистрация: 03.09.2013

Author24 — интернет-сервис помощи студентам

Подскажите как поправить. Код копирует файлы отмеченные галочками в ListView, увеличивает значение прогресс бара и добавляет имя в текст бокс. Прогресс бар работает неплохо, текст бокс обновляется но с запозданием, и главная форма "не отвечает" во время копирования.
Не совсем понимаю работу делегатов, буду рад если подскажете.

C#

private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
 
            this.Invoke((MethodInvoker)delegate
                           {
                               progressBar1.Value = 0;
            double cou = 100 / listView1.CheckedItems.Count;
 
            foreach (ListViewItem c in listView1.CheckedItems)
            {
                if (c.Checked)
                {
                    string curfile = c.SubItems[0].Text;
                    string curpath = c.SubItems[1].Text;
                    string distpach = c.SubItems[2].Text;
 
 
                    
 
                    if (c.SubItems[3].Text == "False") 
                    {
                        try
                        {
                            if (!Directory.Exists(distpach)) 
                            {
                                Directory.CreateDirectory(distpach);
                            
                            }
                            File.Move(Path.Combine(curpath, curfile), Path.Combine(distpach, curfile));
                            richTextBox1.AppendText("переещйн файл из " + c.Name + " в " + distpach);
                        }
                        catch (IOException ex)
                        {
                            MessageBox.Show("Файл " + c.Name + " не перемещён" + Environment.NewLine + ex.Message);
                        }
                    }
                   
                
                }
                
                               progressBar1.Value = progressBar1.Value + (int)(cou* 1.0f);
                
            }
            });
        }

@kolorotur · 13.03.2014, 18:46

Сообщение от Driwars

this.Invoke

Эта строчка перенаправляет указанный аргументом делегат в родительский поток формы.
Фактически, вся работа выполняется в основном потоке, потому и зависания.

Насколько я понял, вы сделали это из-за того, что используете при обработке элементы управления формы и пытаетесь избежать вылета InvalidOperationException при обращении к ним из дочерних потоков.

Но если вам нужно делегировать только обращение к контролам, то зачем перенаправлять всю работу в основной поток?

Вызывайте Invoke только на тех строчках, в которых происходит обращение к элементам управления.
Ну а если уж вы используете BackgroundWorker (судя по сигнатуре метода), то и вообще забудьте про Invoke и используйте метод ReportProgress — он сам отмаршаллит куда надо.

Driwars · 13.03.2014, 20:12 **[ТС]**

kolorotur,
Спасибо.
Насколько я понимаю его можно использовать так

C#

}
                
                               progress= progress +(int)(cou* 1.0f);
                               
                               backgroundWorker1.ReportProgress(progress, "Working...");
            }
            backgroundWorker1.ReportProgress(100, "Complete!");
 
 
 
 private void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
        {
            progressBar1.Value = e.ProgressPercentage;
        }

Проблема не только в отправке в основной поток данных о прогрессе, но и получение из ListView значений это тоже требует делегата. Поэтому всему циклу foreach приходится делать Invoke.
Как нибудь разделить можно? Работать с программой во время выполнения не надо, но по крайней мере надо чтоб её можно было таскать и сворачивать в трей.
Может есть смысл перед копированием файлов занести данные из ListView в массивы, и освободить контрол? Думаю инициализация массива пройдёт почти без задержки.

@kolorotur 17790 / 12941 / 3381 Регистрация: 17.09.2011 Сообщений: 21,215
	13.03.2014, 18:46	2
	Сообщение от Driwars this.Invoke Эта строчка перенаправляет указанный аргументом делегат в родительский поток формы. Фактически, вся работа выполняется в основном потоке, потому и зависания. Насколько я понял, вы сделали это из-за того, что используете при обработке элементы управления формы и пытаетесь избежать вылета InvalidOperationException при обращении к ним из дочерних потоков. Но если вам нужно делегировать только обращение к контролам, то зачем перенаправлять всю работу в основной поток? Вызывайте Invoke только на тех строчках, в которых происходит обращение к элементам управления. Ну а если уж вы используете BackgroundWorker (судя по сигнатуре метода), то и вообще забудьте про Invoke и используйте метод ReportProgress — он сам отмаршаллит куда надо. 1

	13.03.2014, 20:12

Опции темы