Node.js处理多异步操作时forEach未等待完成导致保存项目协作人数组为空

问题根因 #

你代码的核心问题是 Array.prototype.forEach 不支持等待异步回调执行完成：你给forEach传入的每个async回调都是独立的异步任务，外层Promise不会等待这些回调执行完毕，就直接执行了resolve(collaboratorsArray)，此时数据库查询还没返回，数组自然是空的。

解决方案1：基于Promise.all修复原有逻辑 #

不用手动维护数组，也不需要额外嵌套Promise，直接用Promise.all等待所有异步查询完成：

export const newProject = async (req, res) => {
  const { userID } = req.params
  const { name, description, collaborators } = req.body

  try {
    let collaboratorsArray = []
    // 如果有传collaborators，等待所有用户查询完成
    if (collaborators?.length) {
      // 把每个用户名的查询转为Promise，用Promise.all等待全部完成
      collaboratorsArray = await Promise.all(
        collaborators.map(username => User.findOne({ username }))
      )
    }

    // 直接执行保存，不需要额外封装saveProject函数
    const project = await new Project({
      name,
      description,
      collaborators: collaboratorsArray,
      projectOwner: userID
    }).save()

    res.status(201).json({ 
      success: true,
      projectID: project._id,
      name: project.name, 
      description: project.description,
      collaborators: project.collaborators,
      projectOwner: project.projectOwner
    })
  } catch (error) {
    res.status(400).json({ success: false, message: 'Invalid request', error })
  }
}

解决方案2：更优的批量查询（性能更高） #

循环调用findOne会发起N次数据库请求，性能很差，你可以直接用mongoose的$in操作符一次查询所有符合条件的用户：

if (collaborators?.length) {
  // 一次查询拿到所有协作用户，效率远高于循环查询
  collaboratorsArray = await User.find({ username: { $in: collaborators } })
}

这种写法只需要1次数据库请求，且返回的就是匹配的用户数组，不需要额外处理。

额外注意点 #

• 你原有代码里封装的loopCollaborators、saveProject都是不必要的嵌套，async函数里直接按顺序写异步逻辑即可，await会自动等待前一个异步任务完成
• 不要在forEach、map等数组遍历方法里直接写带副作用的异步逻辑，除非你明确知道怎么处理异步等待，优先用Promise.all批量处理并行异步任务

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