可变变量可以从闭包中访问[重复]

Question

我有以下代码：

for (var i = 0; i < data.length; i++) {
        var file = data[i];
        $.getJSON("/types/" + file, function(json) {
            if (json[0] !== undefined) {
                console.log(json[0] + file);
            }
        });
    }

但是我的编辑说“可变变量可以从闭包中访问”。我尝试将function(json) { 更改为function(json, file) {，但这不起作用，因为这是jquery 的默认功能。

希望你能帮我解决问题。

Answer 1

对于此类循环，您需要将内容放入闭包中。

for (var i = 0; i < data.length; i++) {
    (function(){
        var file = data[i];
        $.getJSON("/types/" + file, function(json) {
            if (json[0] !== undefined) {
                console.log(json[0] + file);
            }
        });
    })();
}

Answer 2

您可以按照建议使用闭包，但它也不会使您的代码更易于阅读，尤其是当您的闭包在循环内时！简单地创建一个函数并在每次迭代中调用它会更容易和更简洁。

for (var i = 0; i < data.length; i++) {
    doJson(data[i]);
}

function doJson(file) {
    $.getJSON("/types/" + file, function(json) {
        if (json[0] !== undefined) {
            console.log(json[0] + file);
        }
    });
}

或者您可以简单地更新您的循环以改用forEach，因为每次迭代都会触发一个新函数，范围将受到限制。

data.forEach(function(file) {

    $.getJSON("/types/" + file, function(json) {
        if (json[0] !== undefined) {
            console.log(json[0] + file);
        }
    });
});

Answer 3

如果您可以使用 ES6，您可以简单地将 var 替换为 let，一切都会按预期进行，因为 let 在 for 循环的每次迭代中都会创建一个新范围。

for (let i = 0; i < data.length; i++) {
        var file = data[i];
        $.getJSON("/types/" + file, function(json) {
            if (json[0] !== undefined) {
                console.log(json[0] + file);
            }
        });
    }

Answer 4

你应该使用闭包：（在循环中运行函数时要小心）。

for (var i = 0; i < data.length; i++)
{
    var file = data[i];

    (function(f)
    {

        $.getJSON("/types/" + f, function(json)
        {
            if (json[0] !== undefined)
            {
                console.log(json[0] + f);
            }
        })
    })(file);


}