在传递给Array.prototype.every的函数中,“可以从闭包中访问可变变量”

时间:2013-12-12 00:56:21

标签: javascript closures webstorm

代码会比我更明白地说:

var candidateIndex = 0;
var minValue = Number.MAX_VALUE;
topArray.every(function(element, index) {
    if (element.innerArray && element.innerArray.length < minValue) {
        minValue = element.innerArray.length;
        candidateIndex = index;
        if (minValue == 0) {
            return false;
        }
    }
    return true;
});

// ... use minValue and candidateIndex

这样做是通过topArray,并找到该数组的第一个成员,其长度为innerArray,否则找到长度最小的{{1} }}。它运行正常,但我已经准确报告的“检查器”可以从闭包中访问“Mutable变量”。

我发现这通常是件坏事,特别是对于异步代码。我查看了How to avoid access mutable variable from closureaccessing mutable variable in an event closure,并了解在这些情况下,匿名函数是异步的,并且希望当时存储可变变量的状态,但在我的情况下,我想要我调用的同步匿名函数来更改变量。

在这种情况下,我收到的警告是错误的,我应该忽略它,对吧?除了使用innerArray循环而不是for之外,有没有办法在不发出警告的情况下获得我想要的功能?

更新:,它的价值,警告似乎来自我的WebStorm IDE本身,而不是任何分析工具插件。

1 个答案:

答案 0 :(得分:1)

从上面的评论中得到确认,这个警告基本上是误报,我修改了代码以忽略警告信息:

topArray.every(function(element, index) {
    //noinspection JSReferencingMutableVariableFromClosure
    if (element.innerArray && element.innerArray.length < minValue) {
        minValue = element.innerArray.length;
        candidateIndex = index;
        //noinspection JSReferencingMutableVariableFromClosure
        if (minValue == 0) {
            return false;
        }
    }
    return true;
});

(警告仅在比较值时触发,而不是在设置时触发。)

我渴望听到任何其他答案,但如果我不这样做,我会在大约一周内接受这个答案。