C - gets not working

Question

I'm having a problem with gets() working. The thing is it should be working with just a warning from vars I am not using but the first gets just skips to the second.

int addClube(char *fileName)
{

        char qClube2[100], qClubeSimple[100], qEstadio[100], qCompleto[500],qAcronimo[100];
        int option=0;

        printf("\n  Indique o nome completo do clube:\n  (sem acentos)\n  ");
        gets(qClube2);

        printf("\n  Indique o nome simplificado do clube:\n  (sem acentos)\n  ");
        gets(qClubeSimple);

        printf("\n  Indique o acronimo do clube:\n  ");
        scanf("%s",qAcronimo);

        printf("\n  Indique o nome do estadio:\n  ");
        gets(qEstadio);

        return option;
}

The output I get when the function starts:

Indique o nome completo do clube:
(sem acentos)

Indique o nome simplificado do clube:
(sem acentos)

Once again thanks in advance for the help an if you more info just tell me.

You probably used `scanf("%d"` in a function prior to calling this — M.M, Feb 23 '16 at 03:29
`gets` is inherently unsafe beyond repair, use `fgets` instead. — Sergey Kalinichenko, Feb 23 '16 at 03:30
yes I did with when navigating from the menu but then how do I solve that? — Rafael Botas, Feb 23 '16 at 03:33
`scanf("%s",qAcronimo);` leaves the trailing `'\n'` in `stdin` for the next `gets()` to consume as an empty line. Just use `fgets()`. — chux - Reinstate Monica, Feb 23 '16 at 03:47

score 1 · Answer 1 · answered Feb 23 '16 at 18:54

If you need to clear the input stream, fflush(stdin) will not do the job. It is my understanding that it works on Windows, but most other systems (UNIX, Linux) it is undefined behavior.

I wrote this on the fly, so it may or may not work, but essentially you need to discard all data you don't want. The following discards a line from the input buffer.

void clearinline(void)
{
    int c;
    do {
        c = getchar();
    } while(c != '\n' && c != EOF)
}

score -3 · Accepted Answer · answered Feb 23 '16 at 03:40

-3

Try Using fflush(stdin); after all the gets() Check This; Also, why are u using this option var for ?

answered Feb 23 '16 at 03:40

Joe Kerr

34
9

ill try it; that option var is to get back to the menu or just close the app. – Rafael Botas Feb 23 '16 at 03:42
Thanks it worked. ill vote up your answer when stackoverflow allows me. – Rafael Botas Feb 23 '16 at 03:47
No Problem, glad i could help :D – Joe Kerr Feb 23 '16 at 03:49
1

@Rafael Botas This answer recommended `fflush(stdin)`. This is undefined behavior. – chux - Reinstate Monica Feb 23 '16 at 03:50
`fflush()` is for output streams. Not input. – PC Luddite Feb 23 '16 at 04:05
so do you have a solution for my problem? @PC Luddite – Rafael Botas Feb 23 '16 at 17:33
@RafaelBotas If you need to clear the input stream, there are several tricks and some implementations actually do support `fflush(stdin)`, but this is nonstandard and your code won't be portable. – PC Luddite Feb 23 '16 at 18:11

C - gets not working

2 Answers2