Почему не работает асинхронность?

Question

[Dyikot]

Если делать в одном потоке или запустить в другом с помощью new Thread(...).Start() то все работает.

Отдельный поток

private void DownloadDataAndAddToTable()
        {
            new Thread(() =>
            {
                Invoke(() =>
                {
                    tablesForDataSave = Serialization.DownloadTable(filePath);

                    if (tablesForDataSave.Count != 0)
                    {
                        int i = 1;
                        foreach (var table in tablesForDataSave)
                        {
                            tablesForDataShow.Add(
                                table.ToDataGridView("table" + (i++).ToString()));
                            Controls.Add(tablesForDataShow[i - 2]);
                        }
                    }
                    else tablesForDataShow.Add(defaultTable);

                    MakeTableActive(1);
                });
            }).Start();
        }

Асинхронно

private async void DownloadDataAndAddToTable()
        {
            await new Task(() =>
            {
                tablesForDataSave = Serialization.DownloadTable(filePath);

                if (tablesForDataSave.Count != 0)
                {
                    int i = 1;
                    foreach (var table in tablesForDataSave)
                    {
                        tablesForDataShow.Add(
                            table.ToDataGridView("table" + (i++).ToString()));
                        Controls.Add(tablesForDataShow[i - 2]);
                    }
                }
                else tablesForDataShow.Add(defaultTable);

                MakeTableActive(1);
            });
}

Вызывается тут

private void MainForm_Load(object sender, EventArgs e)
        {
            DownloadDataAndAddToTable();
            InitEvents();
        }

В этом методе я скачиваю данные с файла и потом добавляю в DataGridView(Предполагается что их будет несколько). Но при использовании асинронности DataGridView нету.
Что вообще тут лучше использовать асинхронность или многопоточнсть?

Comments

[freeExec]

А почему в одном месте Invoke есть, в другом нет?

[Dyikot]

freeExec, я добавил и ничего не поменялось

[pkrich]

Я не увидел Invoke в асинхронном варианте. То, что касается пользовательского интерфейса, должно происходить в потоке UI. То есть сначала загружаем в отдельном потоке данные из файла, создаём новые элементы интерфейса пользователя и потом через Invoke добавляем созданные элементы в Controls в потоке UI. Примерно так. А первый вариант с Thread фактически сразу из нового потока вызывает Invoke и управление сразу передаётся в основной поток приложения (UI). С таким же успехом можно было бы сразу всё выполнить безо всяких Thread и Task.
Что касается последнего вопроса, то лучше использовать асинхронность, это гораздо экономичнее с точки зрения системных ресурсов.

[Вячеслав Золотов]

Dyikot, скорее всего из-за async void и вызова без await.

[Dyikot]

pkrich,

Вот Invoke

await new Task(() =>
            {
                Invoke(() =>
                {
                    tablesForSavingData = Serialization.DownloadTable(filePath);

                    if (tablesForSavingData.Count != 0)
                    {
                        int i = 1;
                        foreach (var table in tablesForSavingData)
                        {
                            tablesForViewData.Add(
                                table.ToDataGridView("table" + (i++).ToString()));
                            Controls.Add(tablesForViewData[i - 2]);
                        }
                    }
                    else tablesForViewData.Add(defaultTable);

                    MakeTableActive(1);
                });
            });

[pkrich]

Dyikot, это всё равно сразу же возвращает нас в поток UI. Имеет смысл что то типа

private async Task DownloadDataAndAddToTable()
        {

            tablesForSavingData = await Serialization.DownloadTable(filePath);

            if (tablesForSavingData.Count != 0)
            {
                int i = 1;
                foreach (var table in tablesForSavingData)
                {
                    tablesForViewData.Add(
                         table.ToDataGridView("table" + (i++).ToString()));

                }
            }
            else tablesForViewData.Add(defaultTable);

            Invoke(() =>
            {
                foreach (var table in tablesForViewData)
                {
                    Controls.Add(table);
                    MakeTableActive(table);
                }
            });
        }

Ну и DownloadTable тоже должно быть Task

[Dyikot]

pkrich, у вас отсутсвуют async/await

[pkrich]

Dyikot,

private async Task DownloadDataAndAddToTable()
{

tablesForSavingData = await Serialization.DownloadTable(filePath);...

[Dyikot]

pkrich, Получается так

tablesForSavingData = await Task.Run(()=> Serialization.DownloadTable(filePath));

Получается он все равно ждет пока выполнится Task и потом идет дальше, и метод получается выполняется в главном потоке? но мне нужно чтобы весь метод был в другом потоке.

Если опять обернуть все в методе

await Task.Run(() =>
            {
                tablesForSavingData = Serialization.DownloadTable(filePath);

                if (tablesForSavingData.Count != 0)
                {
                    int i = 1;
                    foreach (var table in tablesForSavingData)
                    {
                        tablesForViewData.Add(
                            table.ToDataGridView("table" + (i++).ToString()));
                    }
                }
                else tablesForViewData.Add(defaultTable);

                MakeTableActive(1);

                Invoke(() =>
                {
                    foreach (var table in tablesForViewData)
                        Controls.Add(table);
                });
            });

то все работает асинхронно вроде.(я первоначально забыл run вызвать).
P.S. Как еще понять когда нужен Invoke, если я создал новые объекты в этом (новом) потоке и методы которые с ними взаимодействуют созданы в ином?

[pkrich]

Dyikot, Да, выглядит хорошо)

Answer 1

[Петр]

Для Windows Form нужно использовать SynchronizationContext.
Вызов таска в отдельном потоке Task.Run(...).

В отдельном потоке должно делаться все, что не касается обновления UI контролов. Как только все данные для отображения подготовлены, вызываете synchContext.Post(...) для обновления UI

Answer 2

[Вячеслав Золотов]

Если делать в одном потоке или запустить в другом с помощью new Thread(...).Start() то все работает.

А потом вы оборачиваете всё в Invoke. Естественно это работает. Только о многопоточности тут речи быть не может, ну и накладные расходы сначала на запуск в треде. потом на invoke. Работает оно так же как без треда, потому и показывает.

Если вы то что в таске обернёте в invoke оно тоже заработает, только так тоже неправильно.

Что вообще тут лучше использовать асинхронность или многопоточнсть?

Что-то мне подсказывает что ни то и ни то. Хз что такое Serialization.DownloadTable, но если там есть асинхронные методы, лучше их дёргать. Ну а так, подгрузка данных порциями всё решает.

Ну и теорию подтянуть надо, и практику.

Comments

[Dyikot]

Теперь это будет в отдельном потоке работать?

code

new Thread(() =>
            {
                tablesForSavingData = Serialization.DownloadTable(filePath);

                if (tablesForSavingData.Count != 0)
                {
                    int i = 1;
                    foreach (var table in tablesForSavingData)
                    {
                        tablesForViewData.Add(
                            table.ToDataGridView("table" + (i++).ToString()));
                    }
                }
                else tablesForViewData.Add(defaultTable);

                MakeTableActive(1);

                Invoke(() =>
                {
                    foreach (var table in tablesForViewData)
                        Controls.Add(table);
                });
            }).Start();

В Serialization.DownloadTable нету асинхронных методов.
Думаю в худшем случае в файле будет данных на таблиц 10 по 10x100. Надо получается скачивать асинхронно или это мало данных и не стоит того?

Answer 3

[reberey172]

Попробуйте в task не оборачивать, а поставить await Task.Delay(0). Если будет работать, то хорошо, но вы должны понимать, что это не решение вопроса. Это скорее маскировка проблемы.

private async void DownloadDataAndAddToTable() {
  await Task.Delay(0);
  tablesForDataSave = Serialization.DownloadTable(filePath);
  if (tablesForDataSave.Count != 0) {
    int i = 1;
    foreach (var table in tablesForDataSave) {
      tablesForDataShow.Add(
      table.ToDataGridView("table" + (i++).ToString()));
      Controls.Add(tablesForDataShow[i - 2]);
     }
  }
  else tablesForDataShow.Add(defaultTable);
   MakeTableActive(1);
}