value of char variable with loops

**david999** · 11-03-2005

the first time around, the program works as expected. the integer x is set to 0 when a response is given other than 'y,Y,n,or N' so that the function will repeat. the second time around, even if 'y,Y,n,or N' is given as the reponse, the program seems to think that answer contains something else...perhaps the previous input? do i need to initialize answer each time around? thanks for the help if u can..extremely new here

Code:

void playgame()
{
char answer;
int x = 0;
     while ( x == 0){
     printf( "Maybe we can play a guessing game instead? Y or  N\n");
     getchar();
     scanf( "%c", &answer );
                if ( (answer == 121) || (answer == 89) )
                   {
                   printf( "Ok, but you will surely give up before you win!\n" );
                   x = 1;
                   }
                else if ( (answer == 110) || (answer == 78) )
                   {
                   printf( "Well then, go find someone else with whom to play.\n" );
                   x = 1;
                   }
                else
                   {
                   printf( "That was an invalid answer.  Please concentrate and try again.\n" );
                   x = 0;
                   getchar();
                   }
                    }

}

**Happy_Reaper** · 11-03-2005

You're missing braces after your first else statement. Try putting those in and see if something new happens.

**dwks** · 11-03-2005

Code:

(answer == 78)

->

Code:

(answer == 'N')

is clearer.

(tolower() in <ctype.h> will convert a letter to lowercase.)

the program seems to think that answer contains something else...perhaps the previous input?

Code:

     getchar();
     scanf( "%c", &answer );

    /* ... */

                else
                   {
                   printf( "That was an invalid answer.  Please concentrate and try again.\n" );
                   x = 0;
                   getchar();
                   }

You don't want two getchar()s. Get rid of one.

You might want to use this code instead of getchar:

Code:

int c;
while((c = getchar()) != '\n' && c != EOF) ;

**dwks** · 11-03-2005

You're missing braces after your first else statement.

No, the braces are fine, they're just not lined up.

**Happy_Reaper** · 11-03-2005

Oups. He's right. Listen to what he says.

**david999** · 11-03-2005

k, i thought the braces were straight. thanks for the info guys, ill need some time to read abit about the other strategies and functions u mentions...thanks for the help

Thread: value of char variable with loops

Thread Tools

Search Thread

Display

value of char variable with loops

Similar Threads

Another syntax error

Obtaining source & destination IP,details of ICMP Header & each of field of it ???

How do i un-SHA1 hash something..

String sorthing, file opening and saving.