循环中Ajax请求无法访问外部变量问题求助

我来帮你搞定这个问题！你的核心困扰其实是两个点：一是回调里的完成判断逻辑写错了，导致永远触发不了回调；二是虽然用了立即执行函数（IIFE），但可以优化变量捕获的方式，彻底避免索引相关的问题，同时完全不用全局变量。

先拆解下原代码的问题：

• 你用paramResults.length和errorResults.length是错误的，这俩是对象不是数组，length属性会返回undefined，所以那个“所有请求完成”的条件永远不会成立，你可能误以为是访问不到变量。
• 虽然IIFE试图捕获index，但直接捕获当前的参数项会更清晰，能彻底避免索引混淆的问题。
• 依赖对象的键数来统计完成数不可靠——如果某个请求成功但解析不到值，paramResults里不会添加这个键，会导致数量统计错误。

下面是修复后的完整代码，我会在后面解释关键修改点：

function getCameraParameter(paramList, x_callback) {
    var paramResults = {};
    var errorResults = {};
    var completedRequests = 0; // 用计数器跟踪已完成的请求数
    var totalRequests = paramList.length;

    // 用forEach替代for循环+IIFE，自动捕获当前参数项，更简洁可靠
    paramList.forEach(function(param) {
        $.ajax({
            url: "/params.cgi?getParameter&" + param,
            type: 'GET',
            dataType: 'text',
            cache: false,
            async: true,
            timeout: 30000,
            success: function(x_data) {
                var match = new RegExp(param + '=([^\n]+)').exec(x_data);
                if (match && match.length > 1) {
                    paramResults[param] = match[1];
                }
                checkIfAllCompleted();
            },
            error: function(e) {
                errorResults[param] = "";
                checkIfAllCompleted();
            }
        });
    });

    // 单独抽离完成判断逻辑，避免重复代码
    function checkIfAllCompleted() {
        completedRequests++;
        // 当完成数等于总请求数时，触发回调
        if (completedRequests === totalRequests) {
            x_callback(paramResults, Object.keys(errorResults).length > 0);
        }
    }
}

function cameraTest() {
    getCameraParameter(["Resolution", "CameraIP", "CameraSerialNumber"], printParameters);
}

function printParameters(paramValues, is_error) {
    if (is_error) {
        console.log("Error occurred\n");
    }
    for (var key in paramValues) {
        if (paramValues.hasOwnProperty(key)) {
            console.log(key + "->" + paramValues[key] + "\n");
        }
    }
}

关键修改说明：

1. 用forEach替代for循环+IIFE：每个迭代的param会被自动捕获到Ajax回调的闭包里，彻底不用管索引，自然不会出现innerIndex未定义的问题。
2. 新增请求计数器：用completedRequests跟踪已完成的请求数，不管请求成功还是失败，只要完成就递增，当计数器等于总请求数时触发回调，这个逻辑比依赖对象键数可靠得多。
3. 抽离完成判断逻辑：把重复的判断代码写成单独的checkIfAllCompleted函数，让代码结构更清晰，也避免了重复代码。
4. 优化正则匹配逻辑：用match替代语义模糊的value，同时增加了match存在的判断，避免出现undefined报错。
5. 保持变量私有：paramResults和errorResults依然是getCameraParameter函数内部的变量，完全不会暴露成全局对象。

这样修改后，你不仅能正确访问每个请求对应的参数项，还能确保所有请求完成后只触发一次回调，完美解决你的问题。

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