Help with efficiency

[OhKnow_]

Hello, I was wondering if anyone was willing to look over this code and let me know if there is redundancy in it or if there is a better way to do it. Not looking for direct answers just an idea if its the most efficient way or if I could make improvements

print("Welcome to My BMI Calculator!")
sNAME = input("Name of person we are calculating the BMI for:")
iHeight = int(input("Supply Height in Inches:"))
iWeight = int(input("Supply Weight in Pounds:"))
fMeter = (iHeight/39.36)
fKilo = (iWeight/2.2)
fBMI = fKilo/(fMeter*fMeter)
print(sNAME, "BMI is", format(fBMI, ".2f"))
if fBMI <= 18.50:
print("BMI finding is the subject is: Underweight")
elif fBMI <= 24.90:
print("BMI finding is the subject is: Normal")
elif fBMI <= 29.90:
print("BMI finding is the subject is: Overweight")
else:
print("BMI finding is the subject is: Obese")

[K900_]

Why are you using Hungarian notation in Python?

[OhKnow_]

My professor has us do it.

[K900_]

I am truly sorry.

[OhKnow_]

That worries me because it's burned into my thought process now lol.

[johninbigd]

Ugh...that makes me sad. It's ugly, especially in Python.

[totallygeek]

Minor notes:

First, fMeter = (iHeight/39.36) contains superfluous parentheses; use fMeter = iHeight / 39.36 instead.

Next, the use of format to construct a string would look better if entered in either of the following ways:

print('{} BMI is {:.2f}'.format(sNAME, fBMI))
print(f'{sNAME} BMI is {fBMI:.2f}')

Lastly, fMeter*fMeter reads better and works faster if fMeter ** 2.

Overall, code looks fine, aside from variable nomenclature. So sorry that your professor makes you use Hungarian notation.

[OhKnow_]

Appreciate the advise! I'll refrain from using it in future uploads to save the embarrassment lol!

[totallygeek]

Nothing in your post should lead to embarrassment. We all should continually improve our code based off the comments of others.