在 .Select lambda 中使用 async / await

Question

我正在使用 Asp.Net Core Identity 并尝试简化一些将用户列表及其角色投影到 ViewModel 的代码。这段代码有效，但为了简化它，我陷入了错误和好奇的疯狂漩涡。

这是我的工作代码：

        var allUsers = _userManager.Users.OrderBy(x => x.FirstName);
        var usersViewModel = new List<UsersViewModel>();

        foreach (var user in allUsers)
        {
            var tempVm = new UsersViewModel()
            {
                Id = user.Id,
                UserName = user.UserName,
                FirstName = user.FirstName,
                LastName = user.LastName,
                DisplayName = user.DisplayName,
                Email = user.Email,
                Enabled = user.Enabled,
                Roles = String.Join(", ", await _userManager.GetRolesAsync(user))
            };

            usersViewModel.Add(tempVm);
        }

在尝试简化代码时，我想我可以做这样的事情 （损坏的代码）：

        var usersViewModel = allUsers.Select(user => new UsersViewModel
        {
            Id = user.Id,
            UserName = user.UserName,
            FirstName = user.FirstName,
            LastName = user.LastName,
            DisplayName = user.DisplayName,
            Email = user.Email,
            Enabled = user.Enabled,
            Roles = string.Join(", ", await _userManager.GetRolesAsync(user))
        }).ToList();

这会中断，因为我没有在 user 之前的 lambda 表达式中使用 async 关键字。但是，当我在 user 之前添加异步时，我收到另一个错误，提示 “异步 lambda 表达式无法转换为表达式树”

我的猜测是 GetRolesAsync() 方法返回一个任务并将其分配给角色，而不是该任务的实际结果。我一生似乎无法弄清楚如何让它发挥作用。

过去一天我研究并尝试了很多方法，但没有运气。以下是我看过的一些：

Is it possible to call an awaitable method in a non async method?

https://blogs.msdn.microsoft.com/pfxteam/2012/04/12/asyncawait-faq/

Calling async method in IEnumerable.Select

How to await a list of tasks asynchronously using LINQ?

how to user async/await inside a lambda

How to use async within a lambda which returns a collection

诚然，我并不完全理解 async / await 是如何工作的，所以这可能是问题的一部分。我的 foreach 代码有效，但我希望能够了解如何使其以我尝试的方式工作。因为我已经花了很多时间在上面，所以我认为这是一个很好的第一个问题。

谢谢！

编辑

我想我必须解释我在我研究的每篇文章中所做的事情，以免被标记为重复问题 - 我非常努力地避免这种情况:-/。虽然这个问题听起来很相似，但结果却并非如此。对于被标记为答案的文章，我尝试了以下代码：

    public async Task<ActionResult> Users()
    {
        var allUsers = _userManager.Users.OrderBy(x => x.FirstName);
        var tasks = allUsers.Select(GetUserViewModelAsync).ToList();
        return View(await Task.WhenAll(tasks));
    }

    public async Task<UsersViewModel> GetUserViewModelAsync(ApplicationUser user)
    {
        return new UsersViewModel
        {
            Id = user.Id,
            UserName = user.UserName,
            FirstName = user.FirstName,
            LastName = user.LastName,
            DisplayName = user.DisplayName,
            Email = user.Email,
            Enabled = user.Enabled,
            Roles = String.Join(", ", await _userManager.GetRolesAsync(user))
        };
    }

我也尝试过像这样使用 AsEnumerable：

    var usersViewModel = allUsers.AsEnumerable().Select(async user => new UsersViewModel
        {
            Id = user.Id,
            UserName = user.UserName,
            FirstName = user.FirstName,
            LastName = user.LastName,
            DisplayName = user.DisplayName,
            Email = user.Email,
            Enabled = user.Enabled,
            Roles = string.Join(", ", await _userManager.GetRolesAsync(user))
        }).ToList();

这两个都会产生错误消息：“InvalidOperationException：在前一个操作完成之前在此上下文上启动了第二个操作。不保证任何实例成员都是线程安全的。”

在这一点上，我原来的 ForEach 似乎是最好的选择，但我仍然想知道如果我使用异步方法来做这件事的正确方法是什么。

编辑 2 - 带答案 感谢 Tseng 的 cmets（和其他一些研究），我能够使用以下代码使事情正常进行：

        var userViewModels = allUsers.Result.Select(async user => new UsersViewModel
        {
            Id = user.Id,
            UserName = user.UserName,
            FirstName = user.FirstName,
            LastName = user.LastName,
            DisplayName = user.DisplayName,
            Email = user.Email,
            Enabled = user.Enabled,
            Roles = string.Join(", ", await _userManager.GetRolesAsync(user))
        });
        var vms = await Task.WhenAll(userViewModels);
        return View(vms.ToList());

虽然现在我已将每个人的 cmets 都考虑在内，但我开始更仔细地查看 SQL Profiler，以了解 DB 实际上获得了多少命中 - 正如 Matt Johnson 所提到的，它很多 (N+1)。

因此，虽然这确实回答了我的问题，但我现在正在重新考虑如何运行查询，并且可能只是将角色放在主视图中，并且仅在选择每个用户时才拉出它们。不过，我确实通过这个问题学到了很多东西（并且学到了更多我不知道的东西），所以谢谢大家。

Answer 1

我认为你在这里混合了两件事。表达式树和代表。 Lambda 可以用来表达这两种情况，但这取决于方法接受的参数类型，它将被转换到哪个参数中。

传递给作为Action<T> 或Func<T, TResult> 的方法的lambda 将被转换为委托（基本上是匿名函数/方法）。

当您将 lambda 表达式传递给接受 Expression<T> 的方法时，您会从 lambda 中创建一个表达式树。表达式树只是描述代码的代码，而不是代码本身。

话虽如此，表达式树无法执行，因为它已转换为可执行代码。您可以在运行时编译表达式树，然后像委托一样执行它。

ORM 框架使用表达式树来允许您编写“代码”，这些“代码”可以翻译成不同的东西（例如数据库查询）或在运行时动态生成代码。

因此，您不能在接受Expression<T> 的方法中使用async。当您将其转换为AsEnumerable() 时它可能会起作用的原因是因为它返回一个IEnumerable<T> 并且其上的LINQ 方法接受Func<T, TResult>。但它本质上是获取整个查询并在内存中完成所有工作，因此您不能使用投影（或者您必须在仅使用表达式和投影之前获取数据），将过滤后的结果转换为列表然后过滤它。

你可以试试这样的：

// Filter, sort, project it into the view model type and get the data as a list
var users = await allUsers.OrderBy(user => user.FirstName)
                             .Select(user => new UsersViewModel
    {
        Id = user.Id,
        UserName = user.UserName,
        FirstName = user.FirstName,
        LastName = user.LastName,
        DisplayName = user.DisplayName,
        Email = user.Email,
        Enabled = user.Enabled
    }).ToListAsync();

// now that we have the data, we iterate though it and 
// fetch the roles
var userViewModels = users.Select(async user => 
{
    user.Roles = string.Join(", ", await _userManager.GetRolesAsync(user))
});

第一部分将完全在数据库上完成，您将保留所有优势（即订单发生在数据库上，因此您不必在获取结果后进行内存排序，并且限制调用会限制数据从数据库等中获取）。

第二部分遍历内存中的结果并为每个临时模型获取数据，最后将其映射到视图模型中。

Answer 2

这是一个解决方案，您可以使用List 作为回报。

var userViewModels = (await allUsers).Select(async user => new UsersViewModel
        {
            Id = user.Id,
            UserName = user.UserName,
            FirstName = user.FirstName,
            LastName = user.LastName,
            DisplayName = user.DisplayName,
            Email = user.Email,
            Enabled = user.Enabled,
            Roles = string.Join(", ", await _userManager.GetRolesAsync(user))
        }).Select(q => q.Result);

更新 1：

感谢@TheodorZoulias，我意识到.Select(q => q.Result) 会导致线程阻塞。 所以我认为在某人找到更好的方法之前使用这个解决方案会更好。 它也可以随机播放项目。

List<UsersViewModel> userViewModels = new();
(await allUsers)
.Select(async user => new UsersViewModel()
{
    //(...)
    Roles = string.Join(", ", await _userManager.GetRolesAsync(user))
})
.ToList()
.ForEach(async q => userViewModels.Add(await q));

Answer 3

扩展方法：

public static async Task<IEnumerable<TDest>> SelectSerialAsync<TSource, TDest>(this IEnumerable<TSource> sourceElements, Func<TSource, Task<TDest>> func)
{
    List<TDest> destElements = new List<TDest>();

    foreach (TSource sourceElement in sourceElements)
    {
        TDest destElement = await func(sourceElement);
        destElements.Add(destElement);
    }

    return destElements;
}

用法：

DestType[] array = (await sourceElements.SelectSerialAsync<SourceType, DestType>(
    async (sourceElement) => { return await SomeAsyncMethodCall(sourceElement); }
)).ToArray();