c++ - vector push_back崩溃

我有以下结构。

#include <iostream>
#include <fstream>
#include <string>
#include <sstream>
#include <vector>

struct station {
    std::string id;
    std::string code;
    std::string station_name;
    station(std::vector<std::string> &in) : id(in[0]), code(in[1]),
                                            station_name(in[2]) {}
    station(): id (""), code (""), station_name(""){}

    bool operator<( const station& rhs ) const {
        return this->station_name < rhs.station_name;
    }
};
int main(int argc, char **argv) {
    std::ifstream ifs(argv[1]);
    if ( ifs.peek() == EOF )  {
        exit ( 1 );
    }
    // Read the input file and update the database
    std::string line;
    station prev, current;
    std::set<station> my_set;
    while( ifs.good()&& std::getline(ifs,line) ) {
        std::stringstream  lineStream(line);
        std::string        token;
        std::vector<std::string> input;
        while(std::getline(lineStream,token,',')) {
            input.push_back(token);
        }
        station st(input);
        my_set.insert(st);

    }
}

我正在读取具有以下格式的railway stations相关信息的文件ID,Station Code,Station Name
我正在逐行读取此文件并创建station对象，然后将其推入std::set<station>
一段时间后，在读取21448行之后，它崩溃了。我有403523行

这里有什么问题。

该程序可以在Linux上正常运行，但不能在Windows上运行

我得到debug assertion failed

最佳答案

带数组的构造函数令我担心。

编辑

基于更新的问题:

问题是采用 vector 的构造函数。
您在访问元素时无需检查它们是否存在。

因此，如果输入的任何行都是错误的(即，并非所有值都存在)，则 vector 将不会像所需的那么大，并导致不确定的行为。

如果更改这些行:

while(std::getline(lineStream,token,',')) {
            input.push_back(token);
        }

进入:

std::getline(linestream, id,   ',');
std::getline(linestream, code, ',');
std::getline(linestream, name, ',');

然后使用这些参数调用站构造函数(在OO代码中会很好)。那么您将不会看到崩溃。一些错误检查也将很好。

注意:

while( ifs.good()&& std::getline(ifs,line) ) {

无需在此处检查good()。如果流状态不佳，则getline()将不执行任何操作。流(getline()的返回值)到bool的转换还检查状态是否为流是否有效以进行进一步读取，否则将转换为false(最终)。

// This is more idiomatic.
while(std::getline(ifs,line) ) {

原版的

试试这个:

struct station
{
    std::string id;
    std::string code;
    std::string station_name;
    friend std::istream& operator>>(std::istream& stream, station& data)
    {
        std::string  line;
        std::getline(stream, line);

        std::stringstream  linestream(line);
        std::getline(linestream, data.id, ',');
        std::getline(linestream, data.code, ',');
        std::getline(linestream, data.station_name);

        return stream;
    }
};

int main()
{
    std::ifstream   file("station.txt");
    std::vector<station>  stations;

    std::copy(std::istream_iterator<station>(file),
              std::istream_iterator<station>(),
              std::back_inserter(stations)
             );
}

关于c++ - vector push_back崩溃，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/8450098/