我的方法真的是异步的吗？

Question

我有这样的方法：

public async Task<IEnumerable<Model>> Get([FromUri]IList<string> links)
{
    IList<Model> list = new List<Model>();
    foreach (var link in links)
    {
        MyRequestAsync request = new MyRequestAsync(link);
        list.Add(await request.GetResult());
    }

    return list;
}

但我只是想知道它是否真的是async，因为我认为list.Add(await request.GetResult()); 和return list; 打破了async 方法的性质。

如果我错了，请纠正我，如果我是对的，我该如何解决？

更新：据我了解，我必须这样做 C# 5.0 async await return a list return await Task.Run(() => new List<string>() {"a", "b"}); 但不确定如何将其应用于我的情况。

Answer 1

您的方法是异步的，但可能没有充分利用资源。

您的方法将做的是进入foreach 循环，创建MyRequestAsync 对象，然后（在await 点）它将放弃其线程，直到结果可用。一旦结果可用，将找到一个合适的线程并且该方法将恢复运行。它将结果添加到list 并返回到循环的顶部，并一遍又一遍地重复整个过程。

但是，请考虑这一点 - 如果这些请求是独立的，您可以改为并行发出每个请求，然后仅在 所有 请求完成后继续运行您的方法。就像这样：

public async Task<IEnumerable<Model>> Get([FromUri]IList<string> links)
{
    IList<Task<Model>> list = new List<Task<Model>>();
    foreach (var link in links)
    {
        MyRequestAsync request = new MyRequestAsync(link);
        list.Add(request.GetResult());
    }

    return new List<Model>(await Task.WhenAll(list));
    //Or just
    //return await Task.WhenAll(list);
    //Since we don't need to return a list
}

---

而且，对于愚蠢的观点，您可以将整个方法重写为：

return await Task.WhenAll(from l in links select new RequestAsync(l).GetResult());

但这可能会降低它的可读性。

Answer 2

在我看来，I/O 是async，因此该方法可以称为“真正异步”。
async 意味着 I/O 在等待某事时不会阻塞线程（这里结果），但不是在“做某事”时（此处为 list.Add）。

Answer 3

这有点不可能，因为你调用的任何东西都可能是阻塞操作。如果某处隐藏了阻塞操作，则此方法也会阻塞。如果你想让一个方法成为非阻塞的和/或只使用可扩展的异步 IO，你必须检查你所做的和你调用的所有事情。

也就是说你的代码看起来像是非阻塞的，因为它使用了await（而不是Task.Wait）。稍微简化一下，此方法将在第一个 await 操作上返回，这可能正是您所需要的。