Ok, the probem is with scanf("%d", &T) == 1).Code:int T; char file[30]; FILE *fp; if(argc <= 2) { while(1) { printf("Please enter the number of threads to be created: "); if(scanf("%d", &T) == 1) { //stuff break; }//end of if(scanf("%d", &T) == 1) else T = -2; }//end of while(true) }//end of if(argc <= 2)
T is an integer.
If I input a string instead of an int, my program goes into an inifinit loop.
WHY???
Because scanf will NEVER read anything other than what it accepts as valid input. So if you have a %d format, "abcd" as input will stay in the input buffer forever. If you want to continue reading valid data, then you need to clean the input buffer. See the FAQ for an example of how to do that.
--
Mats
Compilers can produce warnings - make the compiler programmers happy: Use them!
Please don't PM me for help - and no, I don't do help over instant messengers.
If I input an integer I can still read another valid input.Originally Posted by matsp
FAQ it is.
What is your point?
if scanf returns 0 - there is junk in the stream - get rid of it. FAQ should have samples.
if scanf returns EOF - nothing more to read - get out of your loop
All problems in computer science can be solved by another level of indirection,
except for the problem of too many layers of indirection.
– David J. Wheeler
Also improve your indentation. The if statement is borked.
A closing brace doesn't need to reference its opening control statement in a comment. It's a bad idea for at least three reasons:
1) It makes no sense to put code into comments. Use comments for explanations, not redundancy. You also wouldn't use "exit(1); // exit(1)".
2) Indentation already accomplishes the same thing for free.
3) Not everyone uses a fancy IDE with syntax highlighting and comment hiding. If comments are not distinguishable from code by their color, your code is hard to read. This will probably do more harm to you than to the other readers.
To put it short, you have more work and gain nothing.
Greets,
Philip
All things begin as source code.
Source code begins with an empty file.
-- Tao Te Chip
