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

如果没有编译和测试我的答案，我认为为了正确使用向量'ordered'，你必须确保0 <= position[i]-1 < ordered.size()始终如此。因为'ordered'开始是空的，所以你试图访问越界。请参阅this question/ answer。

因此，您可能需要考虑使用其他修饰符成员函数（如“insert”或“push_back”）来避免出界限问题。虽然，与此同时，您可能希望更改存储数据的方式，因为您尝试依赖“有序”向量的索引值来表示某个整数键/值。

为什么存在这个向量的分段错误？

1 个答案: