[Resolved] if problem

aenirtower

if($exp >= 200){ $level="4";
} elseif($exp >= 400){ $level="5";
}

that doesn't work... whenever $exp goes to 405, $level stays at 4. What's wrong?

tomhath

Darn code is doing what you told it to do :mad:

if ($exp >= 200) { // 405 is greater than or equal than 200, isn't it???
   $level="4"; 
} elseif ($exp >= 400) { 
   $level="5"; 
}

Try this instead:

if($exp >= 200 && $exp < 400) { 
  $level="4"; 
} elseif ($exp >= 400) { 
  $level="5"; 
}

Mordecai

Well, look closely at your code:

if ($exp >= 200){
  $level="4";
}
[b]else[/b]if ($exp >= 400){
  $level="5";
}

Obviously, 405 is above 200. Because it triggers the original if, it's not going to trigger the elseif.
Change the original if:
if (($exp >= 200) && ($exp < 400)) {}

aenirtower

Well, I tried that too, and it didn't work. I'm making a RPG and I am having trouble with highering levels, so the problem is that the if() will not do what its told...

Mordecai

The if is doing exactly what you tell it to.

When you use our code (it seems to be almost identical, tomhath's and mine), what happens?

Weedpacket

Two other (functional) ways of writing the code:

if($exp>=200){$level = 4;}
if($exp>=400){$level = 5;}

If the Exp is at least 200, the level is set to 4.
After that, if the experience is at least 400, the level gets set to 5.
A little wasteful, though. Else would be nicer, but it still seems wasteful to check to see if $exp is greater or less than 400 twice...

if($exp>=400){$level = 5;
}elseif($exp>=200){$level = 4;
}

If the exp is at least 400, set the level to 4; otherwise (i.e., when the exp is less than 400) if the exp is at least 200, set the level to 4.

The difference between the code immediately above and the original is that every number that is at least 400 is at least 200, so every $exp that might have passed the second test has already passed and been dealt with by the first. But there are numbers that are at least 200 that are not at least 400 (300, for example); these would fail the first test and are therefore checked against the second test.

Any of these three solutions would work. If they "don't", the problem is elsewhere.

aenirtower

EDIT: I edited the code of all of yours... and it now works, thanks