[c] problema char

[ekoalon]

salve a tutti...sono nuovo del forum, da pochissimo programmo in C, ho sempre scritto programmi (semplici) in C++...ho un problema: ho scritto un programma che risolve un'equazione di secondo grado, alla fine del programma chiedo all'utente se vuole risolvere una nuova equazione o no...il problema sta proprio qui...vi scrivo il codice per farvi capire meglio:

int main()
{
   char risp;
    
     do
     {
          /*CODICE PER RISOLVERE L'EQUAZIONE*/
          
          printf("\nVuoi risolvere un'altra equazione? s=si n=no\n");
          scanf("%c",&risp);
          do
          {
            if(risp!='n' && risp!='s')
            {
                printf("\nError....\n");
	        scanf("%c",&risp);
            }
          }
          while(risp != 'n' && risp != 's');
     }
     while(risp=='n');
     return 0;   
}

una volta lanciato il programma e risolta l'equazione mi stampa a video: "Vuoi risovere un'altra equazione? s=si n=no" e "Error..." e se premo un qualsiasi tasto esce dal programma...
HELP!!!
p.s. ho sempre programmato in C++ quindi è altamente probabile che stia confondendo la sintassi del C++ con quella del C o che abbia scritto una grossa cavolata che non ha senso?

[Ippo343]

Innanzitutto questo codice è molto ridondante, potresti semplificarlo molto.

In secondo luogo, è sbagliata la condizione del while più esterno: il ciclo continua a girare solo se l'utente risponde di no, il che è ovviamente il contrario di quello che vuoi.

Terzo, dopo ogni risposta l'utente preme invio, inserendo quindi un newline nel buffer: se l'utente risponde n, al ciclo successivo la scanf legge un newline e il programma segnala un errore. Dovresti fare una fflush sullo stdin (se sei su windows, altrimenti cerca nel forum, si è parlato mille volte di una soluzione più portabile).

int main()
{
   char risp;
    
     do
     {
          /*CODICE PER RISOLVERE L'EQUAZIONE*/
          
          printf("\nVuoi risolvere un'altra equazione? s=si n=no\n");
          scanf("%c",&risp);
          do
          {
            if(risp!='n' && risp!='s')
            {
                printf("\nError....\n");
	        scanf("%c",&risp);
            }
          }
          while(risp != 'n' && risp != 's');
     }
     while(risp == 's' );
     return 0;   
}

[ekoalon]

intanto grazie per la risposta...il codice l'avevo controllato molte volte...non mi ero accorto che la condizione dell'ultimo while era sbagliata... ora mi informo coma funziona la fflush...
grazie mille!!!

[MdE2005]

Originariamente inviato da ekoalon
intanto grazie per la risposta...il codice l'avevo controllato molte volte...non mi ero accorto che la condizione dell'ultimo while era sbagliata... ora mi informo coma funziona la fflush...
grazie mille!!!

Ti semplifico la ricerca: http://www.cplusplus.com/reference/c...cstdio/fflush/


Ciao

[Ippo343]

In pratica, lo stdin è un file. Perciò quando ci digiti sopra con la tastiera, tutti i caratteri che digiti finiscono in questo file. Perciò se scrivi:

 |c|\n|...

e poi esegui una scanf chiedendo un solo carattere, verrà estratto un solo carattere.

 |\n|...

Al prossimo ciclo, richiedi un nuovo carattere, e verrà quindi estratto il newline.

La fflush (che funziona solo su windows, ricorda) svuota il buffer di input, cancellando quindi il newline (e se non vado errato, tutto ciò che è stato scritto sullo stdin dopo il newline).