Button.PerformClick () не ждет обработчика асинхронных событий

Похоже, у вас есть некоторые неправильные представления о том, как работают async/await и / или PerformClick(). Чтобы проиллюстрировать проблему, рассмотрим следующий код:

private async Task MyMethodAsync()
{
    await Task.Delay(2000);
    message = "Finished";      // The execution of this line will be delayed by 2 seconds.
}

private void button2_Click(object sender, EventArgs e)
{
    message = "Started";
    MyMethodAsync();           // Since this method is not awaited,
    MessageBox.Show(message);  // the execution of this line will NOT be delayed.
}

Note: the compiler will give us a warning but let's ignore that for the sake of testing.

Итак, что вы ожидаете от MessageBox? Вы, наверное, сказали бы "началось" .¹ Почему? Потому что мы не ждали метода MyMethodAsync(); код внутри этого метода запускает асинхронно, но мы не сделали ждать для его завершения, мы просто перешли к следующей строке, где значение message еще не изменилось.

Если вы до сих пор понимаете такое поведение, остальное должно быть легко. Итак, давайте немного изменим приведенный выше код:

private async void button1_Click(object sender, EventArgs e)
{
    await Task.Delay(2000);
    message = "Finished";      // The execution of this line will be delayed by 2 seconds.
}

private void button2_Click(object sender, EventArgs e)
{
    message = "Started";
    button1_Click(null, null); // Since this "method" is not awaited,
    MessageBox.Show(message);  // the execution of this line will NOT be delayed.
}

Теперь все, что я сделал, это переместил код, который был внутри метода асинхронныйMyMethodAsync(), в обработчик событий асинхронныйbutton1_Click, а затем вызвал этот обработчик событий с помощью button1_Click(null, null). Есть ли разница между этим и первым кодом? Нет, по сути, это одно и то же; в обоих случаях я вызвал асинхронный метод не дожидаясь этого.²

Если вы до сих пор со мной согласны, вы, вероятно, уже понимаете, почему ваш код работает не так, как ожидалось. Приведенный выше код (во втором случае) практически идентичен вашему. Разница в том, что я использовал button1_Click(null, null) вместо button1.PerfomClick(), который, по сути, делает то же самое.

Решение:

Если вы хотите дождаться завершения кода в button1_Click, вам нужно переместить все внутри button1_Click(если это асинхронный код) в метод async, а затем Ждите его в обаbutton1_Click и button2_Click. Это именно то, что вы сделали в своем Раздел "Правка", но имейте в виду, что button2_Click также должен иметь подпись async.

³_{If you thought the answer was something else, then you might want to check this article which explains the warning.}

¹_{The only difference is that in the first case, we could solve the problem by awaiting the method, however, in the second case, we can't do that because the "method" is not awaitable because the return type is voideven though it has an async signature.}

²_{Actually, there are some differences between the two (e.g., the validation logic in PerformClick(), but these differences don't affect the end result in our current situation.}