NodeJS中Promise的.then晚于循环执行？重复输出问题求解

Hey there! Let's break down why you're seeing the same line repeated 4 times and how to fix it.

问题成因 #

1. var变量的作用域陷阱 #

你用var声明了循环变量i和j，但var是函数级作用域，不是块级作用域。这意味着整个循环里只有一个i和j的绑定，当异步的get()回调执行时，循环早就跑完了，i和j都停在了最后一次循环的数值。所有回调都会复用这个最终值，导致反复处理同一个datum，自然就重复输出了。

2. 循环边界错误（额外逻辑bug） #

你的循环条件写的是i < weeks.length-1和j < weeks[i].days.length-1，这会直接跳过最后一个week和最后一个day的处理——虽然这不是重复输出的直接原因，但也是需要修正的逻辑问题。

3. 异步操作未同步 #

你在循环里发起了多个异步数据库请求，但没有等待它们完成就继续下一次循环。这会导致多个回调同时修改weeks数组，可能引发数据覆盖或者顺序混乱，进一步加重重复问题。

解决方法 #

1. 用let替代var声明循环变量 #

let是块级作用域，每次循环都会创建新的变量绑定，回调函数会捕获当前循环的i和j值，不会再复用最终的循环变量。

2. 修正循环边界 #

把循环条件改成i < weeks.length和j < weeks[i].days.length，确保所有的week和day都被处理到。

3. 用async/await同步异步操作 #

把代码包裹在异步函数里，用await等待每个数据库请求完成，避免并发修改的问题，也让代码逻辑更清晰易读。

修改后的代码示例 #

// 把逻辑放在异步函数中
async function checkEventsForWeeks() {
  // 修正循环边界，用let声明i
  for (let i = 0; i < weeks.length; i++) {
    // 用let声明j
    for (let j = 0; j < weeks[i].days.length; j++) {
      const datum = weeks[i].days[j];
      const dateStr = `${datum.year}-${datum.month}-${datum.day}`;
      try {
        const doc = await db.collection('events').doc(dateStr).get();
        // 给当前day设置hasEvent状态
        weeks[i].days[j].hasEvent = doc.exists;
      } catch (err) {
        console.error('获取事件失败:', err);
        // 出错时默认设为false或做其他处理
        weeks[i].days[j].hasEvent = false;
      }
    }
  }
  // 处理完成后可以在这里使用更新后的weeks数组
  console.log(weeks);
}

// 调用函数执行逻辑
checkEventsForWeeks();

这样修改后，每个循环的datum都会被正确处理，异步请求也会按顺序完成，不会再出现重复输出的问题啦。

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