Зависание программы при выполнении длительного цикла

@Vitukr · Регистрация: 08.11.2012

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

В моей программе выполняется долго, очень долго цикл, так как много данных обрабатывается.
Программа работает и отрабатывает цикл нормально.
Но на время выполнения цикла, программа как бы зависает.
Как это происходит.
Если переключиться на другую программу, а потом вернуться, то в тайтле программы добавляется текст "Не отвечает" и при клике на программу она покрывается белым фоном.
Такое состояние до момента завершения цикла, после чего программа возвращается в нормальное состояние и в цикле все выполнено правильно.

У меня стоит задача убрать именно это неприятное явление, а не исправлять код, так как код работает хорошо.
Я пробовал выводить цикл в отдельный поток через BackgroundWorker, но ничего не меняется.

Пробовал также добавлять в каждый шаг итерации цикла Application.DoEvents();, но также не помогает.

Какие еще есть варианты?

@Монфрид · 26.01.2015, 16:30

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

через BackgroundWorker, но ничего не меняется.

значит что то не так. Единственный способ решения - вывод тяжёлых операций в отдельные потоки

@Vitukr · 26.01.2015, 16:36 **[ТС]**

Там много параметров передается и по ходу цикла идет обновление значений, скажем в таблицах. Может это влияет? Параметры вроде я корректно передаю, в объекте аргумента. Там же и таблицы передаются. То есть не нарушается принадлежность потокам, вроде.

@Монфрид · 26.01.2015, 18:00

без кода мы можем бесконечно долго рассуждать, что у вас там не так

@Vitukr · 26.01.2015, 18:18 **[ТС]**

Вот здесь запускается поток

C#
1
2
3
4
5
6
7
8
9
10
11
12
Arguments arguments = new Arguments();
                    arguments.checkBox_same = checkBox_same;
                    arguments.listBox_primary_key = listBox_primary_key;
                    arguments.listBox_primary_key_excel = listBox_primary_key_excel;
                    arguments.listBox_fields = listBox_fields;
                    arguments.listBox_fields_excel = listBox_fields_excel;
                    arguments.dataGridView_excel_file = dataGridView_excel_file;
                    arguments.dataGridView_currenttable = dataGridView_currenttable;
                    arguments.updatedRows = updatedRows;
                    arguments.newPriceColor = newPriceColor;
 
                    backgroundWorker1.RunWorkerAsync(arguments);

Здесь выполняется

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            BackgroundWorker worker = sender as BackgroundWorker;
            Arguments arguments = e.Argument as Arguments;
            update_dataset_background(worker, e, arguments.checkBox_same, ref arguments.listBox_primary_key, ref arguments.listBox_primary_key_excel, ref arguments.listBox_fields, ref arguments.listBox_fields_excel, ref arguments.dataGridView_excel_file, ref arguments.dataGridView_currenttable, ref arguments.updatedRows, ref arguments.newPriceColor);
            e.Result = arguments;
        }
 
public void update_dataset_background(BackgroundWorker worker, DoWorkEventArgs e, CheckBox checkBox_same, ref ListBox listBox_primary_key, ref ListBox listBox_primary_key_excel, ref ListBox listBox_fields, ref ListBox listBox_fields_excel, ref DataGridView dataGridView_excel_file, ref DataGridView dataGridView_currenttable, ref uint updatedRows, ref Color newPriceColor)
        {
            try
            {
                if (dataGridView_currenttable.DataMember == "TableCurrent")
                {
                    if (checkBox_same.Checked)
                    {
                        update_dataset_same(worker, ref listBox_primary_key, ref listBox_primary_key_excel, ref listBox_fields, ref listBox_fields_excel, ref dataGridView_excel_file, ref dataGridView_currenttable, ref updatedRows, ref newPriceColor);
                    }
                    else
                    {
                        update_dataset(worker, ref listBox_primary_key, ref listBox_primary_key_excel, ref listBox_fields, ref listBox_fields_excel, ref dataGridView_excel_file, ref dataGridView_currenttable, ref updatedRows, ref newPriceColor);
                    }
                }
            }
            catch (Exception exc)
            {
                MessageBox.Show(exc.Message);
            }
        }
 
private void update_dataset(BackgroundWorker worker, ref ListBox listBox_primary_key, ref ListBox listBox_primary_key_excel, ref ListBox listBox_fields, ref ListBox listBox_fields_excel, ref DataGridView dataGridView_excel_file, ref DataGridView dataGridView_currenttable, ref uint updatedRows, ref Color newPriceColor)
        {
            try
            {
                updatedRows = 0;
                if (listBox_primary_key.SelectedItems.Count > 0 && listBox_primary_key_excel.SelectedItems.Count > 0)
                {
                    if (listBox_primary_key.SelectedItems.Count == listBox_primary_key_excel.SelectedItems.Count)
                    {
                        if (listBox_fields.SelectedItems.Count > 0 && listBox_fields_excel.SelectedItems.Count > 0)
                        {
                            if (listBox_fields.SelectedItems.Count == listBox_fields_excel.SelectedItems.Count)
                            {
                                // ProgressBar
                                //this.startProgressBar(this.dataGridView_excel_file.RowCount);
                                for (int i = 0; i < dataGridView_excel_file.RowCount; i++)
                                {
                                    // Get primary keys
                                    List<string> excel_keys = new List<string>();
                                    for (int ii = 0; ii < listBox_primary_key_excel.SelectedItems.Count; ii++)
                                    {
                                        if (string.IsNullOrEmpty(dataGridView_excel_file.Rows[i].Cells[listBox_primary_key_excel.SelectedItems[ii].ToString()].Value.ToString().Trim()))
                                        {
                                            excel_keys.Clear();
                                            continue;
                                        }
                                        excel_keys.Add(dataGridView_excel_file.Rows[i].Cells[listBox_primary_key_excel.SelectedItems[ii].ToString()].Value.ToString().Trim());
                                        //Application.DoEvents();
                                    }
                                    if (excel_keys.Count == 0)
                                    {
                                        continue;
                                    }
                                    for (int j = 0; j < dataGridView_currenttable.SelectedRows.Count; j++)
                                    {
                                        if (dataGridView_currenttable.SelectedRows[j].DefaultCellStyle.BackColor != newPriceColor)
                                        {
                                            // Get primary keys
                                            List<string> table_keys = new List<string>();
                                            for (int jj = 0; jj < listBox_primary_key.SelectedItems.Count; jj++)
                                            {
                                                if (string.IsNullOrEmpty(dataGridView_currenttable.SelectedRows[j].Cells[listBox_primary_key.SelectedItems[jj].ToString()].Value.ToString().Trim()))
                                                {
                                                    table_keys.Clear();
                                                    continue;
                                                }
                                                table_keys.Add(dataGridView_currenttable.SelectedRows[j].Cells[listBox_primary_key.SelectedItems[jj].ToString()].Value.ToString().Trim());
                                                //Application.DoEvents();
                                            }
                                            if (table_keys.Count == 0)
                                            {
                                                continue;
                                            }
 
                                            // Exists
                                            if (excel_keys.Intersect(table_keys).Count<string>() == excel_keys.Count)
                                            {
                                                bool updated = false;
                                                for (int k = 0; k < listBox_fields.SelectedItems.Count; k++)
                                                {
                                                    Type valueType = dataGridView_currenttable.Columns[listBox_fields.SelectedItems[k].ToString()].ValueType;
                                                    object valueFrom = dataGridView_excel_file.Rows[i].Cells[listBox_fields_excel.SelectedItems[k].ToString()].Value;
                                                    if (correctValue(valueType, ref valueFrom))
                                                    {
                                                        dataGridView_currenttable.SelectedRows[j].Cells[listBox_fields.SelectedItems[k].ToString()].Value = valueFrom;
                                                        //dataGridView_currenttable.BeginInvoke(dataGridView_currenttable.Update());
                                                        updated = true;
                                                    }
                                                }
                                                if (updated)
                                                {
                                                    dataGridView_currenttable.SelectedRows[j].DefaultCellStyle.BackColor = newPriceColor;
                                                    updatedRows++;
                                                }
                                                break;
                                            }
                                        }
                                        //Application.DoEvents();
                                    }
                                    //this.progressBar.PerformStep();
                                    //int percentComplete = (int)(i / (float)(this.dataGridView_excel_file.RowCount * 100.0));
                                    worker.ReportProgress(i);
                                }
                            }
                        }
                    }
                }
                //this.progressBar.Visible = false;
                dataGridView_currenttable.Select();
            }
            catch (Exception exc)
            {
                MessageBox.Show(exc.Message);
            }
        }

И по завершению цикла

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
        {
            //throw new NotImplementedException();
            this.progressBar.Visible = false;
            MessageBox.Show("Обновилось " + this.updatedRows.ToString() + " строк");
            Arguments arguments = e.Result as Arguments;
            checkBox_same = arguments.checkBox_same;
            listBox_primary_key = arguments.listBox_primary_key;
            listBox_primary_key_excel = arguments.listBox_primary_key_excel;
            listBox_fields = arguments.listBox_fields;
            listBox_fields_excel = arguments.listBox_fields_excel;
            dataGridView_excel_file = arguments.dataGridView_excel_file;
            dataGridView_currenttable = arguments.dataGridView_currenttable;
            updatedRows = arguments.updatedRows;
            newPriceColor = arguments.newPriceColor;
        }

@Монфрид · 26.01.2015, 20:19

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

Здесь выполняется

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

@SharpDeveloper · 27.01.2015, 23:29

Вы пробовали добавить в цикл Application.DoEvents() на каждую 1000 итераций к примеру?

@Vitukr · 27.01.2015, 23:40 **[ТС]**

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

Вы пробовали добавить в цикл Application.DoEvents() на каждую 1000 итераций к примеру?

Да, пробовал. Никакого изменения.
Первым делом я это и сделал, до вывода в другой поток.

@Storm23 · 27.01.2015, 23:56

Vitukr, Нельзя обращаться к визуальным компонентам из неосновного потока.

@Vitukr · 28.01.2015, 00:15 **[ТС]**

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

Vitukr, Нельзя обращаться к визуальным компонентам из неосновного потока.

А какое же тогда может быть решение при таком огромном цикле?

@SharpDeveloper · 28.01.2015, 00:31

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

Да, пробовал. Никакого изменения.

Это странно, покажите код.

@Vitukr · 28.01.2015, 00:38 **[ТС]**

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

Это странно, покажите код.

Выше есть код. Там закомментирован Application.DoEvents() в циклах.
Закомментировал, когда добавлял поток.

@Storm23 · 28.01.2015, 00:38

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

А какое же тогда может быть решение при таком огромном цикле?

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

@Vitukr · 28.01.2015, 00:48 **[ТС]**

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

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

Наверное так и придется переделывать, хотя не хотелось.

@insite2012 · 28.01.2015, 01:10

Vitukr, могу посоветовать хороший и (относительно) простой способ.
Создайте класс, унаследуйте его от BackgroundWorker-а, переопределите его виртуальный метод OnDoWork() и в этом методе реализуйте все что надо.
Так же у вашего класса будет два события: ProgressChanged и RunWorkerCompleted, подписавшись на которые, вы сможете обновлять интерфейс без лишних заморочек, напрямую.

@Vitukr · 08.03.2015, 15:14 **[ТС]**

Нашел причину, почему прогрес бар не работал даже в отдельном потоке.
Сейчас речь идет о приложении WPF, но думаю разницы где использовать backgroundWorker нет.
Вся проблема в добавлении кода

C#
1
Thread.Sleep(1);

Вот код метода, где запускается длительный цикл:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
private int DoSlowProcess(int iterations, BackgroundWorker worker, DoWorkEventArgs e)
        {
            int result = 0;
            for (int i = 0; i <= iterations; i++)
            {
                if (worker != null)
                {
                    if (worker.CancellationPending)
                    {
                        e.Cancel = true;
                        return result;
                    }
                    if (worker.WorkerReportsProgress)
                    {
                        int percentComplete =
                        (int)((float)i / (float)iterations * 100);
                        worker.ReportProgress(percentComplete);
                    }
                }
                Thread.Sleep(1);
                result = i;
            }
            return result;
        }

Если закомментировать Thread.Sleep(1);
то ни прогрес бар, ни обновление текста в текстбоксе не работает.
Как и кнопка остановки цикла.
Если же не комментировать, то все работает.
Я снял видео, чтобы наиболее наглядно было видно это.

https://www.youtube.com/watch?... e=youtu.be

Теперь остался вопрос, как объяснить такое поведение?
В чем тут важность Thread.Sleep(1);
При этом, понятно, значение длительности сна потока можна ставить любое.

Хочу также отметить, что все примеры в Интернете использования backgroundWorker с прогрессбаром, но и наверное без прогрессбара, становятся ничтожными, если также комментировать Thread.Sleep(1);

Например, вот этот пример, который рекомендуется Майкрософтом: http://elegantcode.com/2009/07... to-the-ui/
Я комментировал в нем Thread.Sleep(1); и там ничего не работает в двух случаях.
Видимо, авторы всех этих примеров не знают этот тайный код Thread.Sleep(1);

@Storm23 · 08.03.2015, 23:33

Vitukr, А сколько ядер в вашем процессоре?

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

Теперь остался вопрос, как объяснить такое поведение?
В чем тут важность Thread.Sleep(1);

Очень просто. Пока у вас крутится цикл, процессор не переключается на другие потоки. Но как только он встречает метод Thread.Sleep(1); он понимает, что текущий поток будет спать некторое время и производит переключение на другие потоки. Поэтому начинают работать прогрессбары и другие компоненты, работающие в другом потоке.

@Vitukr · 08.03.2015, 23:53 **[ТС]**

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

Очень просто. Пока у вас крутится цикл, процессор не переключается на другие потоки. Но как только он встречает метод Thread.Sleep(1); он понимает, что текущий поток будет спать некторое время и производит переключение на другие потоки. Поэтому начинают работать прогрессбары и другие компоненты, работающие в другом потоке.

Это я сразу понял, когда закомментировал/разкомментировал код сна.
Может я не правильно сформулировал вопрос.
Суть вопроса, зачем все эти "танцы" с backgroundWorker?
Ведь там же есть ReportProgress, который и без сна должен передавать значения в основной поток в процессе выполнения цикла?
По сути, Thread.Sleep(1); используется в примерах только для увеличения времени работы цикла, искусственно увеличивая время цикла. Так сказать, для демонстрационных целей.
Но, оказывается, этот код есть основной и для не демо программ.

При реально большом цикле использование Thread.Sleep(1); даже с 1 миллисекундой значительно будет увеличивать время работы цикла. Представьте, что одна итерация цикла без Thread.Sleep(1); будет меньше 1 миллисекунды, а реально там сотые доли миллисекунды, то добавление слип увеличивает время цикла в десятки раз.
Это просто недопустимо при программировании.

@Storm23 · 09.03.2015, 00:12

Vitukr, Сколько ядер в вашем процессоре?

@Vitukr · 09.03.2015, 00:19 **[ТС]**

4 ядра.

Добавлено через 2 минуты
Вот здесь:
https://msdn.microsoft.com/ru-... kz4s1.aspx
приводится пример использования беграундворкер.
И в коде интересный момент:

C#
1
2
// Uncomment for testing.
                //System.Threading.Thread.Sleep(5);

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

Новые блоги и статьи Все статьи Все блоги /
Подключение Box2D v3 к SDL3 для Android: физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .	Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .
Влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .	Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .

@Vitukr 14 / 4 / 1 Регистрация: 08.11.2012 Сообщений: 189 Записей в блоге: 1

	.NET 4.x Зависание программы при выполнении длительного цикла 26.01.2015, 16:26. Показов 16196. Ответов 27 Метки нет (Все метки) В моей программе выполняется долго, очень долго цикл, так как много данных обрабатывается. Программа работает и отрабатывает цикл нормально. Но на время выполнения цикла, программа как бы зависает. Как это происходит. Если переключиться на другую программу, а потом вернуться, то в тайтле программы добавляется текст "Не отвечает" и при клике на программу она покрывается белым фоном. Такое состояние до момента завершения цикла, после чего программа возвращается в нормальное состояние и в цикле все выполнено правильно. У меня стоит задача убрать именно это неприятное явление, а не исправлять код, так как код работает хорошо. Я пробовал выводить цикл в отдельный поток через BackgroundWorker, но ничего не меняется. Пробовал также добавлять в каждый шаг итерации цикла Application.DoEvents();, но также не помогает. Какие еще есть варианты? 0

@Vitukr 14 / 4 / 1 Регистрация: 08.11.2012 Сообщений: 189 Записей в блоге: 1
	26.01.2015, 16:36 [ТС]
	Там много параметров передается и по ходу цикла идет обновление значений, скажем в таблицах. Может это влияет? Параметры вроде я корректно передаю, в объекте аргумента. Там же и таблицы передаются. То есть не нарушается принадлежность потокам, вроде. 0

@Монфрид 1245 / 1055 / 293 Регистрация: 07.03.2012 Сообщений: 3,247
	26.01.2015, 18:00
	без кода мы можем бесконечно долго рассуждать, что у вас там не так 0

@SharpDeveloper 141 / 138 / 22 Регистрация: 16.02.2012 Сообщений: 453
	27.01.2015, 23:29
	Вы пробовали добавить в цикл Application.DoEvents() на каждую 1000 итераций к примеру? 0

@Storm23 10425 / 5155 / 1825 Регистрация: 11.01.2015 Сообщений: 6,226 Записей в блоге: 34
	27.01.2015, 23:56
	Vitukr, Нельзя обращаться к визуальным компонентам из неосновного потока. 1

@insite2012 5548 / 4311 / 1218 Регистрация: 12.10.2013 Сообщений: 12,371 Записей в блоге: 2
	28.01.2015, 01:10
	Vitukr, могу посоветовать хороший и (относительно) простой способ. Создайте класс, унаследуйте его от BackgroundWorker-а, переопределите его виртуальный метод OnDoWork() и в этом методе реализуйте все что надо. Так же у вашего класса будет два события: ProgressChanged и RunWorkerCompleted, подписавшись на которые, вы сможете обновлять интерфейс без лишних заморочек, напрямую. 1

@Storm23 10425 / 5155 / 1825 Регистрация: 11.01.2015 Сообщений: 6,226 Записей в блоге: 34
	09.03.2015, 00:12
	Vitukr, Сколько ядер в вашем процессоре? 0