Node.js中MongoDB循环异步调用问题：async模块变量无法访问求助

这个问题的根源是var声明的变量作用域特性+闭包延迟执行导致的：你用var i声明的i是函数级作用域，整个for循环共享同一个i变量。当for循环执行完后，i的值已经变成了sample.length，这时候async.parallel里的异步回调才开始执行，此时访问sample[i]就等于访问sample[sample.length]——这显然是undefined，再访问.item自然就报错了。

下面给你几种可行的解决方案：

方案1：用let代替var声明循环变量（最简单） #

let是ES6引入的块级作用域变量，每次循环都会创建一个独立的i变量，每个异步函数捕获的都是当前循环迭代的那个i值，完美解决问题：

var sample = req.body; // sample will be an array list of items
var stringList = "";
var calls = [];
// 把var改成let，让每次循环都有独立的i
for(let i = 0; i < sample.length; i++) {
 console.log(sample[i].item) // 这里正常打印
 calls.push(function(callback) {
  db3.table.find({column1:sample[i].item}, function(err, temp){
   if (err) return callback(err);
   stringList = stringList + temp[0].item2;
   callback(null, stringList );
  });
 });
}
async.parallel(calls, function(err, result) {
 if (err) return console.log(err);
 console.log(result);
});

方案2：用立即执行函数（IIFE）保存当前循环值（兼容旧环境） #

如果你的环境不能用ES6的let，就用立即执行函数把当前的i值包裹成局部变量，形成闭包保存下来：

var sample = req.body; // sample will be an array list of items
var stringList = "";
var calls = [];
for(var i = 0; i < sample.length; i++) {
 console.log(sample[i].item)
 // 用IIFE捕获当前的i值，作为currentIndex传入
 calls.push((function(currentIndex) {
  return function(callback) {
   db3.table.find({column1:sample[currentIndex].item}, function(err, temp){
    if (err) return callback(err);
    stringList = stringList + temp[0].item2;
    callback(null, stringList );
   });
  };
 })(i));
}
async.parallel(calls, function(err, result) {
 if (err) return console.log(err);
 console.log(result);
});

方案3：改用Promise.all + async/await（更现代的写法） #

现在Node.js已经全面支持Promise和async/await，完全可以不用依赖async模块，代码更简洁还能避免共享变量的竞态问题（你的stringList是共享变量，多个异步回调同时修改可能导致结果混乱）：

首先把MongoDB的回调式调用包装成Promise：

// 封装find操作为Promise
const findItem2 = (targetItem) => {
  return new Promise((resolve, reject) => {
    db3.table.find({column1: targetItem}, (err, temp) => {
      if (err) reject(err);
      else resolve(temp[0].item2);
    });
  });
};

// 用async函数处理流程
async function processSampleItems() {
  try {
    const sample = req.body;
    // 生成所有查询的Promise数组
    const promiseArray = sample.map(item => findItem2(item.item));
    // 等待所有查询完成，得到所有item2的结果数组
    const allItem2s = await Promise.all(promiseArray);
    // 拼接成最终字符串
    const finalString = allItem2s.join('');
    console.log(finalString);
    // 这里执行所有查询完成后的下一步操作
  } catch (err) {
    console.error('查询出错:', err);
  }
}

// 启动执行
processSampleItems();

额外提醒 #

你的原代码里stringList是共享变量，在async.parallel的多个回调中同时修改可能会出现竞态条件（比如两个回调同时读取旧值、修改后写回，导致部分拼接丢失）。用Promise.all的方式先收集所有结果再统一拼接，能彻底避免这个问题，代码也更易维护。

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