仅使用 while 和 if 对数组进行排序答案

【问题标题】：Sorting array only with while and if仅使用 while 和 if 对数组进行排序
【发布时间】：2016-07-15 16:33:51
【问题描述】：

当我尝试运行程序时收到一条消息。为什么？

Segmentation fault

我的代码：

#include <stdio.h>

void sort_array(int *arr, int s);

int main() {
    int arrx[] = { 6, 3, 6, 8, 4, 2, 5, 7 };

    sort_array(arrx, 8);
    for (int r = 0; r < 8; r++) {
        printf("index[%d] = %d\n", r, arrx[r]);
    }
    return(0);
}

sort_array(int *arr, int s) {
    int i, x, temp_x, temp;
    x = 0;
    i = s-1;
    while (x < s) {
        temp_x = x;
        while (i >= 0) {
            if (arr[x] > arr[i]) {
                temp = arr[x];
                arr[x] = arr[i];
                arr[i] = temp;
                x++;
            }
            i++;
        }
        x = temp_x + 1;
        i = x;
    }
}

我认为问题出在if 语句中。你怎么看？为什么会发生？我认为我以积极的方式使用指向数组的指针。

谢谢！

【问题讨论】：

i 会太大并导致超出范围的访问，从而调用未定义的行为。
用-g编译程序并在gdb下运行，这会告诉你哪里出错了
尝试使用bubble sort 方法（就像使用here 对数组进行升序排序一样）

标签： c arrays sorting while-loop bubble-sort

【解决方案1】：

程序中的这个循环

    while (i >= 0) {
        //...
        i++;
    }

没有意义，因为i 无条件增加。

程序可以如下所示

#include <stdio.h>

void bubble_sort( int a[], size_t n )
{
    while ( !( n < 2 ) )
    {
        size_t i = 0, last = 1;

        while ( ++i < n )
        {
            if ( a[i] < a[i-1] )
            {
                int tmp = a[i]; 
                a[i] = a[i-1];
                a[i-1] = tmp;
                last = i;
            }
        }

        n = last;
    }
}   

int main( void ) 
{
    int a[] = { 6, 3, 6, 8, 4, 2, 5, 7 };
    const size_t N = sizeof( a ) / sizeof( *a );

    for ( size_t i = 0; i < N; i++ ) printf( "%d ", a[i] );
    printf( "\n" );

    bubble_sort( a, N );

    for ( size_t i = 0; i < N; i++ ) printf( "%d ", a[i] );
    printf( "\n" );

    return 0;
}

程序输出是

6 3 6 8 4 2 5 7 
2 3 4 5 6 6 7 8

如果您希望排序函数只有一个while循环，那么您可以通过以下方式实现它

void bubble_sort( int a[], size_t n )
{
    size_t i = 0;

    while ( ++i < n )
    {
        if ( a[i] < a[i-1] )
        {
            int tmp = a[i]; 
            a[i] = a[i-1];
            a[i-1] = tmp;
            i = 0;
        }
    }
}

【讨论】：

【解决方案2】：

在您的内部循环中，您将i 递增到超出数组的大小。您的算法应该要求您减少 i，但我不确定这是否足以修复排序算法。

您应该首先尝试使用单个 while 循环实现 冒泡排序，在该循环中比较相邻的项目并在交换它们时后退。

【讨论】：