С ++ вектор плохой доступ - PullRequest
0 голосов
/ 17 марта 2011

Привет, у меня есть следующая программа на С ++:

#include <iostream>
#include <fstream>
#include <vector>
#include <sstream>
#include <boost/foreach.hpp>
#include <stdexcept>
#include <boost/flyweight.hpp>
#include <boost/lexical_cast.hpp>
#include <boost/filesystem.hpp>

namespace fs = boost::filesystem;

struct entry
{
int file;
std::vector<double> a;
};


void my_file(const std::string&file, std::vector<entry> &data, int i){
try{
    std::ifstream in(file.c_str());
    entry e;
    std::string line;
    e.file = i;
    while(getline(in,line)){
        try{
            data[i].a.push_back( boost::lexical_cast<double> (line));
        }catch(boost::bad_lexical_cast bad){
            //std::cerr << bad.what() << std::endl;
        }
    }
}catch(std::runtime_error err){
    std::cerr << err.what() << std::endl;
}

}

void write_file(const std::string &file,std::vector<entry> data,const char* t_path){
try{
    std::string new_file = t_path ;
    new_file = new_file + "/" + file;
    std::ofstream f(new_file.c_str());

    for(size_t i = 0 ;i < data[1].a.size();i++){
        std::cout << "i: " << i;
        for(size_t j = 1; j < data.size();j++){
            std::cout << "j: " << j << std::endl;
            f << data[j].a[i]<< "\t";
        }
        f << "\n";
    }

}catch(std::runtime_error err){
    std::cerr << err.what()<< std::endl;
}
}


int collect_peak(const char*argv,const char*out){
std::cout << "collecting peaks\n";
std::stringstream sstr(argv);
std::string _line;
int c = 0;
std::vector<std::string> files;

while (getline(sstr,_line)){
    std::cout << _line << std::endl;
    fs::path p(_line);
    std::string tmp = p.parent_path().string() +"/_peak_" +      p.filename().string();
    files.push_back(tmp);
    c++;
}

std::cout << "c: " << c << std::endl;
std::vector<entry> data;
for (int i=0 ; i < files.size() ;++i){
    std::cout << files[i] <<std::endl;
    my_file(files[i],data,i);
}
write_file("__peak.txt",data,out);
return 0;

}

Почему-то это всегда дает мне плохой доступ в методе my_file. Программа должна на самом деле следующее:

  1. чтение нескольких файлов с заголовком и десятью двойными символами, разделенными новой строкой
  2. вывести все в один файл, чтобы он выглядел так:

    1. файл \ t 2. файл \ t 3. файл \ t ...

    2. файл \ t 2. файл \ t 3. файл \ t ...

    3. файл \ t 2. файл \ t 3. файл \ t ...

    4. файл \ t 2. файл \ t 3. файл \ t ...

    5. файл \ t 2. файл \ t 3. файл \ t ...

На самом деле это уже сработало, но я снова использовал его в другой программе. Есть идеи?

Спасибо

1 Ответ

5 голосов
/ 17 марта 2011

Эта строка:

std::vector<entry> data;

создает пустой вектор, который вы передаете my_file и получаете доступ к data[i] внутри.Вам нужно зарезервировать пространство для элементов, прежде чем вы сможете получить доступ к произвольным индексам в векторе.Например:

std::vector<entry> data(maxItems);
...