快速排序导致计算器溢出?
Quicksort causing stackoverflow?
这是我对快速排序的实现:
int choosePivotIndex(int l, int r)
{
return l + rand() % (r + 1 - l);
}
void swap(int a[], int l, int r)
{
int tmp = a[l];
a[l] = a[r];
a[r] = tmp;
}
int partition(int a[], int l, int r)
{
int p = choosePivotIndex(l, r);
swap(a, p, r);
int d = l;
for (int i = l ; i < r ; ++i)
{
if (a[i] < a[r])
{
swap(a, i, d);
++d;
}
}
swap(a, d, r);
return d;
}
void quickSort(int a[], int l, int r)
{
if (l < r)
{
int p = partition(a, l, r);
quickSort(a, l, p - 1);
quickSort(a, p + 1, r);
}
}
void quickSort(int a[], int len)
{
srand(static_cast<unsigned int>(time(nullptr)));
quickSort(a, 0, len - 1);
}
我是这样用的:
int a[10];
quickSort(a, 10);
它似乎适用于小数组,但当我为它提供大数组(例如 300 000
元素)时,我收到堆栈溢出错误。我试图通过仅对较小的部分使用递归并在 while
循环中对较大的部分进行排序来补救它:
void quickSort(int a[], int l, int r)
{
while (l < r)
{
int p = partition(a, l, r);
if (p - l < r - p)
{
quickSort(a, l, p - 1);
l = p + 1;
}
else
{
quickSort(a, p + 1, r);
r = p - 1;
}
}
}
但我得到了相同的结果。我做错了什么吗?我怎样才能让它适用于更大的阵列?
根据评论部分的讨论,代码似乎没问题,但有问题的部分是数组在堆栈上的声明
int a[10];
对于较小的数组来说这很好,它很容易 运行 进入带有大数组的堆栈溢出,要测试这一点,您可以声明一个 int a[1000000]
的大数组,如果没有快速排序代码,您会仍然会出现堆栈溢出。因此建议通过 new
在堆上声明它们
这是我对快速排序的实现:
int choosePivotIndex(int l, int r)
{
return l + rand() % (r + 1 - l);
}
void swap(int a[], int l, int r)
{
int tmp = a[l];
a[l] = a[r];
a[r] = tmp;
}
int partition(int a[], int l, int r)
{
int p = choosePivotIndex(l, r);
swap(a, p, r);
int d = l;
for (int i = l ; i < r ; ++i)
{
if (a[i] < a[r])
{
swap(a, i, d);
++d;
}
}
swap(a, d, r);
return d;
}
void quickSort(int a[], int l, int r)
{
if (l < r)
{
int p = partition(a, l, r);
quickSort(a, l, p - 1);
quickSort(a, p + 1, r);
}
}
void quickSort(int a[], int len)
{
srand(static_cast<unsigned int>(time(nullptr)));
quickSort(a, 0, len - 1);
}
我是这样用的:
int a[10];
quickSort(a, 10);
它似乎适用于小数组,但当我为它提供大数组(例如 300 000
元素)时,我收到堆栈溢出错误。我试图通过仅对较小的部分使用递归并在 while
循环中对较大的部分进行排序来补救它:
void quickSort(int a[], int l, int r)
{
while (l < r)
{
int p = partition(a, l, r);
if (p - l < r - p)
{
quickSort(a, l, p - 1);
l = p + 1;
}
else
{
quickSort(a, p + 1, r);
r = p - 1;
}
}
}
但我得到了相同的结果。我做错了什么吗?我怎样才能让它适用于更大的阵列?
根据评论部分的讨论,代码似乎没问题,但有问题的部分是数组在堆栈上的声明
int a[10];
对于较小的数组来说这很好,它很容易 运行 进入带有大数组的堆栈溢出,要测试这一点,您可以声明一个 int a[1000000]
的大数组,如果没有快速排序代码,您会仍然会出现堆栈溢出。因此建议通过 new