【问题标题】:why is there a Segmentation Fault with the storing of this vector?为什么存储此向量时会出现分段错误?
【发布时间】:2013-11-06 02:30:34
【问题描述】:

我在看似不错的代码中有分段错误。我知道故障发生在哪里,但似乎无法修复它。

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;
}

【问题讨论】:

  • 什么是 position[i] 永远是 0?
  • 与输入文件,position[0]=1
  • 您是否确实验证了position 中的所有 值实际上都大于零?我不是指您正在加载的文件中的值,而是position 的实际内容
  • 这里是正在读入的文件,也许这会有所帮助。 '9 约翰 3 简 7 汤姆 2 山姆 6 汤姆 1 妮可 5 汤姆 4 简 8 本'

标签: c++ vector segmentation-fault


【解决方案1】:

如果不编译和测试我的答案,我认为为了正确使用“有序”向量的赋值,您必须确保0 &lt;= position[i]-1 &lt; ordered.size() 始终如此。因为 'ordered' 开始时是空的,所以您正在尝试越界访问。见this question/ answer

因此,您可能需要考虑使用另一个修饰符成员函数,例如“insert”或“push_back”来避免越界问题。尽管同时,您可能希望更改存储数据的方式,因为您试图依靠“有序”向量的索引值来表示某个整数键/值。

【讨论】:

    猜你喜欢
    • 2021-12-25
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2021-07-30
    • 2011-04-23
    相关资源
    最近更新 更多