What's wrong with this??

[jesus45]

So i'm making a stupid little game( i'm a beginner)

I want to make it so that if the user inputs something wrong it will just loop back around, so they can enter it again.

int whereToGo;
       
     
     do {
         cin >> whereToGo;
         if(whereToGo == 1); {
         cout << " You go outside and you are stopped by a guard.\n";
         cout << " He asks you to go on a quest for him";
         }
         else { 
              if(whereToGo == 2);
              cout << " You have chosen to stay and look around\n";
              cout << " You look around the castle is made of marble\n";
              cout << " There are wealthy people all around you\n";
              }
              }  while (whereToGo < 3);

any suggestions are welcome thanks

[GCDEF]

if(whereToGo == 1);

Your braces don't look right in your else and second if statement either.

[jesus45]

Grr i tried to fix it but i still don't get it :P

[GCDEF]

Grr i tried to fix it but i still don't get it :P

What did you try? I assume you got the point that your if statements shouldn't be terminated with a semi-colon?

[jesus45]

this is what i have now.

 int whereToGo;
       
     
     do {
         cin >> whereToGo;
         if(whereToGo == 1); 
         cout << " You go outside and you are stopped by a guard.\n";
         cout << " He asks you to go on a quest for him";
         }
         else  
              if(whereToGo == 2);
              cout << " You have chosen to stay and look around\n";
              cout << " You look around the castle is made of marble\n";
              cout << " There are wealthy people all around you\n";
              }  while (whereToGo < 3);

Still doing something wrong though.

[GCDEF]

this is what i have now.

 int whereToGo;
       
     
     do {
         cin >> whereToGo;
         if(whereToGo == 1); 
         cout << " You go outside and you are stopped by a guard.\n";
         cout << " He asks you to go on a quest for him";
         }
         else  
              if(whereToGo == 2);
              cout << " You have chosen to stay and look around\n";
              cout << " You look around the castle is made of marble\n";
              cout << " There are wealthy people all around you\n";
              }  while (whereToGo < 3);

Still doing something wrong though.

Again - sigh - don't put a semi-colon after your if statement.

Anything you want to execute if the if statement is true should be enclosed in a block using braces.

if(whereToGo == 1)
{  
         cout << " You go outside and you are stopped by a guard.\n";
         cout << " He asks you to go on a quest for him";
}

[resumurof]

int whereToGo;


while ((whereToGo !=1) || (whereToGo!=2))
{

cout<<"Enter I/p"<<endl;
cin >> whereToGo;

if(whereToGo == 1)
{
cout << " You go outside and you are stopped by a guard.\n";
cout << " He asks you to go on a quest for him";
break;
}
else {
if(whereToGo == 2)
{
cout << " You have chosen to stay and look around\n";
cout << " You look around the castle is made of marble\n";
cout << " There are wealthy people all around you\n";
break;
}
}


}

[GCDEF]

((whereToGo !=1) || (whereToGo!=2))

Think about that. See if you can come up with a number where that is never true.