Cosa dovrebbe fare un metodo asincrono se un'attività viene eseguita condizionatamente?

Question

Supponiamo che abbia un metodo che attende un'attività. Questo metodo restituisce anche un'attività. Per esempio:

public async virtual Task Save(String path) 
{ 
    if (NewWords.Any()) 
    { 
     await FileManager.WriteDictionary(path, NewWords, true); 
    } 
    else await Task.Run(() => { }); 
} 

È il

else await Task.Run(() => { }); 

necessario qui o sono libero di lasciarlo? C'è qualche differenza se è presente/assente? Forse c'è qualche altro approccio a questo che dovrei prendere?

Answer 1

È peggio di inutile, mentre stai facendo girare una discussione per non fare nulla e poi aspettare fino a quando non ha finito di fare nulla.

Il modo più semplice per non fare nulla è non fare nulla. In un metodo async il metodo sarà ancora sono tornati un Task, ma che Task sarà completato già, in modo da qualcosa await ing che più in alto si arriva direttamente sulla prossima cosa che deve fare:

public async virtual Task Save(String path) 
{ 
    if (NewWords.Any()) 
    { 
     await FileManager.WriteDictionary(path, NewWords, true); 
    } 
} 

(Inoltre, sarebbe più in linea con la convenzione se SaveAsync e WriteDictionaryAsync fossero i nomi dei metodi qui). Se non si utilizza async (e non c'è bisogno di qui, ma capisco che è un esempio) utilizzare Task.CompletedTask:

public virtual Task Save(String path) 
{ 
    if (NewWords.Any()) 
    { 
     return FileManager.WriteDictionary(path, NewWords, true); 
    } 
    return Task.CompletedTask; 
} 

Se si esegue la codifica a fronte di un quadro prima di 4.6 e, pertanto, non hanno CompletedTask disponibile, Task.Delay(0) è utile come Delay casi particolari il valore 0 per restituire un'attività completata cache (in realtà, la stessa che CompletedTask rendimenti):

public virtual Task Save(String path) 
{ 
    if (NewWords.Any()) 
    { 
     return FileManager.WriteDictionary(path, NewWords, true); 
    } 
    return Task.Delay(0); 
} 

Ma il modo 4.6 è più chiaro come i vostri inten t, piuttosto che dipendere da un capriccio di implementazione.

Answer 2

Non è necessario. async è necessario solo se si utilizza almeno uno await. Tutto all'interno del metodo viene eseguito in modo sincrono ad eccezione di await e cosa fa dopo.