我目前正在尝试学习快速排序,这是我的代码:
#include<iostream>
#include<vector>
#include<cmath>
using namespace std;
void swap(int *a, int *b)
{
int t = *a;
*a = *b;
*b = t;
}
int partition(vector<int> &A, int low, int high)
{
int pivot = A[low];
int i = low;
int j = high;
while(i < j){
do
{
i++;
}while(A[i]>=pivot);
do
{
j--;
}while(A[j]<pivot);
if(i < j)
{
swap(A[i], A[j]);
}
}
swap(A[low], A[j]);
return j;
}
void QuickSort(vector<int> &A, int low, int high)
{
int j = partition(A, low, high);
QuickSort(A, low, j);
QuickSort(A, j+1, high);
}
int main()
{
vector<int> A{-7, 11, -3, 3, 2};
QuickSort(A, 0, A.size()-1);
for(int i:A)
{
cout << i << endl;
}
}
代码运行后,我不断遇到分段错误(核心已转储),如何解决此错误。 另外,任何人都可以推荐一个好的c ++调试器。非常感谢
答案 0 :(得分:1)
您的QuickSort
函数具有无限递归。每当它被调用时,它都会自行调用,并且没有任何条件可以中断周期。
此外,您的swap
函数不起作用。在编写时,A
库中的值将提供给函数并解释为地址。那不应该编译。它可以编译的唯一原因是您不在程序中使用该函数。您使用std::swap
是因为您已经完成using namespace std;
,所以请不要这样做。
您的swap
函数应通过引用获取参数,并且您需要在QuickSort
函数中添加条件。
我不确定您尝试实施哪种分区方案,因此我做了一些更改以使其按照Hoare partition scheme进行工作。
#include <iostream>
#include <vector>
void swap(int& a, int& b) { // take arguments by reference
int t = a;
a = b;
b = t;
}
size_t partition(std::vector<int>& A, size_t low, size_t high) {
int pivot = A[(high + low) / 2];
size_t i = low;
size_t j = high;
while(true) {
while(A[i] < pivot) ++i;
while(A[j] > pivot) --j;
if(i >= j) return j;
swap(A[i], A[j]);
++i;
--j;
}
}
void QuickSort(std::vector<int>& A, size_t low, size_t high) {
if(low < high) { // added condition
size_t j = partition(A, low, high);
QuickSort(A, low, j);
QuickSort(A, j + 1, high);
}
}
int main() {
std::vector<int> A{-7, 11, -3, 3, 2};
QuickSort(A, 0, A.size() - 1);
for(int i : A) {
std::cout << i << '\n';
}
}