QuickSort java.lang.StackOverflowError

Question

我正在尝试计算快速排序算法的执行时间，在三种情况下，阵列的“中间枢轴，第一个枢轴，随机枢轴”已经排序。

mid pivot和random在任何大小都可以正常工作，但如果大小超过25000，则第一个pivot case会给出stackoverflow。

下面是代码：

static void q_sort( int left, int right)
{

int pivot;
int l_hold, r_hold;

  l_hold = left;
  r_hold = right;
  pivot = array[left];
  while (left < right)
  {
    while ((array[right] >= pivot) && (left < right))
      right--;
    if (left != right)
    {
      array[left] = array[right];
      left++;
    }
    while ((array[left] <= pivot) && (left < right))
      left++;
    if (left != right)
    {
      array[right] = array[left];
      right--;
    }
  }
  array[left] = pivot;
  pivot = left;
  left = l_hold;
  right = r_hold;
  if (left < pivot)
    q_sort(left,(int) pivot-1);
  if (right > pivot)
    q_sort( (int)pivot+1, right);
}

并且是调用代码：

double startTime1 = System.currentTimeMillis();
q_sort(0,size -1);
double stopTime1 = System.currentTimeMillis();
double elapsedTime1 = stopTime1 - startTime1;      
System.out.println(elapsedTime1);

Answer 1

你没有告诉数组是如何生成的，但我怀疑它已经被排序了。

问题如下：如果快速排序已排序的数组，第一个数据透视将导致以下递归：0..24999,1 ... 24999,2..24999,3..24999,4..24999其中占用大量的堆栈空间并导致堆栈溢出。这是因为数组为0..24999且pivot为0，然后第二个分区将变为1..24999。

你应该消除其中一个尾调用。消除哪个尾部调用取决于哪个分区更小。您希望递归处理尽可能少的数据。其中一个递归简单地由循环替换。已经在Quicksort and tail recursive optimization

解释了这种尾递归消除

无论如何，我建议使用现有的快速排序算法，而不是自己编码，除非这是一个家庭作业问题。