0

Добрый день. Запутался с многопоточностью. Пишу приложение на winforms.По нажатию кнопки запускается метод, который выполняет длительную работу. По окончании работы метода хочу вывести сообщение что готово. Однако сообщение выпадает сразу.

Код обработчика:

  private void button1_Click(object sender, EventArgs e)
    {
       openFileDialog1 = new OpenFileDialog();
       openFileDialog1.InitialDirectory=System.IO.Path.GetFullPath("С:\\");
       openFileDialog1.Filter = "Log Files |*.log";
       if (openFileDialog1.ShowDialog() == DialogResult.OK)
        {
            DestinationPath = openFileDialog1.FileName;
        }
        else return;
        Action action = () => new  ParcerVMManager(DestinationPath).StartParcer();
        Task task1 = new Task(action);
        task1.Start();
         task1.Wait();
     MessageBox.Show("Запись в БД окончена");
   }

Метод StartParcer тоже асинхронный.

 public async void StartParcer()
    {
        using (StreamReader fs = new StreamReader(destinationPath))
        {
            while (true)
            {
               string temp = fs.ReadLine();
               if (temp == null) break;
               await Task.Run(() =>LogStringHandler(temp));
            }
        }
        foreach (var item in ParcedData)
        {
            if(! repo.CheckDuplicate(new LogDataVMBuilder().GetModel(item))) continue;
            else
            {
              item.UnloadedFile.Title = GetTitle(item);
              await  GetNameByIP(item);
            }
        }
         await WriteToDB();

        return;
    }

Может быть такая ситуация происходит из-за того что я намешал в одном методе и async/await и Task ? Но как быть в таком случае, когда мне нужно асинхронно выполнить методы GetNameByIP(item); await WriteToDB();?

5
  • 1
    Зачем останавливать ui поток?
    – IC_
    12 ноя 2017 в 9:56
  • Зачем вам выполнять их асинхронно если вы все равно хотите приостановить поток? 12 ноя 2017 в 9:57
  • Мне нужно, чтобы сообщение "Запись в БД окончена" выводилась после окончания записи. А у меня она выводится сразу по нажатию кнопки.
    – foxhound
    12 ноя 2017 в 9:58
  • Или не останавливайте UI поток, или откажитесь от асинхронности. 12 ноя 2017 в 9:58
  • @ Pavel Mayorov То есть, чтобы получит сообщение об окончании записи вовремя, мне нужно синхронно вызвать StartParcer(), а уже внутри него асинхронно вызывать GetNameByIP(item); await WriteToDB(); и др?
    – foxhound
    12 ноя 2017 в 10:00

1 ответ 1

4

UI-поток приостанавливать нельзя, т. к. на время приостановки UI-потока ваше приложение подвиснет.

Вы должны просто дождаться результата асинхронно:

await new ParcerVMManager(DestinationPath).StartParcer();
MessageBox.Show("Запись в БД окончена");

Для этого метод button1_Click превратите в async: private async void button1_Click(object sender, EventArgs e), а также поменяйте возвращаемый тип StartParcer на Task: public async Task StartParcer().


Если ваш метод StartParcer окажется медленным (а он делает много синхронной работы, например, синхронно читает файл), вынесите его тоже в фоновый поток:

// локальная переменная нужна, если свойство DestinationPath привязано к потоку
var destinationPathLocal = DestinationPath;
await Task.Run(() => new ParcerVMManager(destinationPathLocal).StartParcer());

Ваша проблема на самом деле нетривиальна. Смотрите, ваш action запускает StartParser, но поскольку это асинхронный метод, то action не дожидается его окончания. Далее, ваш Task task1 занимается тем, что запускает action. То есть, Task task1 запускает таск StartParser. Соответственно ваш Wait дожидается лишь запуска StartParser, а не его окончания.

Дополнительное чтение по теме: Stephen Toub, Task.Run vs Task.Factory.StartNew.

4
  • Сейчас попробую. Это не сильно отличается от того, что я пробовал вчера, но попробую.
    – foxhound
    12 ноя 2017 в 10:05
  • StartParcer() очень медленный. Он и файл обрабатывает и из сети данные тянет и в базу пишет. С нормальным (не тестовым файлом) он работать будет минут 10 наверное. Точнее не он сам, но методы, которые он вызывает
    – foxhound
    12 ноя 2017 в 10:12
  • Спасибо. Ваш вариант работает.
    – foxhound
    12 ноя 2017 в 10:21
  • @foxhound: Пожалуйста, рад, если помогло!
    – VladD
    12 ноя 2017 в 11:13

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.