选择排序功能排序不正确

Question

我正在尝试在 C 中实现我的第一个排序算法，其中一个数字数组作为命令行参数给出，然后调用一个函数对数组进行排序并打印输出。该程序在排序之前重新打印数组，据我所知，错误出在排序算法本身。这是函数：

    void ascending(int n, int arr[])
    {
        for (int i = 0; i < (n - 1); i++)
        {
           //min is equal to i (use as index)
           int min = i;

            //Compare arr[i] to all other elements in the array
            for (int j = i + 1; j < n; j++)
            {
                //If a smaller number is found, its index (j) is now min
                if (arr[j] < arr[min])
                {
                    min = j;

                    //Swapping values to be in correct place
                    int temp = arr[min];
                    arr[min] = arr[i];
                    arr[i] = temp;
                }
            }
        }

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

如果我要求它对 [5, 4, 60, 2, 1] 进行排序，它将正确地将输出排序为已排序：[1, 2, 4, 5, 60]

但是如果我要求它对 [60, 5, 3, 3, 1, 4, 2] 进行排序，它会打印：[2, 1, 4, 3, 3, 5, 60]，对一些数字进行排序但不是其他人。

提前致谢！

Answer 1

在Selection Sort中，内部for循环的目标是找到未排序子数组（即从索引[=12=开始的数组）中最小元素的索引] 并以索引 n-1 结束）。交换应该在找到最小值后发生，以便它正确放置在数组中的索引 i 处：

void ascending(int n, int arr[])
{
    for (int i = 0; i < (n - 1); i++)
    {
       //min is equal to i (use as index)
       int min = i;

       //Compare arr[i] to all other elements in the array
       for (int j = i + 1; j < n; j++)
       {
           //If a smaller number is found, its index (j) is now min
           if (arr[j] < arr[min])
           {
               min = j;
           }
       }

       //Swapping values to be in correct place
       int temp = arr[min];
       arr[min] = arr[i];
       arr[i] = temp;
    }
    printf("sorted: ");
    for (int i = 0; i < n; i++)
    {
        printf("%i, ", arr[i]);
    }
    printf("\n");
}

Answer 2

在这个 if 语句中

           if (arr[j] < arr[min])
            {
                min = j;

                //Swapping values to be in correct place
                int temp = arr[min];
                arr[min] = arr[i];
                arr[i] = temp;
            }

作业后

                min = j;

这次交换

                //Swapping values to be in correct place
                int temp = arr[min];
                arr[min] = arr[i];
                arr[i] = temp;

没有意义。

内层的for循环可以写成下面这样

        for (int j = i + 1; j < n; j++)
        {
            //If a smaller number is found, its index (j) is now min
            if (arr[j] < arr[min])
            {
                min = j;
            }
        }

        if ( min != i )
        {   
                //Swapping values to be in correct place
                int temp = arr[min];
                arr[min] = arr[i];
                arr[i] = temp;
        }

注意函数的声明方式最好是第一个参数指定数组，第二个参数指定数组中的元素个数，

例如

void ascending( int arr[], size_t n )
{
    for ( size_t i = 0; i < n; i++ )
    {
        size_t min = i;

        for ( size_t j = i + 1; j < n; j++ )
        {
            if ( arr[j] < arr[min] ) min = j;
        }

        if ( min != i )
        {
            int tmp = arr[i];
            arr[i] = arr[min];
            arr[min] = tmp;
        }
    }
}

选择排序功能排序不正确

Selection Sort function working sorting incorrectly

c

selection-sort