C# породит новую задачу

Question

Предположим, у меня есть эта функция:

void DoWork() 
{ 
    //do long work 
} 

Я хочу эту функцию, чтобы быть выполнена в другом потоке. Я привык использовать потоки, так что один из способов выполнить его на нити:

new Thread(DoWork).Start(); 

Я пытаюсь узнать новую функцию task/await C#, так что я модифицировал метод

async Task DoWork() 
{ 
    await Task.Run(delegate() { /*do stuff*/}); 
} 

Теперь я могу использовать это так:

DoWork().Wait(0); 

Или я могу изменить метод снова:

async void DoWork() 
{ 
    await Task.Run(delegate() { /*do stuff*/}); 
} 

А теперь просто позвоните:

DoWork(); 

Какое решение я должен использовать? Мне не нужен результат задания и не дожидаюсь его завершения.

Answer 1

Ответ не является ни тем, ни другим. Вы создаете тривиальную асинхронную оболочку вокруг синхронной логики.

Стивен Клири: using Task.Run for asynchronous wrappers is a code smell

Стивен Toub (Microsoft): Should I expose asynchronous wrappers for synchronous methods? (и нет, вы не должны).

Асинхронная часть вашего метода DoWork не добавляет значения. Это уменьшает ценность вашего API, делая невозможным выполнение вашего потенциально сложного «do stuff» логикой любым способом, кроме как «забыть».

Правильный подход заключается в канаву Task.Run и раздеться ваш метод вниз к самым необходимым:

void DoWork() 
{ 
    // Do long CPU-bound work, synchronously 
} 

... 

// Then, somewhere in the calling code: 
Task.Run(() => DoWork()); // Fire and forget unless you await the resulting Task or block on it. 

Task.Run в настоящее время является предпочтительным методом планирования работы на пуле потоков (как в .NET 4.6.x)