Тупик при использовании Результат

Question

Я прочитал, что я должен использовать только Result вместо await, когда я абсолютно уверен, что операция завершена. Я не совсем уверен, что происходит внизу, и хотел бы спросить опытных программистов, если это абсолютно безопасное использование await/Result/async.

public static bool Success() 
{ 
    return 0 < Execute("DELETE FROM Table WHERE Id = 12").Result; 
} 

public static async Task<int> Execute(string sql) 
{ 
    using (var con = Connection) 
    { 
     con.Open(); 
     return await con.ExecuteAsync(sql); 
    } 
} 

Answer 1

Это не безопасно. Операция еще не завершена, когда вы используете Task.Result

Когда вы вызываете метод async, он запускается синхронно, пока не достигнет ожидания, а затем возвращается к вызывающей стороне с задачей, которая представляет асинхронную операцию.

Задача не заполнена и использование Task.Result заблокирует вызывающий поток.

Вместо этого вы должны ждать возвращенной задачи или использовать синхронную опцию.

public static Task<bool> SuccessAsync() 
{ 
    return 0 < await ExecuteAsync("DELETE FROM Table WHERE Id = 12"); 
} 

public static async Task<int> ExecuteAsync(string sql) 
{ 
    using (var con = Connection) 
    { 
     con.Open(); 
     return await con.ExecuteAsync(sql); 
    } 
} 

Answer 2

Это безопасно, хотя в конце концов блокирует, пока Execute не будет завершена. Доступ к Task.Result эквивалентно:

Task<int> task = Execute("..."); 
task.Wait(); // blocks until done 
return 0 < task.Result; 

Если вы хотите, чтобы далее await цепь, вы могли бы Success вернуть Task<bool> вместо этого.

Обратите внимание, что существует вероятность блокировки в зависимости от тока SynchronizationContext. Если ваш SynchronizationContext является однопоточным, тогда ваш вызов Task.Result заканчивается блокировкой и ждет завершения Execute, но Execute ждет вас, чтобы выпустить поток, чтобы он мог продолжить. Если вы используете графическое приложение (с использованием однопоточного SynchronizationContext по умолчанию) или ASP.NET, то вам стоит подумать о добавлении ConfigureAwait(false), чтобы вместо этого выполнить поток нитей пула потоков Execute.