c++stlistringstream

Splitting a string into integers using istringstream in C++


I'm trying to use istringstream to split a simple string into a series of integers:

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

using namespace std;

int main(){

    string s = "1 2 3"; 
    istringstream iss(s);   

    while (iss)
    {
        int n;
        iss >> n;
        cout << "* " << n << endl;
    } 
}

And i get:

* 1
* 2
* 3
* 3

Why is the last element always coming out twice? How to fix it?


Solution

  • It's coming out twice because your looping is wrong, as explained (indirectly) at http://www.parashift.com/c++-faq-lite/input-output.html#faq-15.5 (while (iss) is not dissimilar from while (iss.eof()) in this scenario).

    Specifically, on the third loop iteration, iss >> n succeeds and gets your 3, and leaves the stream in a good state. The loop then runs a fourth time due to this good state, and it's not until the next (fourth) iss >> n subsequently fails that the loop condition is broken. But before that fourth iteration ends, you still output n... a fourth time.

    Try:

    #include <string>
    #include <iostream>
    #include <sstream>
    #include <vector>
    
    using namespace std;
    
    int main()
    {
        string s = "1 2 3"; 
        istringstream iss(s);   
        int n;
    
        while (iss >> n) {
            cout << "* " << n << endl;
        } 
    }