Node.js中for循环runNumber值被跳过，循环结构是否存在问题？

你碰到的这个问题是JavaScript里非常经典的闭包与var变量作用域的坑，咱们先把问题根源说清楚：

你用var声明的runNumber是函数级作用域变量，整个for循环里它都是同一个变量。而runNumberQueue.place()里的回调是异步执行的——等这些回调真正被调用的时候，外层的for循环早已经跑完了，runNumber的值已经变成了numberOfRuns + 1，这就导致所有回调都在使用最后一次循环的runNumber值，自然会出现跳过值、逻辑异常的情况。

下面给你几个可行的修复方案，按推荐程度排序：

方案1：用let替代var（最简洁的现代方案） #

ES6引入的let是块级作用域变量，每次循环都会创建一个新的runNumber绑定，异步回调会精准捕获当前循环迭代的那个变量值，改起来只需要换个关键字：

for (let runNumber=1; runNumber<=numberOfRuns; runNumber++) { 
    runNumberQueue.place(function(){ 
        versionRunningService.runMultiVersionJobs( 
            queue, jobId, sVersion, jobType, runNumber, currentConfig, allJobsConfig, appDir, 
            function(){ 
                runNumberQueue.next(); 
                currentIterationCount++; 
                if (currentIterationCount === numberOfIterations) { 
                    callback(); 
                } 
            } 
        ); 
    }); 
}

方案2：立即执行函数（IIFE）捕获变量（兼容旧环境） #

如果你的项目还需要兼容不支持ES6的老环境，可以用立即执行函数把每次循环的runNumber封装成独立的作用域，确保回调能拿到当前迭代的正确值：

for (var runNumber=1; runNumber<=numberOfRuns; runNumber++) { 
    // 用IIFE创建独立作用域，捕获当前runNumber的值
    (function(currentRun) {
        runNumberQueue.place(function(){ 
            versionRunningService.runMultiVersionJobs( 
                queue, jobId, sVersion, jobType, currentRun, currentConfig, allJobsConfig, appDir, 
                function(){ 
                    runNumberQueue.next(); 
                    currentIterationCount++; 
                    if (currentIterationCount === numberOfIterations) { 
                        callback(); 
                    } 
                } 
            ); 
        });
    })(runNumber); 
}

方案3：改用forEach遍历（更易读的异步友好写法） #

把循环次数转换成一个数组，用forEach遍历，每个迭代的参数会自动绑定到当前值，避免作用域问题：

// 生成从1到numberOfRuns的数组
Array.from({length: numberOfRuns}, (_, i) => i + 1).forEach(runNumber => {
    runNumberQueue.place(function(){ 
        versionRunningService.runMultiVersionJobs( 
            queue, jobId, sVersion, jobType, runNumber, currentConfig, allJobsConfig, appDir, 
            function(){ 
                runNumberQueue.next(); 
                currentIterationCount++; 
                if (currentIterationCount === numberOfIterations) { 
                    callback(); 
                } 
            } 
        ); 
    });
});

本质上这几个方案都是一个思路：让每个异步回调能捕获到当前循环迭代的独立变量副本，而不是共享同一个函数级的runNumber变量，这样就能解决值被跳过的问题了。

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