如何修复C++中的分段故障(核心转储)

Question

我正在编写一个程序，该程序将两个矢量组合并排序，然后打印矢量，但我没有使用第三个矢量。相反，我将一个向量与另一个向量组合，然后对组合后的向量进行排序。但是我得到一个错误，叫做"；分段故障"；。

这是代码：

#include<bits/stdc++.h>
using namespace std;
int main() {
// your code goes here
ios_base::sync_with_stdio(0);
cin.tie(0);
int m,n; 
cin >> m >> n;
vector<int> nums1, nums2;
for(int i=0; i<m; i++) cin >> nums1[i];
for(int i=0; i<n; i++) cin >> nums2[i];
for(int i=0; i<n; i++){ // nums2 
nums1.push_back(nums2[i]); // I am adding all the elements present in nums2 into nums1
}
sort(nums1.begin(), nums1.end());
for(int i=0; i<(m+n); i++) cout << nums1[i] << " ";
return 0;
}

我得到的错误：run:第1行：3分段错误(核心转储(LD_LIBRARY_PATH=/usr/local/gcc-8.3.0/lib64./a.out

请告诉我如何修复这个错误，以及如何在将来避免它。

Answer 1

此处：

vector<int> nums1, nums2;
for(int i=0; i<m; i++) cin >> nums1[i]; // this causes undefined behavior
for(int i=0; i<n; i++) cin >> nums2[i]; // also this one

矢量没有缓冲区来存储数据，所以在使用operator[]:之前需要这样做

vector<int> nums1(m), nums2(n);
nums1.push_back(2); // will add 2 to the back of nums1 so size will become m + 1
nums2.push_back(6); // will add 6 to the back of nums2 so size will become n + 1
// you can do as many push_backs as you want until
// your computer runs out of memory

现在，两者将分别用m和n数量的元素进行初始化。

如果您使用at函数而不是[]，程序将抛出std::out_of_range异常，您会突然注意到您试图越界。但at当然是要付出性能代价的。