Thread: IF error

Code:

// Structure : MONSTER //////////////////////////////////////////
struct mnstr
{
char* name;
int strength;
int constitution;
int magiclevel;
};

mnstr monster;

// Monster Statistics //////////////////////////////////////////
if(player.exp < 2000)
{getmonster = 2;}
else {
if(player.exp < 3000)
{getmonster = 3;}

I get a parse error before the first IF in Monster Statistics... Anyone got an idea ?

Code:

else {
if(player.exp < 3000)
{getmonster = 3;}

you need another '}'

Code:

else {
if(player.exp < 3000)
{getmonster = 3;}
}

or better

Code:

else if (player.exp < 3000){
getmonster = 3;
}

Actually no, the code goes like this :

Code:

// Structure : MONSTER //////////////////////////////////////////
struct mnstr
{
char* name;
int strength;
int constitution;
int magiclevel;
};

mnstr monster;

// Monster Statistics //////////////////////////////////////////
if(player.exp < 2000)
{getmonster = 2;}
else {
if(player.exp < 3000)
{getmonster = 3;}
else {
if(player.exp < 4000)
{getmonster = 4;}
else {
if(player.exp < 5000)
{getmonster = 5;}
else {
if(player.exp < 6000)
{getmonster = 6;}
else {
if(player.exp < 7000)
{getmonster = 7;}
else {
if(player.exp < 8000)
{getmonster = 8;}
else {
if(player.exp < 9000)
{getmonster = 9;}
else {
if(player.exp < 10000)
{getmonster = 10;}
}}}}}}}}

So I don't need another }. Besides, the parse error is before the IF. Any ideas ?

Code:

if(player.exp < 2000)
{getmonster = 2;}
else {
if(player.exp < 3000)
{getmonster = 3;}
else {
if(player.exp < 4000)
{getmonster = 4;}
else {
if(player.exp < 5000)
{getmonster = 5;}
else {
if(player.exp < 6000)
{getmonster = 6;}
else {
if(player.exp < 7000)
{getmonster = 7;}
else {
if(player.exp < 8000)
{getmonster = 8;}
else {
if(player.exp < 9000)
{getmonster = 9;}
else {
if(player.exp < 10000)
{getmonster = 10;}
}}}}}}}}

no you can't do that.

Code:

if(player.exp < 2000)
{getmonster = 2;}
else if(player.exp < 3000){
getmonster = 3;}
else if(player.exp < 4000){
getmonster = 4;}
else if(player.exp < 5000)
{getmonster = 5;}
else if(player.exp < 6000)
{getmonster = 6;}
else if(player.exp < 7000)
{getmonster = 7;}
else if(player.exp < 8000)
{getmonster = 8;}
else if(player.exp < 9000)
{getmonster = 9;}
else if(player.exp < 10000)
{getmonster = 10;}

Each 'if' or 'else if' needs it's own open and closed set of parentheses. You can't close them all at the end like you did.

Yes you can, I've done it before. I just can't see anything wrong with this.

Ok then, you can

Pffff ! So I guess you don't know what's wrong with it ?

Anyone else ?

My code says :

Code:

if(condition1)
{dothis;}
else {
if(condition2)
{do that;}}

Code:

if(condition1)
{dothis;
else {do that;}}

This is the same as

Code:

if (condition1)
{
do this;
else
{
do this;
}
}

and you can't do it

NO ! Look :

There is another IF in my ELSE statement :

Code:

if(myname == "Me")
{cout << "Hello ME";}
else {
if(myname == "You")
{cout << "Hello YOU";}
}

sorry to tell you this dude but that aint's right.....

if you wish to uise if else then you will have to do it like this:


if(condition1)
{
do this
}
else if(condition2)
{
do this
}
else
{
do this
}

Also if you have only one operation that needs to be executed after the if you have no need for the {} e.g.

if(condition1)
do this;
else
do this;

and what you were doin does not work.....

if(condition1)
{
do this;
else if(condition2)
{
do this;
else
{
do this;
}
}
}

shouldn't work because the else does not know which if it should be using...

Actually DirX it works. You can do this :

Code:

if(integer == 1)
{cout << "1";}

else {

if(integer == 2)
{cout << "2";}

else {

if(integer == 3)
{cout << "3";}

} }

I think this may be the same as using ELSE IF, but I'm not sure. Either way, this works, I've tested it in every possible way.

hmm actually now you said it it does work but it is far from what any person with knowledge of c++ would do..... basically what you are doing here is nesting... which would mean that you would have to tab it in before the following if.

using this:

if(condition1)
{
do this;
}
else if(condition2)
{
do this;
}
else
{
do this;
}

is a lot more convetionel and easier to use, read, and understand