Why isn't it working? :/

Question

https://code.sololearn.com/W0FF39tPb7pG/?ref=app

Answer 1

else if(loottable > 25 && loottable <= 50)
You miss loottable in the RHS.

Answer 2

Robin Flickinger , add in every conditional check the variable name again after logical and "&&" operator 🐱

Answer 3

I meant Right hand side.
You must write the variable name on both sides of the && operator.

Answer 4

Robin Flickinger Also you have misspelled the variable name on line 11. Make that correction as well.

Answer 5

You can replace all "else if ... " expresion like first one, when you have sorted level ( in our case increase all the time the level.

if(loottable <= 25){
alert("Bronze 1 dropped");
}
else if(loottable <= 50) {
alert("Bronze 2 dropped");
}
else if(loottable <= 75) {
alert("Bronze 3 dropped");
}
else if(loottable <= 86.5) {
alert("Silver 1 dropped");
}
else if(loottable <= 98) {
alert("Silver 2 dropped");
}

Answer 6

f(loottable <= 25){
alert("Bronze 1 dropped");
}
else if(loottable > 25 && loottable<= 50) {
alert("Bronze 2 dropped");
}
else if(loottable > 50 && loottable<= 75) {
alert("Bronze 3 dropped");
}
else if(loottable > 75 && loottable<= 86.5) {
alert("Silver 1 dropped");
}
else if(loottable > 86.5 && loottable<= 98) {
alert("Silver 2 dropped");
}
else {
alert("Titania Systems dropped!");
}

U have to use Boolean expression after && ,|| ,! because these operators check for only Boolean operands .Here loottable<=50 returns true if condition is true else returns false

Answer 7

Avinesh what is RHS? I am totally a newbie

Answer 8

Avinesh ohhh ok i know what u mean, i had it like that before and it didn't work. I will try this again. Thank you!

Answer 9

Cool it's workin! Thank you guys!