CODEWITHSUNDEEP
cloopswhile-loopscanfbreak
T.Malo
T.Malo

Reputation: 512

C while loop program

New to C. Simply put I just don't see why this wont work. All I want it to do is just keep asking you if you want to play until your input is 'n'. I keep getting (error C2143: syntax error : missing ';' before 'break')

#include <stdio.h>
int main(void)
{
    char answer;
    printf("Would you like to play? Enter Y or N: ");
    while (scanf_s(" %c", &answer))
    {
        if (answer == 'y')
            printf("Answer is %c\n", answer);
        printf("Success!\n");
        printf("Do you want to play again? Y or N: ");
        else (answer == 'n')
        break;
    }
    printf("Goodbye\n");
    return 0;
}

Upvotes: 0

Views: 175

Answers (6)

kerolasa
kerolasa

Reputation: 411

There is no need to duplicate glibc rpmatch() function, that does good job interpreting yes/no answers.

#include <stdio.h>
#include <string.h>
#include <stdlib.h> 
int main(int argc, char **arg)
{
        int r;
        char *answer = NULL;
        size_t len = 0;
        while (1) {
                printf("Would you like to play? Enter Y or N: ");
                getline(&answer, &len, stdin);
                r = rpmatch(answer);
                if (r < 0)
                        printf("????\n");
                else if (r == 0)
                        break;
        }
        free(answer);
        return(0);
}

Upvotes: 0

herohuyongtao
herohuyongtao

Reputation: 50707

Besides @Mike's answer to add the needed {} to enclose the codes in the if's code scope, you also need to change else to elseif.

So the code should be like this:

if (answer == 'y') 
{
    printf("Answer is %c\n", answer);
    printf("Success!\n");
    printf("Do you want to play again? Y or N: ");
}
else if (answer == 'n')
    break;

P.S.: One minor suggestion, based on the guideline messages showing to user ("Would you like to play? Enter Y or N: "), you probably need also to handle uppercase inputs, i.e. 'Y' and 'N'. You can simply do like this:

if (answer=='y' || answer=='Y') 
{
    // ...
}
else if (answer=='n' || answer=='N')
    break;

Upvotes: 0

pizzaEatingGuy
pizzaEatingGuy

Reputation: 880

You have to write

else if (answer == 'n')

Upvotes: 0

deviantfan
deviantfan

Reputation: 11434

You can´t write an else with a condition.
else will be executed if the if wasn´t executed, without a second condition check.
If you want to have something to executed
if the if was not executed AND a contition is true, use else if.

Upvotes: 0

Mike
Mike

Reputation: 49473

if and else have to be next to each other... you need to have curly brackets in there as such:

    if (answer == 'y') 
    {
        printf("Answer is %c\n", answer);
        printf("Success!\n");
        printf("Do you want to play again? Y or N: ");
    }
    else
        break;

to do more than one statement in an if block.

If you want to check more than one condition, don't use an if/else but a if/else if like such:

    if (answer == 'y') 
    {
        printf("Answer is %c\n", answer);
        printf("Success!\n");
        printf("Do you want to play again? Y or N: ");
    }
    else if(answer == 'n')
        break;

Upvotes: 2

pzaenger
pzaenger

Reputation: 11993

You will need to rewrite your if-else using brackets:

if (answer == 'y') {
   printf("Answer is %c\n", answer);
   printf("Success!\n");
   printf("Do you want to play again? Y or N: ");
} else if (answer == 'n') {
   break;
}

Upvotes: 0

Related Questions