冒泡排序不完全正常

时间:2016-05-23 13:33:34

标签: c++ bubble-sort

该程序有效,但问题是排序的数字没有出现。所有它带来的是1,这就是全部。错误在哪里?

#include <iostream>

using namespace std;

int const N = 20;

void pirmaisMasivs(int N);

int main (){

 cout << "Numbers being sorted - 5,4,2,6,1,3,8,9,10,7 > " << pirmaisMasivs;

}


void pirmaisMasivs(int N){
 int temp;
 int masivs[10] = {5,4,2,6,1,3,8,9,10,7};
 for( int i = 0; i < N - 1; i++ ){
    for( int j = 0; j < N - 1; j++ ){
        if( masivs[ j ] > masivs[ j + 1 ]){
        temp = masivs[ j ]; 
        masivs[ j ] = masivs[ j + 1];
        masivs[ j + 1 ] = temp;
 }
 }
 }


}

1 个答案:

答案 0 :(得分:1)

您的代码有几个问题。

首先,您声明要将整体数量排序为int const N = 20;,但稍后您会使用10作为固定文字计数。 N = 20显然是错误的。循环将导致数组边界超限。

另一个问题是,您不以任何方式输出已排序的数组。 您只需尝试调用sort方法。此外,您声明要在sort方法中对整数数组进行排序,因此您无法访问此方法之外的已排序数组以进行打印。

以下是您的计划的完全重写版本:

#include <iostream>

using namespace std;

int const N = 10;
int masivs[N] = {5,4,2,6,1,3,8,9,10,7};

void pirmaisMasivs() {   
    int temp;
    for( int i = 0; i < N - 1; i++ ) {
        for( int j = 0; j < N - 1; j++ ) {
            if( masivs[ j ] > masivs[ j + 1 ]) {
                temp = masivs[ j ]; 
                masivs[ j ] = masivs[ j + 1];
                masivs[ j + 1 ] = temp;
            }
        }
    }
}

void printMasivs() {
    for( int i = 0; i < N; i++ ) {
        if ( i == 0 ) {
            cout << masivs[ i ];
        }
        else {
            cout << ", ";
            cout << masivs[ i ];            
        }
    }
}

int main () {
    cout << "Numbers being sorted:\n";
    printMasivs();
    cout << "\n";
    pirmaisMasivs();
    cout << "\n";
    printMasivs();
}