异步签名方法执行同步逻辑时返回Task.CompletedTask是否正确？

这种做法不是错误，是.NET生态中非常常见的合法实现方式，完全符合TAP（基于任务的异步模式）的规范要求。

虽然本身没有问题，但有几个容易踩的坑需要规避：

• 不要在这种同步实现里直接抛出未处理的异常：异步签名的方法如果直接抛异常，调用方用await接收时符合预期，但如果调用方没有立刻await、而是先把返回的Task存到别处后再等待，异常会在调用await的时候才抛出，和纯同步方法的异常抛出时机有差异。如果要适配这种场景，建议把异常封装到Task里返回，可以用Task.FromException(Exception)实现。
• 避免执行耗时过长的同步逻辑：如果你的同步逻辑要跑几十毫秒甚至更久，调用方默认会认为这个返回Task的方法会立刻返回，调度到线程池执行后续逻辑时可能会出现预期外的线程阻塞。如果逻辑确实耗时高，建议用Task.Run()把同步逻辑包起来返回，虽然会多一次线程调度，但符合调用方对异步方法的行为预期。
• 不需要强行加async关键字：很多人会下意识给方法加async然后直接跑同步代码，这样反而会生成多余的状态机开销，直接返回Task.CompletedTask的性能更好，是更推荐的写法。

如果接口的同步实现占比很高，也可以考虑拆分接口，分别提供同步和异步版本的方法，或者用默认接口实现给同步版本做适配，不过这个需要根据你的业务场景判断，并不是必须的。

内容的提问来源于stack exchange，提问作者Ahmed Fawzy