运行合并排序递归算法时EXC_BAD_ACCESS错误

EXC_BAD_ACCESS error while running merge sort recursion algorithm

本文关键字:BAD ACCESS 错误 EXC 合并 排序 递归算法 运行      更新时间:2023-10-16

我尝试在 MacOSx 的 Xcode11 上使用递归程序运行我的合并排序算法,但出现以下错误 -线程 1:EXC_BAD_ACCESS(代码 = 2,地址 = 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;
}

你有无限递归。您的mergeSort函数总是调用自身(两次(,从而导致堆栈溢出。

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

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

在函数的开头。但你也可能有其他问题。

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

不过,为提供 MRE 而点赞。