Infinite Loop of Doom!

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)

---

Ok, the probem is with scanf("%d", &T) == 1).
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

Quote:

---

Originally Posted by matsp

Because scanf will NEVER read anything other than what it accepts as valid input.

--
Mats

---

If I input an integer I can still read another valid input.
FAQ it is.

Quote:

---

Originally Posted by jordanguyoflove

If I input an integer I can still read another valid input.
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

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