这是async/await的正确实现吗？

Question

异步上传文件的方法：

public async void UploadScreenshot(DateTime? date = null)
{
    var uploadTask = Task.Factory.StartNew(() => _ftp.UploadFile(_screenshotLocalFile,
                                                                 date.HasValue
            ? _screenshotRemoteFile.Replace("{1}", date.Value.ToString(Helper.StandardTimeFile))
            : _screenshotRemoteFile.Replace("{1}", DateTime.Now.ToString(Helper.StandardTimeFile))));
    await uploadTask;
}

用法：

UploadScreenshot();

这是异步上传文件的有效方法吗？

Answer 1

是的，但是为什么要在方法内部等待呢？

public Task UploadScreenshot(DateTime? Date = null)
{
    var uploadTask = Task.Factory.StartNew(() => 
        _ftp.UploadFile(_screenshotLocalFile, date.HasValue 
                ? _screenshotRemoteFile.Replace("{1}", date.Value.ToString(Helper.StandardTimeFile))
                : _screenshotRemoteFile.Replace("{1}", DateTime.Now.ToString(Helper.StandardTimeFile))));

    return uploadTask;
}

用法：

await UploadScreenshot();

另外，据我所知，异步方法返回void不是一个好的做法，最好返回Task/

Answer 2

好吧，它有几个问题：

1. async void应仅用于事件处理程序。在所有其他情况下，您应该使用async Task (或async Task<T>)，这样您就可以异步等待，直到方法完成，并正确处理其execution.
2. Your方法包装同步UploadFile()方法期间发生的任何错误。如果有异步替代方法(UploadFileAsync())，那么您应该改用它。
3. 您可以不使用await，而是直接返回从StartNew() (或UploadFileAsync()).

)获得的Task，从而提高效率