此修改是否使我的方法异步运行

Question

以下是记录器调用，即阻塞：

public static void Info(Category category, SubLevel subLevel, object message)
{
    CommonLogger?.Info($"{category}, {(int) subLevel}, {message}");
}

我想让它异步执行，因此我做了以下修改，因为根据我的理解，将 Async-Await 用于 void 返回方法，就像当前的那样不是一个好策略：

public static void InfoAsync(Category category, SubLevel subLevel, object message)
{
    Task.Run(() => CommonLogger?.Info($"{category}, {(int)subLevel}, {message}"));
}

代码上下文-

• 使日志记录异步运行
• 一劳永逸，主要业务逻辑分开进行

我的问题是：

1. 我做出此更改是否正确？

2. 还有更好的方法吗？

Answer 1

此修改是否使我的方法异步运行？

不，它没有。 Concurrency and parallelism are two different concepts。人们倾向于使并发和并行可互换，但实际上并非如此。您的代码将与其他执行单元并行运行，但操作本身并不是异步。如果你真的想进行异步操作，你可以使用异步 IO 来写入底层记录器。

我进行此更改是否正确？

我会说不。这称为async over sync 反模式（有关其含义的详细说明，请参阅链接）。我会避免这种模式，因为它会产生一种错误的异步感。你的方法并不是真正的自然异步，true asynchronous methods don't need to use extra thread-pool threads。

还有更好的方法吗？

是的，不要这样做。让该方法的调用者显式调用Task.Run，以便他们知道所述委托将在线程池上执行。或者更好的是，将 async IO 与记录器实现一起使用。毕竟，我假设这可能会写入底层文件或执行某种网络操作以通过网络发送日志。如果是这样，请改用真正的异步 API。

Answer 2

是的，这现在是一种异步方法，但是，其他两个不是正确的问题（正如 JayMee 所提到的）。它们纯粹是人们发展的方式。例如，我会这样做，但 Joe Bloggs 可能不会。

希望这会有所帮助。

Answer 3

1. 是的，现在 异步运行（在某种意义上，您的方法在不等待日志记录完成的情况下返回，而不是 Yuval Itzchakov 解释的含义）。

2. 那么，您为什么不想更改该方法以返回 Task 并使其也可供调用者等待：

   public static async Task InfoAsync(Category category, SubLevel subLevel, object message)
   {
       await Task.Run(() => CommonLogger?.Info($"{category}, {(int)subLevel}, {message}"));
   }
   

但正如您和其他评论者指出的那样：如果您不需要这个，它只会增加不必要的复杂性（例如，编译器会构建一个不必要的状态机）。

一般来说，要回答“能否做得更好”之类的问题，需要了解“更好”的标准。这取决于如何以及在哪些条件下会调用它以及要求是什么。