c# - 使用 Task.Run 是一种不好的做法吗？

Question

一般在类上实现异步方法时，我会这样写:

public Task<Guid> GetMyObjectIdAsync(string objectName)
{
    return Task.Run(() => GetMyObjectId(objectName));
}

private Guid GetMyObjectId(string objectName)
{
    using (var unitOfWork = _myUnitOfWorkFactory.CreateUnitOfWork())
    {
        var myObject = unitOfWork.MyObjects.Single(o => o.Name == objectName);
        return myObject.Id;
    }
}

这种模式允许我根据情况同步和异步使用相同的逻辑(我的大部分工作都在旧代码库中，还没有很多支持异步调用)，因为我可以公开同步方法公开并在需要时获得最大的兼容性。

最近我读了一些 SO 帖子，建议使用 Task.Run() 是个坏主意，应该只在特定情况下使用，但这些情况似乎不太清楚。

我上面描述的模式实际上是个坏主意吗？这样做我会失去异步调用的一些功能/预期目的吗？或者这是一个合法的实现？

Answer 1

您正在做的是将同步操作卸载到另一个线程。如果您的线程是“特殊的”，那很好。 “特殊”线程的一个示例是 UI 线程。在这种情况下，您可能希望卸载它的工作以保持 UI 响应(另一个示例是某种监听器)。

然而，在大多数情况下，您只是将工作从一个线程转移到另一个线程。这不会增加任何值(value)，而且会增加不必要的开销。

所以:

Is the pattern I've depicted above actually a bad idea?

是的，是的。将同步工作卸载到 ThreadPool 并假装它是异步的并不是一个好主意。

Am I losing some of the functionality/ intended purpose of async calls doing it this way?

这个操作实际上并没有什么异步的。如果您在远程机器上执行此操作并且您可以从异步执行中受益，则操作本身需要真正异步，这意味着:

var myObject = await unitOfWork.MyObjects.SingleAsync(o => o.Name == objectName);

您目前正在做的事情称为“同步之上的异步”，您可能不应该这样做。更多内容 Should I expose asynchronous wrappers for synchronous methods?