Whats wrong w/ my loop?? Please help?

Hey Guys,
This is my function..when I put in 'b' or 'c' or 't' it works just fine..type in 'j' and the output looks like this..Why is it doing that..help please! Thanks
Aspan

Output:
What kind of car would you like to park?
(C)ar or (B)us or (T)ruck:j
What kind of car would you like to park?
(C)ar or (B)us or (T)ruck:
What kind of car would you like to park?
(C)ar or (B)us or (T)ruck:

CODE:
char inputCar(void)
{

char c;
char car;


printf("What kind of car would you like to park?\n");
printf("(C)ar or (B)us or (T)ruck:");
scanf("%c", &car);
c = toupper(car);

while(!(c == 'C' || c == 'B' || c == 'T'))
{
printf("What kind of car would you like to park?\n");
printf("(C)ar or (B)us or (T)ruck:\n");
scanf("%c", &car);
c = toupper(car);
}


return c;
}

What happens if you fflush() the stdin after each scanf()? Maybe it interprets the "enter" as another key or something like that? =) Actually I have no clue so it's just a thought.

The problem is not you. It's with scanf(). Try getting it as a string with fgets, then either use strstr(), !strmp(), or extract the char from the string and compare as you did. The code below worked.

Code:

---




char inputCar(void)
{
char c;
char car[5];

do{

printf("What kind of car would you like to park?\n");
printf("(C)ar or (B)us or (T)ruck:\n");
fgets(car, 5, stdin);
c = toupper(car[0]);

}while(c != 'C' && c != 'B' && c != 'T');

return c;
}

---

Sparks suggestion worked.

Code:

---




char inputCar(void)
{
char c;
char car;

do{
fflush(stdin);
printf("What kind of car would you like to park?\n");
printf("(C)ar or (B)us or (T)ruck:\n");
scanf("%c", &car);
c = toupper(car);
}while(c != 'C' && c != 'B' && c != 'T');

return c;
}

---

Quote:

---

What happens if you fflush() the stdin after each scanf()? Maybe it interprets the "enter" as another key or something like that? =) Actually I have no clue so it's just a thought.

---

Quote:

---

Sparks suggestion worked.

---

fflush(stdin) is bad coding. This only gets highlighted like, once a day! :)

Use a portable solution like:

Code:

---

...
scanf("%c", &car);
while (getchar() != '\n');
...

---

Quote:

---

Originally posted by Hammer


fflush(stdin) is bad coding. This only gets highlighted like, once a day! :)

---

Damn, I knew my book about C/C++ would be crap. :)

This is how the author suggests to do keyboard input:

Code:

---

printf( "Please enter two numbers: ");
scanf( "%d %d", &num1, &num2);
fflush( stdin );

---

Lucky me that I didn't actually read this book.

Quote:

---

Lucky me that I didn't actually read this book.

---

Burn it.... burn it now!!! :D