Question

所以我在程序中使用了quicksort，但现在想把我的复杂性降低到O（n）。我需要使用桶排序来实现这一点。

我的程序

我的程序读入一个整数文件，以及文件中的整数，并输出文件中最小的数字，该数字超过文件中至少90％的数字。

我确实设法使用quicksort让这个工作。但是，我没有使用存储桶排序获得正确的输出，我不知道为什么，因为我的代码似乎是正确的。

运行时出现分段错误

我的Bucket sort＆amp; amp;的代码输出结果

void Bucket_Sort(int array[], int n)
{   
 int i, j;   
 int count[n];  
 for(i=0; i < n; i++)
 {   
  count[i] = 0;   
 }     
 for(i=0; i < n; i++)
 {    
  (count[array[i]])++; 
 }     
 for(i=0,j=0; i < n; i++)
 {   
  for(; count[i]>0;(count[i])--) 
  {       
   array[j++] = i; 
  }  
 }   
}    Bucket_Sort(array, numberOfNumbers);   

     //Output the lowest number in the file which exceeds at least 90% of the numbers in the file.
     for (count = floor(0.9 * numberOfNumbers); count < numberOfNumbers; count ++)
     {
      if (array[count] != array[count + 1])
     {
      output = array[count];
      break;    
     }  
     }  
      printf("The outputs is : "); 
      printf("%d \n", output);

我的程序输出编译，但运行时出现分段错误。

关于我在BucketSort中做错了什么的想法？

谢谢，

丹尼尔

Answer 1

如果n <20且array包含的数字高达703K

，这两行就会出现问题

int count[n];  
(count[array[i]])++;

你正在写waaaay越界。

试试这个：

void Bucket_Sort(int array[], int n)
{
    int i, j;
    int *count = NULL;

    // find largest
    int mymax = array[0]+1;
    for (i=1; i<n; ++i)
    {
        if (mymax < (array[i]+1))
            mymax = array[i]+1;
    }

    // allocate and zero-fill a proper-sized array
    count = calloc(mymax, sizeof(*count));

    for(i=0; i < n; i++)
        (count[array[i]])++;

    for(i=0,j=0; i < mymax; i++)
    {
        for(; count[i]>0;(count[i])--)
            array[j++] = i;
    }
    free(count);
}

Bucket Sort - 分段错误

1 个答案: