将同步代码包装成异步方法的最佳方法是什么

Question

我正在使用 async-await 方法创建一个应用程序。但是使用它们对我来说存在很大的问题。在阅读了几篇文章后，我仍然不知道将繁重的同步操作包装到异步方法的最佳方法是什么。

我有两个想法。哪个最好？

1) 当前实现。

private Task<List<UploadedTestModel>> ParseTestFiles(List<string> filesContent)
{
    var tcs = new TaskCompletionSource<List<UploadedTestModel>>();
    Task.Run(() =>
    {
        var resultList = new List<UploadedTestModel>();
        foreach (var testBody in filesContent)
        {
            try
            {
                var currentCulture = Thread.CurrentThread.CurrentCulture;
                var serializerSettings = new JsonSerializerSettings
                {
                    Culture = currentCulture
                };

                var parsedData = JsonConvert.DeserializeObject<UploadedTestModel>(testBody, serializerSettings);
                resultList.Add(parsedData);
            }
            catch(Exception exception)
            {
                tcs.SetException(exception);
            }
        }
        tcs.SetResult(resultList);
    });
    return tcs.Task;
}

我正在使用 Task.Run 和 TaskCompletionSource

2) 仅使用没有 TaskCompletionSource 的 Task.Run

private Task<List<UploadedTestModel>> ParseTestFiles(List<string> filesContent)
{
    return Task.Run(() =>
    {
       . . . .
       return resultList;          
    });
}

Answer 1

我都不会使用。你会对调用你的方法调用的人撒谎。当你公开一个异步操作时，调用者希望它自然是异步的，这意味着它背后没有线程在工作。

你所有的方法都是天生同步的，你应该这样公开它们。让调用者决定是要同步调用它们还是使用线程并在那里排队，不要为它们决定。

Stephan Toub 写了一篇名为Should I expose asynchronous wrappers for synchronous methods? 的精彩文章，其中谈到了不做你想做的事的所有原因。我建议阅读它。