合并排序代码中的分段错误

Question

我已经用 C++ 编写了这个合并排序程序，但在运行代码后出现“分段错误（核心转储）”错误。即使没有编译错误。你能告诉我我在做什么错吗？在数组中输入时，它显示该错误。如果我将其更改为 push_back，则输入很好，但稍后在合并功能中，它会显示相同的错误。

//merging 2 sorted subarrays.
#include <iostream>
#include <vector>
using namespace std;

void merge(vector <int> &a,vector <int> &b,vector <int> &c)
{
    int i=0,j=0,k=0,bL=b.size(),cL=c.size();
    while(i<bL && j<cL)
    {
        if(b[i]<c[j])
        {
            a[k]=b[i];
            i++;k++;
        }
        else
        {
            a[k]=c[j];
            j++;k++;
        }
    }
    while(i<bL)
    {
        a[k]=b[i];
        i++;k++;
    }
    while(j<cL)
    {
        a[k]=c[j];
        j++;k++;
    }
    cout<<"array a inside merge is: "<<endl;
    for(int p=0;p<a.size();p++)
    {
        cout<<a[p]<<endl;
    }

}
void mergeSort(vector <int> &a)
{
    vector <int> l, r;
    int mid;
    if(a.size()<2) return;

    mid = a.size()/2;
    for(int i=0;i<mid;i++)
    {
        l[i]=a[i];
    }
    for(int i=mid;i<a.size();i++)
    {
        r[i-mid]=a[i];
    }
    mergeSort(l);
    mergeSort(r);
    merge(a, l, r);
}
int main()
{
    int n;
    vector <int> a;
    cin>>n;
    for(int i=0;i<n;i++)
    {
        cin>>a[i];
    }
    mergeSort(a);
    for(int p=0;p<n;p++)
    {
        cout<<a[p]<<endl;
    }
    return 0;
}

Answer 1

无论您使用[] 访问向量中的元素，都可能会遇到段错误。 这段代码，

vector <int> a;

给你一个空向量。

如果a 中还没有任何内容，则请求a[0] 将不起作用。 尝试将 a[0] 设置为一个值也不起作用。 它不存在。 然而。

你在mergeSort使用的时候也有类似的问题

vector <int> l, r;

这些也是空向量。

您可以使用push_back（甚至emplace_back）添加新元素。 或者使用constructor overload 来说明你想要多少元素。 例如，

vector <int> a(10);

给你一个十个整数的向量，所以a[0] 可以读取或写入。 a[11] 不是。

先练习使用向量，然后尝试合并排序。

Answer 2

获得分段错误的原因是您访问了不存在的内存位置（更准确地说是未分配）。假设您有一个长度为 3 的向量并且您尝试访问第 4 个位置，您会遇到分段错误。

与@doctorlove 的回答相反，我想说可以使用[]。但是，您需要以下实现（仅针对main() 显示，请在其他功能中使用相同的逻辑实现）。有关详细信息，请参阅std::vector 的文档。

int main()
{
    size_t n;

    std::cin >> n;
    std::vector <int> a(n);

    for(int i=0;i<n;++i)
    {
        std::cin >> a[i];
    }

    // mergeSort(a);

    for(int i=0;i<n;++i)
    {
        std::cout << a[i] << "\n";
    }
    return 0;
}

希望这会有所帮助。干杯。

Answer 3

这是修改后的最终代码：

//merging 2 sorted subarrays.
#include <iostream>
#include <vector>
using namespace std;

void merge(vector <int> &a,vector <int> &b,vector <int> &c)
{
    int i=0,j=0,k=0,bL=b.size(),cL=c.size();
    while(i<bL && j<cL)
    {
      if(b[i]<c[j])
      {
        a[k]=b[i];
        i++;k++;
      }
      else
      {
        a[k]=c[j];
        j++;k++;
      }
    }
    while(i<bL)
    {
      a[k]=b[i];
      i++;k++;
    }
    while(j<cL)
    {
      a[k]=c[j];
      j++;k++;
    }
    cout<<"array a inside merge is: "<<endl;
    for(int p=0;p<a.size();p++)
    {
      cout<<a[p]<<endl;
    }

}
void mergeSort(vector <int> &a)
{
    vector <int> l, r;
    int mid;
    if(a.size()<2) return;

    mid = a.size()/2;
    for(int i=0;i<mid;i++)
    {
      l.push_back(a[i]);
    }

    //change2
    for(int i=0;i<a.size()-mid;i++)
    {
      r.push_back(a[mid+i]);
    }
    mergeSort(l);
    mergeSort(r);
    merge(a, l, r);
}
int main()
{
    int n;
    cin>>n;

    //change1
    vector <int> a(n);
    for(int i=0;i<n;i++)
    {
      cin>>a[i];
    }
    mergeSort(a);
    cout<<"Final array is:"<<endl;
    for(int p=0;p<n;p++)
    {
      cout<<a[p]<<endl;
    }
    return 0;
}