2

我尝试在 MacOSx 的 Xcode11 上使用递归程序运行我的合并排序算法，但出现以下错误 - 线程 1：EXC_BAD_ACCESS (code=2, address=0x7ffeef3ffff8)

我无法找到解决方案。请在下面找到我使用的代码。

#include <iostream>
#include <vector>

using namespace std;

void printArray(vector<int> arr){
    for(int x: arr){
        cout<<x<<" ";
    }
    cout<<endl;
}

vector<int> mergeSort(vector<int> arr){
    vector<int> B,C;
    unsigned long Na = arr.size();
    unsigned long Nb, Nc;
    Nb = Na/2 + (Na%2);
    Nc = Na - Nb;
    for (int i =0;i<Na;i++){
        if (i<Nb)
            B.push_back(arr[i]);
        else
            C.push_back(arr[i]);
    }
    B = mergeSort(B);
    C = mergeSort(C);
    int i=0,j=0;
    for(int k=0;k<Na;k++){
        if(B[i]<C[j]){
            arr[k] = B[i];
            i++;
        }
        else if(C[j]<=B[i]){
            arr[k] = C[j];
            j++;
        }
    }
    return arr;
}

int main() {
    vector<int> A{5,4,1,8,7,2,6,3};
    cout<<"Unsorted Array: "<<endl;
    printArray(A);
    A = mergeSort(A);
    cout<<"Sorted Array: "<<endl;
    printArray(A);
    return 0;
}
4

1 回答 1

2

你有无限递归。您的mergeSort函数总是调用自身（两次），导致堆栈溢出。

至少你应该放一些类似的东西

if (arr.size() <= 1) return arr;

在您的功能开始时。但你也可能有其他问题。

我用调试器花了大约两分钟才发现这个问题。我运行了你的代码，直到它崩溃，检查了调用堆栈，并立即看到了无限递归。您应该学习如何使用调试器。当您拥有此技能时，编程会容易得多。

感谢提供一个mre。

于 2020-05-19T16:46:49.703 回答