Promise.all结合map处理数据导致原有排序错乱问题如何解决？

你推测的问题方向完全正确，乱序是Promise.all的并行执行逻辑导致的：Promise.all会同时触发所有User.findById查询，每个查询的返回耗时不固定，先得到结果的查询会先执行arr.push(item)，最终arr的插入顺序和somelist的原始排序没有关联，就会出现偶发乱序。

方案1：保留并行查询效率，按原始顺序整理结果（推荐） #

Promise.all的返回结果数组顺序和传入的异步任务顺序完全一致，我们可以调整写法，直接拿到按原始顺序返回的结果后再做过滤，既保留并行查询的高性能，又能保证顺序稳定：

const somelist = await Model.find({condition blah blah})
                        .sort({created_at: -1})
                        .select('some fields');
if(!somelist){
    res.status(500).json({ success: false });
}else{
    // Promise.all返回结果顺序与somelist原始顺序完全对齐
    const checkResult = await Promise.all(somelist.map(async (item) => {
        const user = await User.findById(item.some);
        return user ? item : null;
    }));
    // 过滤无效项，顺序保持不变
    const filteredArr = checkResult.filter(Boolean);

    console.log(filteredArr)
    res.status(200).json(filteredArr);
}

方案2：串行执行查询（仅适合极小数据量场景） #

如果数据量很小，也可以选择串行逐个执行查询，天然保证顺序，但执行效率会比并行方案低很多：

const somelist = await Model.find({condition blah blah})
                        .sort({created_at: -1})
                        .select('some fields');
if(!somelist){
    res.status(500).json({ success: false });
}else{
    const arr = [];
    for (const item of somelist) {
        const user = await User.findById(item.some);
        if(user) arr.push(item);
    }
    console.log(arr)
    res.status(200).json(arr);
}

你当前的写法会给每条数据都发起一次数据库查询，数据量较大时会产生大量IO开销。可以先批量提取所有需要查询的用户ID，一次查询得到所有有效用户后再做过滤，性能提升非常明显：

const somelist = await Model.find({condition blah blah})
                        .sort({created_at: -1})
                        .select('some fields');
if(!somelist){
    res.status(500).json({ success: false });
}else{
    // 提取所有待查询的用户ID
    const toQueryUserIds = somelist.map(item => item.some);
    // 批量查询所有存在的用户
    const validUsers = await User.find({ _id: { $in: toQueryUserIds } }).select('_id');
    const validUserIdSet = new Set(validUsers.map(u => u._id.toString()));
    // 按原始顺序过滤有效项
    const filteredArr = somelist.filter(item => validUserIdSet.has(item.some.toString()));

    console.log(filteredArr)
    res.status(200).json(filteredArr);
}

内容的提问来源于stack exchange，提问作者rod james