我的 JavaScript 代码中的索引越界错误

Question

以下代码有错误：

generate: function() {
    var generated = [];
    for (var j = 0; j < objectDefinitions.length; j++) {
        var randomNumber;

        if (!objectDefinitions[j].restrictGeneration) {
            continue;
        }

        randomNumber = Math.random();

        if (randomNumber < objectDefinitions[j].probability) {
            generated.push(objectDefinitions[j].createObject());
            objectDefinitions[j].restrictGeneration = true;
            if (j > 5) {
            }
            setTimeout(function() {                 
                //console.log(j);
                objectDefinitions[j].restrictGeneration = false;
            }, objectDefinitions[j].cooldown);
        }
    }
    return generated;
}

当 setTimeout “触发”时，变量 j 为 6，这会导致数组索引超出范围异常，因为数组中只有 6 个项目。我真的不明白发生了什么，我在 setTimeout 之外检查了 j 并且它永远不是 6，但它似乎在声明之后发生了变化。感谢您的帮助。

Answer 1

这是由于闭包。

您会看到 6 的值，因为 j++ 在最后一次运行时触发了 for 循环的退出。因此，该 闭包 现在的 j 为 6。当 setTimeout 执行时（在 for 循环完成之后，它在该闭包的上下文中执行。

您需要将此代码移动到它自己的闭包中，这是通过将它移动到它自己的函数中来完成的。比如：

generate: function() {
    var generated = [];
    for (var j = 0; j < objectDefinitions.length; j++)
    {
       handleIteration(generated, j); // Re-factor your code into a new function
    }

    return generated;
}

Answer 2

你的空 if 语句执行（不做任何事情），然后它进入设置超时，j 仍然大于 5。

基本上，您应该在该点退出，而不是在 if 语句之后继续执行更多代码，或者在那里创建一个 else 语句，以便在 j > 5 时不再执行代码。

Answer 3

它读取的变量j 是您在循环中使用的变量，因此它没有被“复制”。

setTimeout(
    (function(k){
        return function() {            
            objectDefinitions[k].restrictGeneration = false;
        };
    }(j))
, objectDefinitions[j].cooldown);

这样你j 被复制到k 变量中。但是每次循环迭代都有一个k 变量。