无法将向量<string> 输出到文件

Question

我是 C++ 新手。我无法将数据输出到文件。我正在使用迭代器打印出地图。 print 方法接受 i，一个键值，并打印出它对应的向量。现在，当我使用 cout

    public: void print(int i, vector<string> in, ostream& outfile) // print method for printing a vector and it's key
{

    sort(in.begin(), in.end()); // sort the vector alphabetically first

    vector<string>::iterator it; 

    it= unique(in.begin(), in.end()); // makes sure there are no duplicate strings

    in.resize( distance(in.begin(),it) );

    for( it = in.begin(); it != in.end(); it++ ) // iterate through it

    cout << i << ": "<< *it<<endl; // and print out the key value and each string in the vector
   // outfile<< i << ":" << *it<< endl; // prints to file
}

Answer 1

您是否同时使用cout 行？如果是这样，我想我知道它是什么。

for 循环，不带大括号将执行下一条语句作为其循环体。如果您同时使用cout 行和outfile 行，您将打印所有内容，然后在循环之后，it 将位于数组末尾的后面。然后，您尝试取消引用它并将其写入文件，这当然会失败，因为您取消引用无效的迭代器。

简短的回答，用大括号将语句包装在你的 for 循环中。

例如，您有以下内容（正确缩进时）：

for( it = in.begin(); it != in.end(); it++ ) // iterate through it
    cout << i << ": "<< *it<<endl; 
outfile<< i << ":" << *it<< endl; // prints to file

在最后一行，it = in.end()，其中in.end() 是元素刚刚过去向量的末尾。然后，您尝试访问该位置不存在（并且无效）的元素，因此它失败了。相反，您需要将其移动到循环内，该循环应为

for( it = in.begin(); it != in.end(); it++ ) // iterate through it
{
    cout << i << ": "<< *it<<endl; // and print out the key value and each string in the vector
    outfile<< i << ":" << *it<< endl; // prints to file
}

Answer 2

@slugonamission 已经给了你正确的答案，所以我只想指出你的函数实际上可以被简化并且更不容易出错。我写这个作为答案只是因为需要代码格式化，否则我会在评论中发布：

void print(int i, vector<string> v, ostream& o)
{
    sort(begin(v), end(v));
    v.erase(unique(begin(v), end(v))); // Do this in one step, no iterator mess
    for (auto const& s : v) // Avoid manual iteration cycles if you can
    {
        o << i << ":" << s << endl;
        ...
    }
}

编辑：

正如@juanchopanza 正确指出的那样，实现相同目标的更快方法是将矢量的内容传输到关联容器中以确保唯一性。这将允许您通过const & 传递向量：

void print(int i, vector<string> const& v, ostream& o)
{
    unordered_set<string> c(begin(v), end(v));
    for (auto const& s : c)
    {
        o << i << ":" << s << endl;
    }
}