Thread: Clueless for a portion of code.

scanf() leaves everything it cannot assign to a variable in the input buffer.

Suppose user enters A. scanf() will fail to assign that to num (mismatched type?) and leave it in the buffer. On the next iteration of the loop, that A (okay, "A\n") is still in the buffer and scanf() will fail to assign it to num yet again, and so on and so forth, ad nauseum.

Flush() simple retrieves characters from the buffer until the character retrieved is a newline. The effect is that you get a clear buffer. But it can fail if there nothing in the buffer when you call it.

The do-while loop is not the best way to fetch input. Consider:

Code:

printf("Enter DVD rating (1 - 10): ");
while ( scanf("%d", &num) != 1 || (num < 1 || num > 10)) {
  printf("Enter DVD rating: ");
  Flush();
}

Nice condition for the loop...... NOT .

Originally Posted by msh

Mine?

No, the OP's. It is doing the exact opposite of what I suspect he is trying to do. Hence the Borat "NOT" joke

EDIT: Nevermind, I'm an idiot. I should remember to sleep at least a few hours before posting advice.