Hi i have a question. Im using Dev C++ and this simple program takes in values of x and outputs *prints them on the screen. Although x should be an integer If i put in for example the letter "u" or any letter for that matter it goes on and on without asking me for anohter input and uses the last input on the screen. Why?

#include <iostream>
#include <stdio.h>
using namespace std;
main()
{
int x=0;
while (true)
{
scanf("%d",&x); 
printf("%d", x);
}

}

since you declare x as an integer so system allocate 2 byte for this variable & when u give input as character it converts into an integer through ASCII value & print that one.............

The above poster is not exactly correct.

When you call scanf(), if the function fails (which will happen when you try to read an integer but it is given a character), it will not actually consume the input character from the input. This means the next time you call scanf() it will end up trying to read the same character again!

To catch this, you can check the return value from scanf(). It tells you the number of inputs read successfully. When there is an error it should return less than the number of variables you wanted to read in (in this case, less than 1). If this happens, you need to flush the input stream. The easiest way to do this is just by calling fflush(stdin).

Apparently fflush() is a "bad practice", but in small programs it should be fine. There are other "better" ways to deal with this.

If you're going to include the C++ headers, why not use cin and cout ?

By the way im actually programming in C for something else thats why im using it and not Cin/cout.

AND THANKS


fflush(stdin);

seems to work,
just a question, wat is the equivalent statement in C++?

I don't believe so. No reason for one.

>im actually programming in C
Then keep in mind that C89 doesn't support the <iostream> header, namespaces, or the true keyword. Your code won't compile as C, and failing to return a value from main is undefined behavior.

>wat is the equivalent statement in C++?
I thought you were programming in C. :icon_rolleyes: Anyway, the equivalent statement in C++ is fflush(stdin) . It's equally broken in both languages, I strongly recommend you forget that it happens to work on your compiler. A more correct C++ solution is istream::ignore:

#include <iostream>
#include <ios>
#include <limits>

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

And the equivalent C code uses a loop:

#include <stdio.h>

int ch;

do
  ch = fgetc ( stdin );
while ( ch != EOF && ch != '\n' );

Which can also be done in C++:

#include <iostream>

char ch;

do
  std::cin.get ( ch );
while ( std::cin && ch != '\n' );
Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.