Bmi

Question

This is my bmi calculator let me know what you think?

#your code goes here
weight=int(input())
height=float(input())
i=weight/(height**2)

if i <= 18.5:
print ('Underweight')
elif i<=25 and i>18.5:
print ('Normal')
elif i<=30 and i>25:
print ("Overweight")
elif i>=30:
print ("Obesity")

Answer 1

You code looks good.

You can write an else statement at the end instead of elif as it does the same thing (meaning you get the same output) but it's shorter:

...
else:
print("Obesity")

Answer 2

I see what yoh mean the last line should be
Elif i>30

Answer 3

i<=30 and i>=30 — in 2 different statements do not make sense. If the answer is 30, which one will be printed?

Answer 4

Hi james ewell you could use for range testing the annotation:
if 18.5 < i <= 25:

Answer 5

james ewell you can reduce the logic. For example, when i<=18 is false, then i>18 must be true so there is no need to explicitly test for that. Simply do one conditional at each threshold.

Also, the task indicates that if it is equal to 18.5 then bmi is 'Normal', not 'Underweight'. And the rules are likewise at the other thresholds. The conditonals need correction as below:

#your code goes here
weight=int(input())
height=float(input())
i=weight/(height**2)

if i<18.5:
print ('Underweight')
elif i<25:
print ('Normal')
elif i<30:
print ("Overweight")
else:
print ("Obesity")

Answer 6

Hey james ewell it won't enter because it's elif. In case you code another if your thought would be right.
Elif is entered like else, when the if statement results false and tests a new condition. Else would be entered in any case that none of the if/elif is true.

Answer 7

Ah okay that makes sense, thank you for you comment!!

Answer 8

Brain, correct me if im wrong. But if i do elif i<25 let say the number was 17 it would be true for all statments that its less the 25 and 30 so wouldnt it print underweight,normal,and overwieght?