实现快速排序时出现分段错误

Question

我想实现堆排序。为此，我浏览了这个http://faculty.simpson.edu/lydia.sinapova/www/cmsc250/LN250_Tremblay/L06-QuickSort.htm#basic 教程并编写了以下代码：

#include <stdio.h>


int quick_sort(int a[],int first,int last);

int main()
{
    int a[]= {12,3,4,23,1,7,9,34,89,45};
    int i;
    printf("Enter 10 integers: \n");

    for ( i = 0 ; i < 10 ; i++ )
    {
        scanf("%d",&a[i]);
        printf("\t%d\n",a[i]);
    }
    for ( i = 0 ; i < 10 ; i++ )
    {
        printf("\n%d ",a[i]);
    }
    quick_sort(a,0,9);

    for ( i = 0 ; i < 10 ; i++ )
    {
        printf("%d ",a[i]);
    }
    return 0;
}

int quick_sort(int a[],int first,int last)
{
    int i,j,pivot,temp ;

    if ( first - last <= 1 && first - last >= -1 )
    {
        return 0;
    }
    else
    {
        i = first ;
        j = last ;

        pivot = a[(i+j) / 2 ] ;

        while ( i != j )
        {
            while ( a[i] < pivot )
            {
                i++;
            }
            while( a[j] > pivot )
            {
                j--;
            }

            temp = a[i] ;
            a[i] = a[j] ;
            a[j] = temp ;
        }
    }
    quick_sort(a,0,i-1);
    quick_sort(a,j+1,9);
    return 0;

}

在使用 gcc 编译器运行它时，我遇到了分段错误。请帮我解决。

Answer 1

问题的quick_sort() 函数中有几件事对我来说是个谜。并不是说他们错了；只是各种操纵的目的逃不过我。

经过一段时间的研究，这是我的版本：

void quick_sort(int *a, int first, int last)
   {
   int i,j,pivot,temp;

   if(last <= 1)
       return;

   pivot = a[first + last/2];

   j = first + last/2;
   temp = a[first];
   a[first] = a[j];
   a[j] = temp;

   j = first;
   for(i = first+1; i < first+last; i++)
      {   
      if(a[i] < pivot)
         {
         j++;
         temp = a[i];
         a[i] = a[j];
         a[j] = temp;
         }
      }

   temp = a[first];    
   a[first] = a[j];
   a[j] = temp;

   quick_sort(a, first, j-first);
   quick_sort(a, j+1, first+last-j-1);
   return;
   }

工作扰流测试用例here。

Answer 2

我会尝试：

quick_sort(a,first,i-1);
quick_sort(a,j+1,last);

代替：

quick_sort(a,0,i-1);
quick_sort(a,j+1,9);

它至少允许排序适用于不同于 10 个值的列表....

您还需要在每次更改后检查 i != j 是否存在，所以我认为这在您的代码中也存在问题。或者至少使用 i 而不是 j 并且 while 永远不会结束。