web-dev-qa-db-ja.com

バブルソートアルゴリズムJavaScript

JavaScriptでのバブルソートアルゴリズムのこの実装の何が問題になっているのか教えてください。

for (var i=1; i<records.length; i++){
        for (var j=records.length; j<1; j--){
            if (parseInt(records[i-1]) < parseInt(records[i])){
                var temp = records[i-1];
                records[i-1] = records[i]
                records[i] = temp;
            }   
        }    
    }
9
kostas trichas

バブルソートのコードのカップル

バブルソートは、大きな配列には使用しないでください。単純にするために、小さな配列に使用できます。

方法1

var a = [33, 103, 3, 726, 200, 984, 198, 764, 9];

function bubbleSort(a) {
    var swapped;
    do {
        swapped = false;
        for (var i=0; i < a.length-1; i++) {
            if (a[i] > a[i+1]) {
                var temp = a[i];
                a[i] = a[i+1];
                a[i+1] = temp;
                swapped = true;
            }
        }
    } while (swapped);
}

bubbleSort(a);
console.log(a);

方法2

function bubbleSort(items) {
    var length = items.length;
    //Number of passes
    for (var i = 0; i < length; i++) { 
        //Notice that j < (length - i)
        for (var j = 0; j < (length - i - 1); j++) { 
            //Compare the adjacent positions
            if(items[j] > items[j+1]) {
                //Swap the numbers
                var tmp = items[j];  //Temporary variable to hold the current number
                items[j] = items[j+1]; //Replace current number with adjacent number
                items[j+1] = tmp; //Replace adjacent number with current number
            }
        }        
    }
}

方法

function bubbleSort() {
    var numElements = this.dataStore.length;
    var temp;
    for (var outer = numElements; outer >= 2; --outer) {
        for (var inner = 0; inner <= outer-1; ++inner) {
            if (this.dataStore[inner] > this.dataStore[inner+1]) {
                swap(this.dataStore, inner, inner+1); }
        }
        console.log(this.toString()); 
    }
}
30
Ignatius Andrew

ES6JavaScriptでの簡単な実装は次のようになります。

    function BubbleSort(arr) {
      const sortedArray = Array.from(arr);
      let swap;
      do {
        swap = false;
        for (let i = 1; i < sortedArray.length; ++i) {
          if (sortedArray[i - 1] > sortedArray[i]) {
            [sortedArray[i], sortedArray[i - 1]] = [sortedArray[i - 1], sortedArray[i]];
            swap = true;
          }
        }
      } while (swap)
      return sortedArray;
    }
console.log(BubbleSort([3, 12, 9, 5]));
2
Collin

2番目のループではiの代わりにjを使用する必要があり、j <1をj> 1に変更することを忘れないでください

1
Mansuro
for (var j=records.length; j<1; j--){

それはすべきではありません

for (var j=records.length; j>1; j--){
1
davbryn

バブルソートでは、iループが反復を完了すると、i番目の要素が正しい位置にあると思います。つまり、jループを次のように記述する必要があります。

for (var j = i + 1; j < records.length; j++)

そうしないと、バブルソートは(さらに)非効率になります。

0
James