С# Taks и Invoke почему то блокируется форма?

Question

[Pantuchi]

Всем привет. Столкнулся с весьма странной проблемой. А именно блокировкой формы даже когда используется Invoke. Имеется условно форма с прогресс баром и две таски с вычисления мира. Каждая таска запускается со своей кнопки. Внутри задачи через Invoke вызывается прогресс бар и его счетчик инкрементируется. Условно с первой таски форма не блокируется после клика в любую область все работает как и задумывалось, а вот со второй таски после клика в произвольную область форма вешается но прогресс бар идет дальше до завершения таски и после завершения обрабатывается этот клик как будто то действие встало в очередь. Все классы и методы перепроверил ни что не должно было блокировать форму.

Comments

[Петр]

Показали бы пример кода как task вызываете.

Answer 1

[twobomb]

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

Comments

[Pantuchi]

Решил вопрос установив временной интервал вызова Invoke допустим через 15-30-50мс

[twobomb]

Pantuchi, изменяйте прогресс бар тогда, когда он изменился, не нужно колхозить всякие интервалы

[Борис Животное]

Pantuchi, звучит как костыль. Покажи код. Решим проблему, если он читаемый :)

[Pantuchi]

Борис Животное,
private Task NetworkEfficiency()
{
return new Task(() =>
{
try
{
Console.WriteLine("NetworkEfficiency : Start");

List result = new List();
List mix = new List();

if(cuurModeNumber)
result = Enumerable.Range(0, 10).Select(x => new NeuralNetwork.ResultEffect() { labl = x }).ToList();
else
result = Enumerable.Range(0, NeuralNetwork.alf.Length).Select(x => new NeuralNetwork.ResultEffect() { labl = x }).ToList();

if (rb_numbers.Checked)
mix = MixArray(mNIST.digitImages.ToList(), mNIST.imagesNumbersPackeg.myImages);
else
mix = mix = MixArray(mNIST.imagesLettersPackeg.myImages);

Invoke((MethodInvoker)(()=>
{
progressBar1.Value = 0;
progressBar1.Minimum = 0;
progressBar1.Maximum = mix.Count();
}));

Stopwatch time = new Stopwatch();
time.Start();
for (int i = 0; i < mix.Count; i++)
{
result = NeuralNetwork.EfficiencyEvaluation(mix[i].pixels,
mix[i].width, mix[i].height,
mix[i].label, result);
edit = true;

int pers = (int)((i / (float)mix.Count) * 100f);
string str = $"{i} / {mix.Count} : ({pers}%)";

if(time.Elapsed.Milliseconds > 50)
{
Invoke((MethodInvoker)(() =>
{
progressBar1.Value = i;
labelCountMNIST.Text = str;
}));
time.Reset();
time.Start();
}
}

int sum_true = 0;
int count = 0;

foreach (var item in result)
{
count += item.False + item.True;
sum_true += item.True;
}

double percent_true = ((double)sum_true / count) * 100;

MessageBox.Show($"Успешных распознаваний: {percent_true.ToString("#.##")}%", "Результат");

Invoke((MethodInvoker)(() => { progressBar1.Value = 0; }));
Invoke((MethodInvoker)(() => { labelCountMNIST.Text = $"{0} / {0}"; }));

for (int i = 0; i < result[0].neurons.Length; i++)
{
Invoke((MethodInvoker)(()=>
{
(flowLayoutPanel1.Controls[i] as OutNeuron).Value = result[0].neurons[i].Value;
}));
}

Console.WriteLine("NetworkEfficiency : End");
}
catch (Exception ex)
{
MessageBox.Show(ex.Message, "Сообщение");
}
});
}

[Борис Животное]

Pantuchi, оберни код в теги. Только не этот, а снова из проекта скопируй. Из этого кода, по-моему, удалились некоторые символы, такие как угловые скобки и он стал нерабочим. А лучше скинь весь проект, если это не какой-то там огромный важный проект, который нельзя выкладывать. А я пока гляну код так, с ошибками в синтаксисе.

И где код клика? Покажи его тоже, как ты там что запускаешь.

[Борис Животное]

Pantuchi, немного не в тему, но такое

Stopwatch time = new Stopwatch();
time.Start();

Можно заменить на такое
var time = Stopwatch.StartNew();

------------------
Task можно запускать сразу

private Task NetworkEfficiency()
{
    return Task.Run(() => { /* Код*/ });
}

[Борис Животное]

Pantuchi, интересует почему тебе на каждый клик нужно запускать Task, который одновременно с другими Task(ами) пишет что-то в UI? Ведь то, что они пишут конфликтует между собой.

[Борис Животное]

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

[Pantuchi]

Борис Животное, у меня таски лежат в массиве, и пока есть рабочая такска я не запускаю другие такски. Можно конечно с флагом сделать, но вот почему то решил так)

Answer 2

[Борис Животное]

Посмотри как я реализовал класс Analyzer, напиши свой алгоритм по такому же принципу. Внимательно прочитай комментарии в коде класса. Почитай про события в C#. Напиши отдельный класс с алгоритмом, который генерирует события, а уже в форме подпишись на события и обновляй UI. Сообщения из алгоритма отправляй через контекст синхронизации, в алгоритме не дожидайся завершения работы обработчика события (метод Post у контекста синхронизации потоков). Не пиши весь код в методах обработчиках событий форм (окон).

В классе Analyzer
await Task.Delay(250);
просто для имитации долгой работы.

Код ниже слегка отличается от кода по ссылке.

using System;
using System.Threading;
using System.Threading.Tasks;

namespace ProgressBarExample
{
    internal class Analyzer
    {
        private readonly SynchronizationContext _synchronizationContext;

        public Analyzer()
        {
            // Если экземпляр класса будет создан в UI потоке,
            // то здесь будет контекст синхронизации UI потока, иначе пула потоков
            _synchronizationContext = SynchronizationContext.Current ?? new SynchronizationContext();
        }

        public event EventHandler<AnalyzerEventArgs> ProgressChanged;

        public Task<Data> DoWork()
        {
            return Task.Run(async () =>
            {
                for (int i = 0; i < 100; i++)
                {
                    // Имитация долгой работы
                    await Task.Delay(250);
                    // Здесь ты можешь так же как в своём коде вызывать OnProgressChanged
                    // раз в несколько миллисекунд. В форме в UI потоке без Invoke обрабатывать 
                    // событие, выводя те данные, которые ты поместишь в AnalyzerEventArgs
                    OnProgressChanged(new AnalyzerEventArgs("line " + (i + 1), 100));
                }
                return new Data() { Text = "Данные " };
            });
        }

        private void OnProgressChanged(AnalyzerEventArgs args)
        {
            // Перенаправляем выполнение в UI поток не ожидая пока отработает метод обработчик события.
            _synchronizationContext.Post(state =>
            {
                ProgressChanged?.Invoke(this, (AnalyzerEventArgs)state);
            }, args); // args передаётся в переменную state (грубо говоря)
        }
    }
}

namespace ProgressBarExample
{
    public class AnalyzerEventArgs
    {
        public int MaxLines { get; }

        public string CurrentLine { get; }

        public AnalyzerEventArgs(string currentLine, int maxLines)
        {
            CurrentLine = currentLine;
            MaxLines = maxLines;
        }
    }
}

namespace ProgressBarExample
{
    public class Data
    {
        public string Text { get; set; }
    }
}

По ссылке полный код WPF приложения (не важно какой проект, в Windows Forms всё так же будет работать).
Как сделать, чтобы ProgressBar работал во время нагрузки на приложение?

Comments

[Pantuchi]

Кстати про события это тема, чет не додумался про такой подход