为何仅更新数组中第二个用户？如何实现多用户数据批量更新？

原代码核心问题 #

• Array.map搭配异步函数时，不会等待每个异步操作完成，多个更新操作并行执行，且全局变量updateUserData会被每次迭代覆盖，最终仅保留最后一次更新结果。
• 错误调用await createNotifications()，实际应调用内部定义的createData()函数。
• 混合使用await与.then()，导致异步流程逻辑混乱，无法准确控制执行顺序。
• 依赖setTimeout等待异步操作完成完全不可靠，无法确保所有更新操作都已执行完毕。

修复后的代码 #

exports.createNotifications = asyncHandler(async (req, res, next) => {
  const { userPhone, someData } = req.body;

  // 封装单个用户的处理逻辑
  const processSingleUser = async (phone) => {
    // 去除手机号前后空格，避免因格式问题查询不到用户
    const cleanPhone = phone.trim();
    const user = await Users.findOne({ phone: cleanPhone });
    if (user) {
      return await Users.findOneAndUpdate(
        { phone: cleanPhone },
        { $push: { someData: someData } },
        { new: true }
      );
    }
    return null; // 用户不存在时返回null标记
  };

  // 用Promise.all等待所有用户的更新操作完成
  const updateResults = await Promise.all(userPhone.map(processSingleUser));
  
  console.log('更新结果列表:', updateResults);

  res.status(200).json({
    success: true,
    updatedCount: updateResults.filter(item => item !== null).length
  });
});

关键优化点 #

• 用Promise.all包裹异步操作数组，确保所有更新完成后再执行后续逻辑，彻底解决异步流程失控问题。
• 将单用户处理逻辑封装为独立函数，避免全局变量被覆盖，同时让代码结构更清晰。
• 对手机号做trim()处理，解决示例中手机号带空格可能导致的查询失败问题。
• 收集所有更新结果，方便排查问题或返回更新统计给前端。

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