Need to give user more time

Anyone know how I can stop the '1' (pressed for first choice) being displayed as the first char in the string asked for by fcn2? I have tried using kbhit to only call fcn2 when nothing is being pressed and using fflush(stdin) to clear anything in the buffer, but nothing works.

Another recent post here mentions using delays... how would that be done?

void menu() {
char choice;
printf("Enter Choice:\n\n 1 - First\n 2 - Second\n");
choice = getch();
switch (choice) {
case '1':
fcn2();
break;
case '2':
fcn3();
break;
}
}

void fcn2() {
char accept='n';
clrscr();
printf("Type something: ");
fgets(player.name, 80, stdin);
}

thanks...

> choice = getch();
You type in "1\n" in response to this, and what you have left on the input stream is "\n"

> fgets(player.name, 80, stdin);
fgets returns at the first \n (which will also be the first character).


So you need to get rid of that spare \n,

Replace your getch() with this...
> choice = getch();
char buff[20];
fgets( buff, sizeof(buff), stdin );
choice = buff[0];

> and using fflush(stdin)
Because it does nothing - its an undefined operation. Some misguided compilers actually flush the input just to puzzle newbies.

[B]> choice = getch();
>You type in "1\n" in response to this, and what you have left on the input stream is "\n"<

Isn't that getchar? Pressing '1' with the code I have jumps straight to the second function... just too quickly, meaning the user is still pressing '1' when asked to input a string.

If the buffer contained '\n' I wouldn't even be able to write anything would I? player.name would just contain the newline...


> and using fflush(stdin)
Because it does nothing - its an undefined operation. Some misguided compilers actually flush the input just to puzzle newbies. <

Really? Seems to be a common answer here to related questions... oh well... like I said, it didn't do anything for me so I removed it from the code... thought I'd mention I'd tried it tho...

Cheers...

Pressing '1' with the code I have jumps straight to the second function... just too quickly, meaning the user is still pressing '1' when asked to input a string.

You could try and use a pause function before getting the user input in the second function. But as these functions are compiler specific I'm not sure which one you'll need.

If you're needing a pause funtion here's a simple one that does so for any specified number of whole seconds:

Code:

int poz(int x)
 {
 long T1,T2,T3;
 T1=time(&T2);
 T3=(time(&T2)) + x;
 while(time(&T2) < T3);
 return(x);
}

poz(3); /// Pause for 3 sec...

...be sure to include <time.h>

> Isn't that getchar? Pressing '1' with the code I have jumps straight to the second function... just too quickly, meaning the user is still pressing '1' when asked to input a string.
getch() is unbuffered input and fgets is buffered input.
There is no reason to assume that these two input streams are synchronised - they appear to be in DJGPP, but for your compiler, it looks as if they are not.

Code:

#include <stdio.h>
#include <conio.h>

void foo ( ) {
    char buff[10];
    printf( "Enter name > " );
    fflush( stdout );
    fgets( buff, sizeof(buff), stdin );
    printf( "Name=%s", buff );
}
int main ( ) {
    int ch;
    printf( "Press '1' > " );
    fflush( stdout );
    ch = getch();
    if ( ch == '1' ) foo();
    return 0;
}

I can press the keys "1fred\n", and get this output
Press '1' > Enter name > fred
Name=fred

Cheers everyone, I think I'll try all the above suggestions... it only seems a small problem but I'm sure if I ever start writing large programs, it'll be the small bugs that'll get to me the most....

P.S. Salem:

I'm using gcc as it came on the cd I got when I started my course. Since then I've found a couple of other compilers but am too used to gcc. I'm all too ready to switch if there is a markedly better one tho...