1

我在 jsPerf 的测试用例中有以下代码：

var arr = [0, 45, 96, 8, 69, 62, 80, 91, 89, 24, 6, 23, 49, 88, 26, 40, 87, 61, 83, 2, 60, 53, 43, 82, 67, 3, 65, 37, 42, 77, 73, 38, 9, 46, 75, 10, 63, 15, 47, 28, 79, 55, 59, 95, 11, 93, 70, 98, 25, 48, 30, 5, 72, 12, 84, 1, 29, 13, 50, 33, 19, 7, 31, 57, 32, 44, 74, 51, 35, 90, 86, 54, 4, 64, 92, 71, 22, 41, 16, 17, 27, 76, 39, 18, 99, 94, 36, 66, 85, 20, 21, 56, 34, 81, 14, 78, 68, 58, 97, 52];

Array.prototype.remove = function(from, to) {
  var rest = this.slice((to || from) + 1 || this.length);
  this.length = from < 0 ? this.length + from : from;
  return this.push.apply(this, rest);
};

function quicksort( arr ) {
  if ( arr.length <= 1 ) 
    return arr;
  var i = 0, 
      len = arr.length, 
      less = [], 
      greater = [], 
      random = Math.floor( Math.random() * len ), 
      pivot = arr[ random ];
  arr.remove( random );
  for ( ; i < len - 1; i++ ){
    if ( arr[ i ] <= pivot )
      less.push( arr[ i ] );
    else 
      greater.push( arr[ i ] );
  }
  return quicksort( less ).concat( pivot, quicksort( greater ) );
};

如果将其复制到控制台并运行quicksort( arr )，您会看到它正确返回了一个排序数组。

但由于某种原因，在jsPerf 上的这个测试用例中，我的快速排序函数似乎只返回一个数字（如“Perparation Code Output”所示）。它的运行速度似乎也比它应该的要快。

任何人对正在发生的事情的想法将不胜感激。

4

1 回答 1

2

我认为问题在于您在原始数组上.remove()调用该函数，因此它很快将其剥离为空。换句话说，每次对快速排序函数的初始调用都会删除一个元素。

当我首先创建数组的副本时，它似乎可以工作。

于 2012-01-29T17:22:26.257 回答