Не могу достать данные из потока во время выполнения цикла

@aridscrut · Регистрация: 16.02.2012

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

Доброго времени суток.
Проблема вот в чём: есть код формы

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
private void button1_Click(object sender, EventArgs e)
        {
            
            var processor = new code();
            processor.Progress += ProcessorProgress;
 
            var thread = new Thread(processor.ThreadTask);
            thread.Start();
        
        }
        void ProcessorProgress(int progress)
        {
            if (progressBar1.InvokeRequired)
            {
                progressBar1.Invoke(new ProgressHandler(ProcessorProgress), progress);
            }
            else
            {
                progressBar1.Value = progress;
            }
        }

Он запускает процедуру в другом потоке

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 namespace MyProject
{
    public delegate void ProgressHandler(int progress);
    public class code
 
    {
        public event ProgressHandler Progress;
        //бла-бла переменные
        public void ThreadTask()
        {
         //объявление экземпляров и т.д.
         using (StreamReader st = baseinfo.OpenText())
            {
                int i = 0;
                p = (int)baseinfo.Length;                  //Замеряю кол-во строк в файле
                while ((sText = st.ReadLine()) != null) //Построчно ввожу данные из файла
                {
                    
                    i++;                  //кол-во уже занесенных строк
                 
                    Progress((i / (p)) * 100); //высчитывание данных для ПрогрессБара(макс. знач. 100)
                }
            }

Суть в том, что ПрогрессБар заполняется уже после цикла и сразу встает на максимальное значение, далее по коду еще цикл for, там тоже самое. Как сделать, чтоб прогрессБар обновлялся во время циклов?

Заранее спасибо!

@Casper-SC · 07.04.2012, 20:04

Создай новый проект, воспроизведи в нём эту проблему (ну чтобы без лишнего ненужного кода) и скинь сюда, так будет намного легче тебе помочь. На первый взгляд пока не увидел в чём беда конкретно. Самому проект мутить относительно долго, а так скачал, затестил, попробовал как-то решить, если решил, то тут рассказал в чём проблема. Хотя может кто-то что-то посоветует и поможет без проекта, смотри сам.

Добавлено через 3 минуты
А, кстати, скорее всего просто цикл выполняется моментально, вот и не видно никакого продвижения...

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

Он запускает процедуру в другом потоке

Запускает

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

метод.

@aridscrut · 07.04.2012, 20:08 **[ТС]**

Ну смотрим, код формы:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
        private void btnStart_Click(object sender, EventArgs e)
        {
            var processor = new FileProcessor();
            processor.Progress += ProcessorProgress;
 
            var thread = new Thread(processor.DoWork);
            thread.Start();
        }
        void ProcessorProgress(int progress)
        {
            if (progressBar1.InvokeRequired)
            {
                progressBar1.Invoke(new ProgressHandler(ProcessorProgress), progress);
            }
            else
            {
                progressBar1.Value = progress;
            }
        }

Код класса с вызываемой процедурой:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
namespace Invoke
{
    public delegate void ProgressHandler(int progress);
 
    public class FileProcessor
    {
        public event ProgressHandler Progress;
 
        public void DoWork()
        {
            for (i = 1; i <= 100; i++)
            {
                Thread.Sleep(100);
                if (Progress != null) Progress(i);
                    
            }
        }
    }
}

И всё робит на УРА. В предыдущем же примере, при точно таком же синтаксисе ПрогрессБар упорно отказывается обновляться с положенным ему шагом

@Casper-SC · 07.04.2012, 20:11

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

В предыдущем же примере, при точно таком же синтаксисе ПрогрессБар упорно отказывается обновляться с положенным ему шагом

А значения передаваемые на каждой итерации точно правильно просчитываются?

Добавлено через 1 минуту

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

C#
1
Progress((i / (p)) * 100); //высчитывание данных для ПрогрессБара(макс. знач. 100)

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

@aridscrut · 07.04.2012, 20:13 **[ТС]**

Расчитываются правильно, я их и в int конвертировал спецом уже. И цикл этот довольно долгий, файлы по 2-3 метра

Добавлено через 1 минуту

Сообщение от Casper-SC

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

Да это-то бог с ним, это не проблема, основная проблема у меня описана в топе

@Casper-SC · 07.04.2012, 20:15

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

Да это-то бог с ним, это не проблема, основная проблема у меня описана в топе

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

@aridscrut · 07.04.2012, 20:17 **[ТС]**

Сообщение от Casper-SC

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

Интерфейс тормозить не может, так как все расчеты у меня запущены в другом потоке

Добавлено через 1 минуту
Если б тормозил интерфейс, то форма бы не отвечала, а она держится молодцом=)

@Casper-SC · 07.04.2012, 20:30

Что-то там не правильно считается. Точка останова срабатывает 1 раз и значение t == 100

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
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            Shown += new EventHandler(Form1_Shown);
        }
 
        void Form1_Shown(object sender, EventArgs e)
        {
            OperationStart();
        }
 
        private void OperationStart()
        {
            var processor = new FileProcessor();
            processor.Progress += ProcessorProgress;
 
            var thread = new Thread(processor.DoWork);
            thread.Start();
        }
 
        void ProcessorProgress(int progress)
        {
            if (progressBar1.InvokeRequired)
                progressBar1.Invoke((ProgressHandler)ProcessorProgress, progress);
            else
                progressBar1.Value = progress;
        }
    }

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
    public delegate void ProgressHandler(int progress);
 
    public class FileProcessor
    {
        public event ProgressHandler Progress;
 
        public void DoWork()
        {
            const int IterationsQuantity = 100000;
            for (int i = 1; i <= IterationsQuantity; i++)
            {
                int t = (i / (IterationsQuantity)) * 100;
                if (t > 1)
                { //Тут поставь точку останова
                }
                Progress(t); 
            }
        }
    }

Добавлено через 4 минуты
И ещё, если не вызывать метод Progress каждую итерацию, то цикл выполняется моментально, если вызывать, то можно уснуть пока он закончит своё выполнение.

Добавлено через 51 секунду
Вот сравни и глянь на увеличенное кол-во итераций:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
    public delegate void ProgressHandler(int progress);
 
    public class FileProcessor
    {
        public event ProgressHandler Progress;
 
        public void DoWork()
        {
            const int IterationsQuantity = 10000000;
            int prevPercent = 0;
 
            for (int i = 1; i <= IterationsQuantity; i++)
            {
                int currentPercentage = (i / (IterationsQuantity)) * 100;
                if (prevPercent != currentPercentage)
                {
                    Progress(currentPercentage);
                    prevPercent = currentPercentage;
                }
            }
        }
    }

@aridscrut · 07.04.2012, 20:34 **[ТС]**

Этот код и так пашет норм. Или ты хочешь, чтоб я сравнил рабочий и нерабочий примеры?

@Casper-SC · 07.04.2012, 20:36

Вот держи:

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
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            Shown += new EventHandler(Form1_Shown);
        }
 
        void Form1_Shown(object sender, EventArgs e)
        {
            OperationStart();
        }
 
        private void OperationStart()
        {
            var processor = new FileProcessor();
            processor.Progress += ProcessorProgress;
 
            var thread = new Thread(processor.DoWork);
            thread.Start();
        }
 
        void ProcessorProgress(int progress)
        {
            if (progressBar1.InvokeRequired)
                progressBar1.Invoke((ProgressHandler)ProcessorProgress, progress);
            else
                progressBar1.Value = progress;
        }
    }

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
    public delegate void ProgressHandler(int progress);
 
    public class FileProcessor
    {
        public event ProgressHandler Progress;
 
        public void DoWork()
        {
            const float IterationsQuantity = 1000000000F;
            int prevPercent = 0;
 
            for (int i = 1; i <= IterationsQuantity; i++)
            {
                double d = i / IterationsQuantity;
                int currentPercentage = (int)((i / IterationsQuantity) * 100);
                if (prevPercent != currentPercentage)
                {
                    Progress(currentPercentage);
                    prevPercent = currentPercentage;
                }
            }
        }
    }

Добавлено через 2 минуты

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

Этот код и так пашет норм.

Незнаю что там сравниваешь, но вызывать метод Progress каждую итерацию, это просто говнокод (который ещё и замедляет скорость работы всего цикла).

@aridscrut · 07.04.2012, 20:39 **[ТС]**

Я имею ввиду, что ты перепутал коды! Ты показываешь на примере, где всё работает как надо

Добавлено через 34 секунды
В общем, суть ясна, ща я перекурю, всё опробую и отпишусь

@Casper-SC · 07.04.2012, 20:41

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

Я имею ввиду, что ты перепутал коды! Ты показываешь на примере, где всё работает как надо

Не работает там как надо, если считать проценты так как ты считал. А если считать так как я считаю в последнем примере, то всё работает. Тебе нужно просто считать также как у меня считается процент и инициировать событие Progress, только при изменении процента, иначе искуственно значительно увеличиваешь время выполнения кода в цикле.

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

@aridscrut · 07.04.2012, 21:07 **[ТС]**

Ну что ж, проблема не решена!

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
                p = (int)baseinfo.Length;
                while ((sText = st.ReadLine()) != null)
                {
                    
                    
                    i++;
                    int currentPercentage = (int)((i / p) * 100);
                  
                   
                    if (prevPercent != currentPercentage)
                    {
                        Progress(currentPercentage);
                        prevPercent = currentPercentage;
                        
                    }
                }

Теперь прогрессБар отказывается вообще подавать признаки жизни. Т.е. даже после выполнения цикла не показывает сразу полную загрузку

@Casper-SC · 07.04.2012, 21:11

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

Ну что ж, проблема не решена!

Проблема помоему как раз решена. Читать внимательнее надо.

C#
1
2
3
            const float IterationsQuantity = 1000000000F;
            //-------------
                int currentPercentage = (int)((i / IterationsQuantity) * 100);

Вроде так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
                p = (float)baseinfo.Length; //приводим к типу float
                while ((sText = st.ReadLine()) != null)
                {
                    i++;
                    int currentPercentage = (int)((i / p) * 100);
 
                    if (prevPercent != currentPercentage)
                    {
                        Progress(currentPercentage);
                        prevPercent = currentPercentage;
                        
                    }
                }

@aridscrut · 07.04.2012, 21:11 **[ТС]**

на этом примере всё работает отлично

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 public void DoWork()
        {
            const float IterationsQuantity = 1000000000F;
            int prevPercent = 0;
 
            for (int i = 1; i <= IterationsQuantity; i++)
            {
               
                int currentPercentage = (int)((i / IterationsQuantity) * 100);
                if (prevPercent != currentPercentage)
                {
                    Progress(currentPercentage);
                    prevPercent = currentPercentage;
                }
            }
        }

@Casper-SC · 07.04.2012, 21:14

Я выше показал решение, может ты просто не заметил...

Добавлено через 57 секунд
Просто, если делишь целое на целое, то и получаешь целое, а если на вещественное, то получаешь тоже вещественное.

@aridscrut · 07.04.2012, 21:18 **[ТС]**

Хм... да с float'ом всё начало шуршать потихоньку, только не совсем так как надо, он после 1 шага дальше не движется, ну с этим разберемся. А почему с int не работало?

Добавлено через 2 минуты
Всё, спасибо большое. Общий смысл ясен. Пошел далее работать=)

Новые блоги и статьи Все статьи Все блоги /
Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .	Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем	10 пpимет, которые всегда сбываются Maks 31.03.2026 1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .	Перемещение выделенных строк ТЧ из одного документа в другой Maks 31.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .
Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .	Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .	Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .

@aridscrut 2 / 2 / 0 Регистрация: 16.02.2012 Сообщений: 12
	07.04.2012, 20:34 [ТС]
	Этот код и так пашет норм. Или ты хочешь, чтоб я сравнил рабочий и нерабочий примеры? 0

@aridscrut 2 / 2 / 0 Регистрация: 16.02.2012 Сообщений: 12
	07.04.2012, 20:39 [ТС]
	Я имею ввиду, что ты перепутал коды! Ты показываешь на примере, где всё работает как надо Добавлено через 34 секунды В общем, суть ясна, ща я перекурю, всё опробую и отпишусь 0

@Casper-SC 4434 / 2094 / 404 Регистрация: 27.03.2010 Сообщений: 5,657 Записей в блоге: 1
	07.04.2012, 21:14
	Я выше показал решение, может ты просто не заметил... Добавлено через 57 секунд Просто, если делишь целое на целое, то и получаешь целое, а если на вещественное, то получаешь тоже вещественное. 1

@aridscrut 2 / 2 / 0 Регистрация: 16.02.2012 Сообщений: 12
	07.04.2012, 21:18 [ТС]
	Хм... да с float'ом всё начало шуршать потихоньку, только не совсем так как надо, он после 1 шага дальше не движется, ну с этим разберемся. А почему с int не работало? Добавлено через 2 минуты Всё, спасибо большое. Общий смысл ясен. Пошел далее работать=) 0