我正在尝试在 c/c++ 中实现快速排序,但每当我运行代码时,我总是收到此错误"调试断言失败_crtisValidHeapPointer(块)"。
我的代码是:
void QuickSort(int *A, int size) {
if (size < 2) return;
int *L = NULL, *R = NULL, RSize = 0, LSize = 0;
R = (int *)malloc(sizeof(int));
L = (int *)malloc(sizeof(int));
for (int i = 0; i < size; i++) {
if (A[size - 1] <= A[i])
if( ((int *)realloc(R, (sizeof(int) * (RSize + 2) ))) != NULL )
R[RSize++] = A[i];
else
if ( ((int *)realloc(L, (sizeof(int) * (LSize + 2) ))) != NULL )
L[LSize++] = A[i];
}
QuickSort(L, LSize);
QuickSort(R, RSize);
Merge(A, L, R, LSize, RSize);
free(L);
free(R);
return;
}
我知道这与我的数组 L 和 R 的内存分配有关,但我似乎无法弄清楚问题到底是什么。
编辑:找到解决方案
法典:
void QuickSort(int *A, int size) {
if (size < 2) return;
int *L = NULL, *R = NULL, RSize = 0, LSize = 0;
for (int i = 0; i < size; i++) {
if (A[size - 2] < A[i]) {
if ((R = (int *)realloc(R, (sizeof(int) * (RSize + 1)))) != NULL)
R[RSize++] = A[i];
}
else {
if ((L = (int *)realloc(L, (sizeof(int) * (LSize + 1)))) != NULL)
L[LSize++] = A[i];
}
}
QuickSort(L, LSize);
QuickSort(R, RSize);
Merge(A, L, R, LSize, RSize);
free(L);
free(R);
return;
}
问题就在这里:
if( ((int *)realloc(R, (sizeof(int) * (RSize + 2) ))) != NULL )
R[RSize++] = A[i];
realloc
函数返回新的数组指针,该指针与if
内的 NULL 进行比较,然后丢弃。 然后取消引用原始(无效)指针,从而损坏堆。
若要修复,请确保将realloc
的结果分配给指针。
if( (R = (int *)realloc(R, (sizeof(int) * (RSize + 2) ))) != NULL )
您还应该包括一些错误检查以中止循环,以防分配失败,以else
语句的形式对此if
。 如当前所述,如果realloc
失败,则循环将继续完成,在每次迭代时跳过数组,然后返回损坏的结果。
好吧,只有单个int
分配给两个变量L
和R
。但在稍后的代码中,您将许多整数作为LSize
分配,RSize
递增。好吧,我看到调用了realloc
,但结果没有分配给L
和R
所以这会导致问题。
无论如何,快速排序算法可以很好地在原始数组上运行,没有理由为其分配新的临时数组。