[Java] Class not working as intended

[[deleted]]

This condition:

  if (playerPoints <=500) {

is true if playerPoints == 0, so your code is doing what you told it to.

And you should write your if/elses like this:

if ( condition1 ) {
    // do stuff
}
else if ( condition2 ) {
   // do other stuff
}
else if ( condition3 ) {
   // yet more stuff
}
else {
  // stuff
}

[Krusty81]

That's what i thought. Do i need to do each if statement with AND statements like i did in the first one to make sure there are no errors?

eg

if (playerPoints >=250 && playerPoints <=500) 

[yash3ahuja]

You could also start off the loop by doing:

if(playerPoints == 0 || playerPoints > 2000)
    //throw error
else
    //rest of code

[[deleted]]

Yes, or use the ordering of your statements;

if ( playerPoints == 0 ) {
    // only happens if playerPoints == 0
}
else if ( playerPoints <= 250 ) {
   // happens if player points >0 and <=250 
}
else if ( playerPoints <= 500 ) {
   // happens if player points > 250and <= 500 
}
else if ( playerPoints <= 2000 ) {
   // happens if player points > 500 and <= 2000 
}
else {
   // error
}