Thread: cin.ignore()??

Hi, guys.I am new to c++ programming and I've got a question about the cin.ignore() function I found in a program that displays the number of digits in a number entered by the user.
This is the code:

Code:

#include <iostream>
using namespace std;

int main()
{
    unsigned int n;
    int count = n;
while (true)
  {
 
    cout << "Please enter an integer: ";

  
    if (cin >> n) break;

  
    cin.clear();

    cin.ignore( 1000, '\n' );
 // I don't understand what's in the braces: 1000 and '\n' :confused:
  
    cout << " It's not correct! ";
  }

    cout << " You entered the number " << n << ".\n";
    while ( n > 0 )
    n /= 10,count ++ ;
    cout << "Your number has " << count << " digits.";

    return 0;
}

I really hope you can explain me that line with cin.ignore() ( I know that adding cin.ignore clears the new line from the stream but what is with 1000? )and when is the perfect time to use it and as well the while(true) loop used in the program.
I really want to understand these things so please explain me .

Thanks in advance !

I think I know the '\n' in the cin.ignore() but 1000?

That means it can ignore up to a 1000 characters in the stream.
I heard that this isn't a good practice(Why...anyone?).There is a maxsize() function that can be put there which does what you want to do (Look up a reference for details).

Originally Posted by Ushakal

It might be because the stream doesn't contain 1000 characters, so there might be issues trying to read and discard that many characters. It may be better to use cin.sync(), which effectively discards all unread characters in the stream (istream::sync - C++ Reference).

I see, thanks for that information.

Originally Posted by manasij7479

It is unwise to think in this haphazard manner ! [EDIT] Don't edit out pieces from your post.

Yeah, sorry, I tried to edit out before anyone read it. :P I tested it myself and it seemed to work (and you're right, of course, about initialising count to 0). I didn't realise it would work like that, since the extraction operator returns a reference the stream object itself.

Thank you for replies. So should I put something else in 1000 ' s place ? A number < 1000 and the program will work,right?
About that ( if ( cin >> n ) break; ), I never used it this way. It's the same with this : cin >> n (after the cout) break; .I 've compiled it and it works.
The while(true)! Could you explain this to me too? I didn't meet it so often in programs.

Originally Posted by Sky_Daughter

Thank you for replies. So should I put something else in 1000 ' s place ? A number < 1000 and the program will work,right?
About that ( if ( cin >> n ) break; ), I never used it this way. It's the same with this : cin >> n (after the cout) break; .I 've compiled it and it works.
The while(true)! Could you explain this to me too? I didn't meet it so often in programs.

You can do something like:

Code:

cin.ignore(cin.rdbuf()->in_avail);

rdbuf returns a pointer to the underlying stream buffer, and in_avail returns the number of characters available to read. That should discard every character in the buffer.

while(true) is an infinite loop, which is a loop that doesn't end until interrupted by either returning from the function or using the keyword break. I think someone said it was slightly more efficient to use for( ; ; ) to make an "infinite" loop in another thread, because while(true) checks the truth value of its parameter every loop, even though it's always true. I imagine it's only a very slight performance boost though.

Originally Posted by Ushakal

Yeah, sorry, I tried to edit out before anyone read it. :P I tested it myself and it seemed to work (and you're right, of course, about initialising count to 0). I didn't realise it would work like that, since the extraction operator returns a reference the stream object itself.

There's an implicit conversion from stream -> void* that happens whenever you use a stream in a bool context (such as here), which is "false" if the stream is in a fail state and "true" if the stream is in a good state.

Originally Posted by tabstop

There's an implicit conversion from stream -> void* that happens whenever you use a stream in a bool context (such as here), which is "false" if the stream is in a fail state and "true" if the stream is in a good state.

Ah, I see. Thanks for the info.

Originally Posted by Elysia

Ushakal: in_avail is a member function; you forgot the ().

Yeah, sorry, my mistake.

Originally Posted by Prelude

In theory, sure. In practice, it's dicey because in_avail() isn't required to return a meaningful result. In fact, at least one popular implementation always returns 0. The most common suggestion for clearing an interactive stream buffer is:

Code:

#include <iostream> // For cin
#include <ios>      // For streamsize
#include <limits>   // For numeric_limits

std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

I've delved deeply into "flushing" the input stream and posted my results here. Several common yet incorrect solutions are described.

Thanks for the article; it was very informative, thanks. I only just stopped using system("PAUSE") a few days ago myself, after I learned how bad that was in another thread.