汗学院:JavaScript循环耗时太长

时间:2017-06-07 11:50:31

标签: javascript khan-academy

enter image description here

我正在阅读可汗学院的算法课程。我在https://www.khanacademy.org/computing/computer-science/algorithms/insertion-sort/p/challenge-implement-insertion-sort。到目前为止,我有:

var insert = function(array, rightIndex, value) {
for(var j = rightIndex;
    j >= 0 && array[j] > value;
    j--) {
    array[j + 1] = array[j];
}   
array[j + 1] = value; 
};

var insertionSort = function(array) {

for(var i= 1;  i < array.length ; i++ ) {
    insert(array, i ,array[i+1] );
}

};

var array = [22, 11, 99, 88, 9, 7, 42];
insertionSort(array);

你可以看到截图中的代码行似乎是问题所在,但它对我来说很好看。我做错了什么?

1 个答案:

答案 0 :(得分:1)

您在rightIndex处开始i并移动了值array[i + 1],但i到达array.length并且insert从设置开始rightIndex + 1处的元素。这将导致阵列增长。

移动当前元素并从上一个索引开始:

for (var i = 1; i < array.length; i++) {
    insert(array, i - 1,array[i]);
}

在调试时捕获此信息的一种方法是密封您的阵列,使其无法增长:

var array = Object.seal([22, 11, 99, 88, 9, 7, 42]);

这仅适用于strict mode如果你处于草率模式,它只会隐藏错误。