[C] fgets() strcmp(): validazione password (ERA:dov'è l'errore??)

[Peppewt]

Ciao a tutti, dopo un lungo periodo di inattività, ho ripreso finalmente a scrivere qualcosina in C, e subito arrivano i primi problemi... Vi posto subito il codice x nn perderci in chiacchere:

#include <stdlib.h>
#include <stdio.h>
#include <string.h>

int main(void)
{

char    code[50];

printf("Indovina la password \n");

while (1)
 {
   fgets(code, 50, stdin);

    if (! strcmp(code, "Vado a casa"))
       {
          printf("ok\n");
       break;
       }
       
    else if (! strcmp(code, "Bobby"))
            {
            printf("bobby\n");
            }
            
    else if (! strcmp(code, "Boby"))
            {
            printf("boby\n");
            }
            
    else
        {
        printf("riprova\n");
        }
 }
system("pause");
return 0;
}

Il problema è ke mi da errore, cioè nn parte proprio il prog.
come mai??
eppure nn credo sia il mio compilatore..
Grazie in anticipo...
Bye
Peppe

Ps. nn fate caso alle password sono prese a caso...

[Johnny_Depp]

char code[50];

    
printf("Indovina la password: ");

while (1)
{
    fgets(code, 50, stdin);

    if (! strcmp(code, "ciao ciao\n"))
    {
        printf("ok\n");
        break;
    }
    else if (! strcmp(code, "uno\n"))
        printf("uno\n");
    else if (! strcmp(code, "due\n"))
        printf("due\n");
    else
        printf("Errata: riprova\n");
}

la funzione fgets() agginge un newline in coda alla stringa...
o lo togli (prima...) o lo aggiungi alla validazione.

[Peppewt]

Grazie Johnny, funziona aggiungendo semplicemente, come hai detto tu, un new line alla validazione....
Cmq. volevo sapere cosa intendevi dire con:

la funzione fgets() agginge un newline in coda alla stringa, o lo togli (prima...)...

Lo togli prima cioè...
Mi fai un esempio?? pls..
Bye e grazie...

[Johnny_Depp]

...togli il newline dalla stringa "code[]" prima di fare il confronto:

    char code[50];
    char i = 0;
        
    printf("Indovina la password: ");

    while (1)
    {
        fgets(code, 50, stdin);
        
        while( 1 ) // itero per l'intera stringa
        {
            if( code[i] == '\n' ) // se trovo il newline
            {
                code[i] = '\0';  // lo sostituisco con un terminatore di stringa "NULL"
                i = 0; // (...)
                break; // ...e esco    
            }
            i++; // altrimenti continuo ad iterare (passando al carattere successivo)    
        }
            
        
        if (! strcmp(code, "ciao ciao")) // ora non servono più i newline!
        {
            printf("ok\n");
            break;
        }
        else if (! strcmp(code, "uno"))
            printf("uno\n");
        else if (! strcmp(code, "due"))
            printf("due\n");
        else
            printf("Errata: riprova\n");
    }

p.s. se inserisci: "ciao ciao" utilizzando la fgets()
viene aggiunto un carattere di newline in coda alla stringa
quindi vai a confrontare "ciao ciao\n\0" con "ciao ciao\0"
e le due stringhe sono diverse.

[Peppewt]

Grazie mille, mi 6 stato di grande aiuto...

Ciao...

[Johnny_Depp]

p.s. visto che hai scomodato l'header <string.h>
ti conviene sfruttarlo. Utilizza la funzione "strlen()" al posto del ciclo while()

    printf("Indovina la password: ");

    while (1)
    {
        fgets(code, 50, stdin);

/*        
        while( 1 )
        {
            if( code[i] == '\n' )
            {
                code[i] = '\0';
                i = 0;
                break;    
            }
            i++;    
        }
*/            
  
        i = strlen(code) - 1;
        code[i] = '\0';