Node.js中使用mssql npm时map函数外返回空数组问题求助

嘿，我一眼就看出问题出在哪了——你这里的异步操作没有被正确等待，导致返回响应的时候，noneDuplicateArrayContainer和duplicateArrayContainer还没来得及被填充，自然是空的。下面给你拆解问题和修复方案：

核心问题分析 #

1. Array.map() + 异步函数的坑：你在items.map(async (...) => { ... })里用了async函数，但map本身不会等待这些异步回调执行完毕，它会直接返回一堆Promise对象，而你没有等待这些Promise完成就继续往下走了。
2. for循环里的异步操作未被阻塞：for循环本身不会等待每一次迭代里的异步操作完成，所以循环快速走完，而里面的queryCostToDb这些异步请求还在后台跑，等你返回响应的时候，push操作根本还没执行。

修复方案 #

我们需要把异步操作的等待逻辑补全，这里推荐两种方式：用Promise.all()批量等待Promise，或者用for...of逐个等待异步操作。下面是修改后的完整代码，我标注了关键修改点：

module.exports = { 
  someRouteHandler: async function(req, res, next) { 
    const fileStream = await readFileFromS3(req.body.filename); 
    if (req.body.productName === "Nike" && type === "Male") { 
      const result = await getBrandInformation(req.body, fileStream); 
      const { brandInformation, brandItems } = result; 
      const noneDuplicateArrayContainer = []; 
      const duplicateArrayContainer = []; 

      for ( const { itemNumber } of brandItems ) { 
        let items = await getMatchingList(itemNumber); 

        if (items.length > 1) { 
          // 修改点1：用Promise.all等待所有map里的异步操作完成
          await Promise.all(items.map(async({ identifier }) => { 
            if (identifier) { 
              let cost = await queryCostToDb(identifier); 
              duplicateArrayContainer.push({ brandItems, identifier, cost }) 
            } else { 
              duplicateArrayContainer.push({ brandItems, identifier }) 
            } 
          })); 
        } else { 
          // 修改点2：同样用Promise.all等待这里的异步操作
          await Promise.all(items.map(async({ identifier }) => { 
            let cost = await queryCostToDb(identifier); 
            noneDuplicateArrayContainer.push({ brandItems, identifier, cost }); 
          })); 
        } 
      } 

      // 现在所有异步push操作都完成了，再返回响应
      return res.status(200).json({ brandInformation, noneDuplicateArrayContainer, duplicateArrayContainer }) 
    } 
  } 
}

为什么这样改？ #

• Promise.all(items.map(...))会把map返回的所有Promise收集起来，等待它们全部执行完成后才继续执行后续代码，确保所有push操作都已经完成。
• 如果你更倾向于顺序执行（而不是并行），也可以把map换成for...of循环，比如：

// 替代上面的Promise.all写法
for (const { identifier } of items) {
  if (identifier) {
    let cost = await queryCostToDb(identifier);
    duplicateArrayContainer.push({ brandItems, identifier, cost })
  } else {
    duplicateArrayContainer.push({ brandItems, identifier })
  }
}

这种方式会逐个处理每个item，上一个异步操作完成后才会处理下一个，适合对顺序有要求的场景。

额外注意点 #

• 永远记住：异步操作必须被显式等待，尤其是在循环、数组方法里，async/await不会自动帮你等待所有嵌套的异步操作。
• map、forEach这类数组方法设计的时候就没考虑异步回调，所以处理异步逻辑时尽量用for...of或者配合Promise.all。

内容的提问来源于stack exchange，提问作者mugs-amr