C++ 代码转储“随机”数据

Question

首先我会说我是一个相当新的 C++ 程序员。 不过我懂 PHP 和 VBA，所以对编程基础的各个方面都有很好的了解。

因为我在日常工作中经常使用 CSV，所以我认为编写一个操作 CSV 文件的库是一个很好的学习练习。

我写了这个函数：

int getHeaders(ifstream & os, vector<string> & head2){
    string STRING;
    getline(os,STRING);
    cout << STRING << endl;
    STRING.erase(remove(STRING.begin(), STRING.end(), '\"'), STRING.end());
    string::iterator it = STRING.begin();
    int x = 0;
    for (int index = 0; it < STRING.end(); it++, index++) {
        if (*it == ',') {
            head2.push_back(STRING.substr(0,index));
            STRING.erase(0,index+1);
            cout << endl << head2[x];
            cout << endl << STRING;
            x++;
        }
    }
    return head2.size();
}

由以下调用：

int addRowCount = 0;
vector<string> head1;

ifstream outfile;
outfile.open("default.csv", ios_base::app);

cout << getHeaders(outfile, head1) << endl;
cout << head1[0] << endl << head1[1] << endl;

但是当我运行程序时，程序只是将一堆随机垃圾转储到控制台（并使应用程序崩溃） 我使用的是 windows，所以不能使用 valgrind。

有谁知道为什么会发生这种情况？显然这个“转储”不是我希望应用程序做的。我希望有人能指出我的代码中会导致这种情况发生的部分。

提前致谢。

Answer 1

当您对字符串调用擦除时，该字符串的迭代器将失效，因此在调用STRING.erase() 之后使用it 是错误的。

提示：当您查看支持迭代器的类的方法的文档时，请留意有关使迭代器无效的说明。例如，在this page 上，阅读标题为Iterator validity 的部分

[与答案无关，而是样式问题：在 C 和 C++ 中，对像 STRING 这样的变量名使用全大写通常被认为是不好的样式。所有大写名称都用于#defined 符号]

Answer 2

所以结果表明没有设置 head2[1]，所以这是某种内存泄漏。

这是对 cme​​ts 部分中的建议进行一些进一步修改后完成的函数：

int getHeaders(ifstream & os, vector<string> & head2){
    string STRING;
    getline(os,STRING);
    STRING.erase(remove(STRING.begin(), STRING.end(), '\"'), STRING.end());
    int strle = count(STRING.begin(),STRING.end(), ',') + 1;
    for(int x = 0; x != strle; x++){
        if (count(STRING.begin(), STRING.end(), ',') > 0) {
            head2.push_back(STRING.substr(0,STRING.find_first_of(',')));
        } else {
            head2.push_back(STRING.substr(0,STRING.length()));
        }
        STRING.erase(0,STRING.find_first_of(',')+1);
    }
    return head2.size(); 
}