Thread: fgets in switch-case problem

Hi,

I have a problem with fgets inside a case where it doesn't even allow me to enter input, it just suddenly goes into the function after or back to the 'choose' menu, can someone tell me why it's like this and how to fix it? Doesn't seem to have any problems outside fo a switch-case statement.

Code:

#define MAX_INPUT 5
char string[5];
switch (choose())
{
	case 1:
		printf("Enter no more than 5 characters:  \n");
		fgets(string, MAX_INPUT, stdin);  
		useInput(string);
		break;
 	case 2:
		funct2();
		break;
	case 3:
		funct3();
		break;

}

Yes you're right about that , guess I'll have to change it. =/

Code:

int choose(void)
{
	int option = 0;
	printf("Menu: \n"
	"1) Input 1 \n"
	"2) Implementation \n"
	"3) Directive \n"
	"4) Quit \n"
	"\n"
	"Select your option: \n");
	scanf("%d", &option);
	printf("\n");

	return option;


}

I also left out the fact that the switch case is inside a menu loop:

Code:

	while(quit != 1)
	{
              switch..
         ....
         ....
         ....
         }

I'm guessing that is probably also a big part of the problem. Thanks anyway going to try and change it, and so far when I tried to change everything I kept on getting errors though when trying to make choose work with fgets.

Originally Posted by icestorm

Yes you're right about that , guess I'll have to change it. =/

Code:

int choose(void)
{
	int option = 0;
	printf("Menu: \n"
	"1) Input 1 \n"
	"2) Implementation \n"
	"3) Directive \n"
	"4) Quit \n"
	"\n"
	"Select your option: \n");
	scanf("%d", &option);
	printf("\n");

	return option;


}

I also left out the fact that the switch case is inside a menu loop:

Code:

	while(quit != 1)
	{
              switch..
         ....
         ....
         ....
         }

I'm guessing that is probably also a big part of the problem. Thanks anyway going to try and change it, and so far when I tried to change everything I kept on getting errors though when trying to make choose work with fgets.

Hint: fgets() into a smallish buffer, say BUFSIZ (look it up) or even 32 bytes. Then use sscanf() on the buffer.