Задача не работает?

Question

Текущая проблема, с которой я столкнулся, - это когда нажата кнопка, кажется, ничего не происходит. Я не знаю, почему это происходит.

Вот по щелчку метод кнопка будет нажата:

private void computeStart_Click(object sender, EventArgs e) 
{ 
    _computeTokenSource = new CancellationTokenSource(); 
    GenerateAll(_computeTokenSource.Token); 
} 

Вот метод вызывается по щелчку.

private async void GenerateAll(CancellationToken token) 
{ 
    await new Task(() => 
    { 
     var total = (long) Math.Pow(36, 6); 
     var options = new ParallelOptions {CancellationToken = token}; 
     Parallel.For(0, total, options, a => GenerateCodeAndHash()); 
    }, TaskCreationOptions.LongRunning); 
} 

Наконец это метод вызывается в Parallel.For

private void GenerateCodeAndHash() 
{ 
    var result = new string(
     Enumerable.Repeat(Chars, 6) 
      .Select(s => s[new Random().Next(s.Length)]) 
      .ToArray()); 
    if (_dictionary.ContainsKey(result)) return; 
    var hash = MD5.Create().ComputeHash(Encoding.UTF8.GetBytes(result)); 
    var sb = new StringBuilder(); 
    for (var j = 0; j < 2; j++) 
     sb.Append(hash[j].ToString("x2")); 
    _dictionary.TryAdd(result, sb.ToString()); 
} 

Answer 1

Не используйте Task конструктор с async/await.

В этом случае, вы хотите использовать Task.Run:

private async void computeStart_Click(object sender, EventArgs e) 
{ 
    _computeTokenSource = new CancellationTokenSource(); 
    await Task.Run(() => GenerateAll(_computeTokenSource.Token)); 
} 

private void GenerateAll(CancellationToken token) 
{ 
    var total = (long) Math.Pow(36, 6); 
    var options = new ParallelOptions {CancellationToken = token}; 
    Parallel.For(0, total, options, a => GenerateCodeAndHash()); 
} 

Для получения дополнительной информации см мой async/await intro.

Answer 2

Конструктор Task создает Task, который не запущен. Если вы хотите его использовать, вам нужно позвонить по телефону Start(). Но большую часть времени вы хотите создать и запустить Task в то же время, что вы можете сделать, используя Task.Run().

Также, как указано другими, вы должны использовать async void только в обработчиках событий, больше нигде; метод GenerateAll() должен быть async Task, и вы должны await его от вашего обработчика событий, который должен быть async void.