到 Task.Run 还是不到 Task.Run

Question

假设我有一个包含异步方法的接口，并且该接口有两个不同的实现。这两种实现中的一种自然是异步的，而另一种则不是。实现非异步方法的“最正确”方式是什么？

public interface ISomething {
    Task<Foo> DoSomethingAsync();
}

// Normal async implementation
public class Implementation1 : ISomething {
    async Task<Foo> ISomething.DoSomethingAsync() {
        return await DoSomethingElseAsync();
    }
}

// Non-async implementation
public class Implementation2 : ISomething {
    // Should it be:
    async Task<Foo> ISomething.DoSomethingAsync() {
        return await Task.Run(() => DoSomethingElse());
    }
    // Or:
    async Task<Foo> ISomething.DoSomethingAsync() {
        return DoSomethingElse();
    }
}

我尝试跟上 Stephen Cleary 的 blog，我知道这些都没有真正提供任何异步好处，我对此表示满意。第二个对我来说似乎更正确，因为它不会假装它不是，但它确实会发出编译器警告，并且这些加起来会让人分心。

这一切都在 ASP.NET 中（Web MVC 和 WebAPI），如果有区别的话。

Answer 1

您可以完全放弃async 修饰符并使用Task.FromResult 同步返回已完成的任务：

Task<Foo> ISomething.DoSomethingAsync()
{
    return Task.FromResult(DoSomethingElse());
}

这会处理警告并具有更好的性能，因为它不需要async 方法的状态机开销。

但是，这对change the semantics of exception handling 有点作用。如果这是一个问题，那么您应该使用同步 async 方法并接受警告（或通过评论将其关闭）：

#pragma warning disable 1998
    async Task<Foo> ISomething.DoSomethingAsync() 
#pragma warning restore 1998
    {
        return DoSomethingElse();
    }

正如 Stephen Cleary 所建议的，您还可以通过等待已完成的任务来处理该警告（同时保持方法同步）：

async Task<Foo> ISomething.DoSomethingAsync() 
{
    await Task.FromResult(false); // or Task.CompletedTask in .Net 4.6
    return DoSomethingElse();
}

Answer 2

这真的取决于你的方法在做什么：

• 没有 I/O，CPU 工作量可忽略不计
• CPU 密集型工作
• I/O 密集型工作

无 I/O，CPU 工作量可忽略不计

您应该同步计算结果并创建一个保存结果的任务。

Task<Foo> ISomething.DoSomethingAsync() {
    Foo result;
    // do something not hurting performance
    // no I/O here
    return Task.FromResult(result);
}

但是请注意，任何异常都将在调用方法时引发，而不是在等待任务时引发。对于符合其他工作类型的后一种情况，您仍然应该使用 async：

async Task<Foo> ISomething.DoSomethingAsync() {
    Foo result;
    // do something not hurting performance
    // no I/O here
    return result;
}

CPU 密集型工作

您应该使用Task.Run 启动一个任务，并在任务中执行 CPU 密集型工作。

Task<Foo> ISomething.DoSomethingAsync() {
    return Task.Run(() => {
        Foo result;
        // do some CPU intensive work here
        return result;
    });
}

I/O 密集型工作

您应该使用 async 关键字并等待任何异步 I/O 方法。不要使用同步 I/O 方法。

async Task<Foo> ISomething.DoSomethingAsync() {
    Stream s = new ....

    // or any other async I/O operation
    var data = await s.ReadToEndAsync(); 

    return new Foo(data);
}