tags:

views:

142

answers:

5

Here is the code:

string str;
cin>>str;
cout<<"first input:"<<str<<endl;
getline(cin, str);
cout<<"line input:"<<str<<endl;

The result is that getline never pauses for user input, therefore the second output is always empty.

After spending some time on it, I realized after the first call "cin>>str", it seems '\n' is still stored in cin (using cin.peek() to check), which ends getline immediately. The solution will be adding one more line between the first usage and the second one: cin.ignore(numeric_limits::max(), '\n');

However, I still don't understand, why is '\n' left there after the first call? What does istream& operator>> really do?

+1  A: 

By default, the stream insertion operator reads until it sees whitespace. Your first call isn't returning until it sees a space, tab, newline, etc. Then the next character needs to be consumed so that you can get to the next one.

jwismar
+4  A: 

The \n is left in the input stream as per the way operator>> is defined for std::string. The std::string is filled with characters from the input stream until a whitespace character is found (in this case \n), at which point the filling stops and the whitespace is now the next character in the input stream.

You can also remove the \n by calling cin.get() immediately after cin>>str. There are many, many different ways os skinning this particular I/O cat, however. (Perhaps a good question in and of itself?)

fbrereto
A better way of getting rid of trailing whitespace characters would probably be to do `std::cin >> str >> std::ws;`.
sbi
Run some tests, realizing it is not the problem of how it is defined for <code>std::string</code>. In fact, operator>> leaves the newline in the stream whatever variable is following it. But thanks for starting the discussion of getting rid of white space!
gc
A: 

Good writeup that explains some of the reasons why you are running into this issue, primarily due to the behavior of the input types and that you are mixing them

curtisk
A: 

As others have said, th problem is that the newline is left from the first extraction. One solution I do is to discard all the left characters in the stream:

std::cin.ignore( std::numeric_limits<std::streamsize>::max() );
AraK
+1  A: 

I generally recommend only doing line-oriented input from std::cin. So, your code could look something like this:

string str;
int val;
// Read an entire line and parse an integer from it
{
    string line;
    getline(cin, line);
    istringstream iss(line);
    iss >> val;
}
cout<<"first input:"<<val<<endl;
getline(cin, str);
cout<<"line input:"<<str<<endl;

Be sure to add error checking too.

The getline-only approach avoids having to think about line buffering of the input, of clearing the input, etc. If you directly read something with >>, the input does not terminate if the user hits enter instead of inputting what is required, but instead continues until a token is input (this behavior is usually not wanted).

Tronic