荒谬的javascript错误:必须多次调用函数才能正常运行

Question

我毫不怀疑我自己的白痴对此负有责任.我不是一个程序员,而是一个科学家,而且我只是一直在捣乱,直到它起作用,这就是我最终得到如此奇怪的错误.基本上,任何帮助将非常感激.

好的,我的功能是这样的:

function discardDuplicates(threshold) {
    for (var m = 0; m < xCo2.length; m++){
        var testX = xCo2[m];
        var testY = yCo2[m];
        for (var n = 0; n < xCo2.length; n++){
            if (m != n) {
                if ((Math.abs(xCo2[n] - testX) < threshold)
                    && (Math.abs(yCo2[n] - testY) < threshold)
                    && deltas[m] > deltas[n]){

                    xCo2.splice(n,1);
                    yCo2.splice(n,1);
                    deltas.splice(n,1);
                }
            }
        }
    }
}

我正在检测具有存储在xCo2和yCo2阵列中的坐标(x,y)的特征,每个坐标也具有称为"delta"的属性.我想检查一下我是否在基本相同的地方识别了几个功能 - 如果我有,它们可能是重复的,所以除了列表中具有最高delta的那个之外我删除了所有功能.

对,基本上,这不起作用!

目前我必须这样做:

//ugly hack 
var oldLength = 0;
var newLength = 1;
while (oldLength != newLength) {
    oldLength = xCo2.length;
    discardDuplicates(10);
    newLength = xCo2.length;
}

因为第一次调用它的函数不会删除任何重复项.第二次它删除了大部分.第三次它通常都拥有它们......所以我只是继续调用它直到它停止删除重复项.该功能绝对有效; 它正在删除正确的点.

我问这个问题的原因是这个错误现在已经发生了第二次,第二次使用了第二个函数,这次尝试删除delta值太低的任何坐标.

我们将非常感激启蒙运动!

Answer 1

调用splice数组将删除一个条目并将下一个条目移动到此位置,这是非常常见的错误.

让我们来看看:

xCo2 = [1, 2, 3, 4, 5];
n = 0;

xCo2.splice(n, 1); // n = 0, removed 1
>> [2, 3, 4, 5];
n ++;

xCo2.splice(n, 1); // n = 1, removed 3
>> [2, 4, 5];
n++;

看到问题？您正在跳过条目,您需要做的是n每次从阵列中删除条目时减少.由于您总是删除1个条目,因此在拼接阵列后需要将n减少一次.

if ((Math.abs(xCo2[n] - testX) < threshold)
     && (Math.abs(yCo2[n] - testY) < threshold) && deltas[m] > deltas[n] ){

    xCo2.splice(n,1);
    yCo2.splice(n,1);
    deltas.splice(n,1);
    n--;
}

PS:一些空白大大增加了代码的可读性.