我看不到为什么我会遇到细分错误错误

时间:2020-06-15 19:04:50

标签: c++

我目前正在尝试学习快速排序,这是我的代码:

#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 ++调试器。非常感谢

1 个答案:

答案 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';
    }
}