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のこのテストケースでは、私のクイックソート関数は単一の数値のみを返すようです（「PerparationCodeOutput」に見られるように）。また、おそらく必要以上に高速に実行されているようです。

何が起こっているのかについてのアイデアは誰でも大歓迎です。

4

1 に答える 1

2

問題は、元の配列.remove()でその関数を呼び出しているため、すぐに何も削除されないことだと思います。つまり、クイックソート関数を最初に呼び出すたびに要素が削除されます。

最初に配列のコピーを作成するようにすると、それは機能しているようです。

于 2012-01-29T17:22:26.257 に答える