c++ - 为什么存储此 vector 会出现段错误？

Question

我在看似不错的代码中有一个段错误。我知道故障出在哪里，但似乎无法修复它。

for(int i=0; i<position.size();i++)
    {
            ordered[position[i]-1]= name[i];
      }

这就是错在哪里该代码应该读取具有相应编号的名称文件，然后按编号顺序对名称进行排序。这是供引用的完整代码:

#include<iostream>
#include<string>
#include<vector> 
#include<fstream>
#include<sstream>
#include<algorithm>
using namespace std;

void print_vector(vector<string> ordered){
    for(int i = 0; i < ordered.size(); i++)
            cout << ordered[i] << " ";
    cout << endl;
}
int main() 
{
    ifstream inf;
    inf.open("input2.txt");
    string s;
    string word;
    vector<int> position;
    vector<string> name;
    vector<string> ordered;
    string n;
    int p;

    while( !inf.eof())
    {

            getline(inf, s);
            istringstream instr(s);
            instr>>p;
            instr>>n;
            while(!instr.eof()){
                    position.push_back(p);
                    name.push_back(n);
                    instr>>p;
                    instr>>n;
            }

    }
    for(int i=0; i<position.size();i++)
    {
            ordered[position[i]-1]= name[i];


    }
    print_vector(ordered);
    inf.close();
    return 0;
}

Answer 1

在不编译和测试我的答案的情况下，我认为为了正确使用 vector “有序”赋值，你必须确保 0 <= position[i]-1 < ordered.size()总是这样。因为“已订购”一开始是空的，所以您正试图越界访问。参见 this question/ answer .

因此，您可能要考虑使用另一个修饰符成员函数，例如 'insert' 或 'push_back' 来避免越界问题。尽管与此同时，您可能想要更改存储数据的方式，因为您正试图依赖“有序” vector 的索引值来表示一些整数键/值。