Node.js+Express+Mongoose接口在React中随机丢数据，Postman正常

问题根源分析 #

你的后端代码存在几个关键问题导致数据返回不稳定：

• 全局变量污染：循环中的i和j没有用let/const声明，属于全局变量。在异步循环（await）执行时，全局变量会被后续循环迭代覆盖，导致部分迭代逻辑混乱，数据无法正确推入newObject。
• 缺少空值判断与错误处理：如果某个p_id在priceModel中查询不到数据，package[0]会是undefined，此时组装obj时访问package[0]._id等属性会直接抛出错误，中断后续循环，导致返回的数据不完整。
• 串行异步效率低：用普通for循环串行执行异步查询，不仅响应慢，还容易因异步时序问题导致数据错乱。

修复后的代码 #

exports.getPackages = async function (req, res) {
  try {
    const pids = req.query.p_id;
    // 用Promise.all并行处理所有p_id的查询，避免变量污染+提升效率
    const newObject = await Promise.all(pids.map(async (pid) => {
      // 查询当前pid的价格数据
      const package = await priceModel
        .find({ pid })
        .select({ pro_config_options: 0 });
      if (!package.length) {
        // 当前pid无数据时返回null，后续过滤掉，不中断整体流程
        return null;
      }
      // 查询当前pid的特性数据
      const packageFeatures = await featuresModel
        .find({ pid })
        .select({ pid: 0 });
      // 用map替代嵌套for循环处理currency数据，更简洁
      const cr = package.map(item => ({
        currency: item.currency,
        annually: item.annually,
        biennially: item.biennially,
        triennially: item.triennially,
        monthly: item.monthly,
        quarterly: item.quarterly,
        semiannually: item.semiannually,
        qsetupfee: item.qsetupfee,
        ssetupfee: item.ssetupfee,
        asetupfee: item.asetupfee,
        bsetupfee: item.bsetupfee,
        tsetupfee: item.tsetupfee,
      }));
      // 组装当前pid的返回对象
      return {
        id: package[0]._id,
        pid: package[0].pid,
        name: package[0].name,
        title: package[0].title,
        tid: "",
        fname: package[0].fname,
        currency: cr,
        addons: [],
        freedomain: [],
        packageFeatures,
        addonids: package[0].addonids,
      };
    }));
    // 过滤掉无数据的pid项
    res.json(newObject.filter(item => item !== null));
  } catch (err) {
    // 统一错误处理，避免接口静默失败
    console.error(err);
    res.status(500).json({ message: '服务器内部错误', error: err.message });
  }
};

关键改动说明 #

1. 替换循环方式：用Array.map结合Promise.all并行处理所有p_id的异步查询，彻底解决全局变量污染问题，同时大幅提升接口响应速度。
2. 添加空值判断：查询priceModel后判断是否有数据，无数据时返回null，后续过滤掉这些空值，保证返回数据的有效性，且不会中断整体流程。
3. 全局错误捕获：用try/catch包裹整个逻辑，出现错误时返回明确的错误信息，避免接口静默崩溃。
4. 简化数据组装：用map替代嵌套for循环处理currency数据，代码更简洁易维护。
5. 局部变量声明：所有变量都用const/let声明，彻底避免全局变量污染。

额外优化建议 #

• 前端请求时可先对p_id数组去重，避免重复查询相同ID。
• 可以用MongoDB的$in语法批量查询所有p_id的price和features数据，再按pid分组处理，进一步减少数据库查询次数，提升性能。

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