Question

我的sort函数遇到了一些麻烦，它也使用了swap函数。我一直试图找到我的错误，但无法找到它。其余的代码完美无缺，只有sort函数不起作用（它不会对任何东西进行排序。）

如果有人对如何解决我的问题有任何想法，我会在这里留下代码，请回复。

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

void sort(int *values, int n);
void swap(int arr[], int i);
void populate(int arr[]);
const int arraysize = 10;


int main()
{

    srand((long int) time(NULL));

    int ela[10] = {0};

    populate(ela); // populates with random values

    printf("before func\n");
    for(int i = 0; i < arraysize; i++)
    {
        printf("%i\n", ela[i]); 
    }


    sort(ela, arraysize); // this is the function that is not working

    printf("\n\nafter func\n");

    for(int i = 0; i < arraysize; i++)
    {
        printf("%i\n", ela[i]); 
    }

    return 0;
}





void sort(int *values, int n)
{
int count = 1;
    while(count != 0)
    {
        count = 0;
        for (int i = 0; i < n; i++) {

            if(values[i] > values[(i + 1)] && values[(i + 1)] != '\0')
            {
                swap(values, i);
                count++;
            }
            if (count == 0) break;
        }

    }

}

void swap(int arr[], int i)
{
    int save = arr[i];
    arr[i] = arr[i+1];
    arr[i + 1] = save;
    return;
}

void populate(int arr[])
{
    for(int i = 0; i < arraysize; i++)
    {
        arr[i] = (rand() % 15);
    }
}

Answer 1

void sort(int *values, int n)
{
int count = 1;
    while(count != 0)
    {
        count = 0;
        for (int i = 0; i < n-1; i++) {

            if(values[i] > values[(i + 1)] )
            {
                swap(values, i);
                count++;
            }
            //if (count == 0) break;

        }

    }

}

我怀疑这永远不会是真的values[(i + 1)] != '\0'因为值是整数。所以运行for循环直到条件i<n-1能够正常工作，因为你要用i+1元素交换{ {1}}。
如果有一些像i这样的输入，那么你的for循环中的if (count == 0) break;也应该被移除，因为循环会中断，因此不会排序。

工作Ideone：http://ideone.com/k1LJau

Answer 2

以下是sort()功能的更新版本：

void sort(int *values, int n)
{
int count = 1;
    while(count != 0)
    {
        count = 0;
        for (int i = 0; i < n; i++) {

            if(values[i] > values[(i + 1)] && values[(i + 1)] != '\0')
            {
                swap(values, i);
                count++;
            }
        }
    }
}

即使您要检查数组的初始值，您的if(count==0) break;语句也会退出for循环。因此，例如[5 6 9 8]将退出，因为count在检查前两项时为0。

Answer 3

如果要使用sort（）函数，则必须添加算法头文件。但它不适用于c。如果您添加using namespace std; 然后它会工作。然后你必须这样打电话。

sort(ela, ela+arraysize);

您可以像下面一样编辑排序功能。以上答案也很好。

void Sort(int *values, int n)
{
    int cnt=0;
    while(1)
    {
        cnt++;
        if(cnt==n-1)
            break;
        for (int i = 0; i < n-cnt; i++)
        {

            if(values[i] > values[(i + 1)] && values[(i + 1)] != '\0')
            {
                swap(values, i);
            }
        }

    }

}

Answer 4

您的冒泡排序功能有误。

这是一个演示程序，显示如何编写函数

#include <stdio.h>

void swap( int a[], size_t i )
{
    int tmp = a[i];
    a[i] = a[i+1];
    a[i+1] = tmp;
}

void bubble_sort( int a[], size_t n )
{
    for ( _Bool sorted = 0; n-- != 0 && !sorted;  )
    {
        sorted = 1;
        for ( size_t i = 0; i < n; i++ )
        {
            if ( a[i+1] < a[i] )
            {
                sorted = 0;
                swap( a, i );
            }
        }
    }
}

int main(void) 
{
    int a[] = { 9, 8, 7, 6, 5, 4, 3, 2, 1, 0 };
    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;
}

输出

9 8 7 6 5 4 3 2 1 0 
0 1 2 3 4 5 6 7 8 9

排序功能无法正常工作（冒泡排序） - c

4 个答案: