Проблема с ветвлением else if?

Question

[spoT1]

let w = prompt('Какой твой вес?')
let h = prompt('Какой твой рост')

if(w/(h/100*h/100)<18.5){
alert('Обратитесь к врачу, у вас дефицит массы')
}
else if(w/(h/100*h/100)>=18.5 && w/(h/100*h/100)<=24.9){
alert('Нормальная масса тела')

}
else if(w/(h/100*h/100)>=25 && w/(h/100*h/100)<=29.9){
alert('У вас предожирение')

}
else if(w/(h/100*h/100)>=30 && w/(h/100*h/100)<=34.9){
alert('Обратитесь к врачу, 1 степень ожирения')

}
else if(w/(h/100*h/100)>=35 && w/(h/100*h/100)<=39.9){
alert('Обратитесь срочно к врачу, 2 степень ожирения')

}
else if(w/(h/100*h/100)>40){
alert('У вас огромные проблемы, 3 степень ожирения')
}
else if(w < 0 || h < 0 || typeof w == 'string' || typeof h == 'string'){
alert('Введите корректное число')
}

Вот код, когда число отрицательно все равно считает, как пофиксить

Comments

[Rsa97]

Какой великолепный образчик говнокода! Его надо не фиксить, а переписывать с нуля.

[ThunderCat]

Rsa97, ну зачем же все портить, человек старался, г**кодил от души, с чувством... Многабукав написал... ))

[SagePtr]

Вместо массы одинаковых вычислений, вынесите хотя бы для начала w/(h/100*h/100) в отдельную переменную - сразу станет на порядок проще и читабельнее)
Я уже молчу о том, что границы заданы абы как, что если окажется 24.95, попадающий между диапазонами [18.5, 24.9] и [25, 29.9].
И основное: сначала валидируем числа, а потом уже производим над ними арифметические операции.

[twobomb]

Говорит у меня нормальная масса тела, надеюсь алгоритм не врет :D

Answer 1

[DAnya78]

надо было ошибку в начало поместить

let w = prompt('Какой твой вес?')
let h = prompt('Какой твой рост')
if(w < 0 || h < 0 || typeof w == 'string' || typeof h == 'string'){
alert('Введите корректное число')
}
else if(w/(h/100*h/100)>=18.5 && w/(h/100*h/100)<=24.9){
alert('Нормальная масса тела')

}
else if(w/(h/100*h/100)>=25 && w/(h/100*h/100)<=29.9){
alert('У вас предожирение')

}
else if(w/(h/100*h/100)>=30 && w/(h/100*h/100)<=34.9){
alert('Обратитесь к врачу, 1 степень ожирения')

}
else if(w/(h/100*h/100)>=35 && w/(h/100*h/100)<=39.9){
alert('Обратитесь срочно к врачу, 2 степень ожирения')

}
else if(w/(h/100*h/100)>40){
alert('У вас огромные проблемы, 3 степень ожирения')
}
else if(w/(h/100*h/100)<18.5){
alert('Обратитесь к врачу, у вас дефицит массы')
}

Comments

[ThunderCat]

надо было ошибку в начало поместить

Логично, чтобы легче искать ошибку - помещай ее в начало ))

[spoT1]

DAnya78 , спасибо

[DAnya78]

ThunderCat, Так да :). Он тебе написал что новичок, пусть хотя бы поймет в чем тут ошибка

Answer 2

[ThunderCat]

Вот код, когда число отрицательно все равно считает, как пофиксить

Во первых открыть для себя switch case конструкцию. Во вторых включить голову и обозначить граничные условия. А так же дотумкать почему у вас при отрицательном весе будет дефицит массы )

Comments

[spoT1]

я новичок окда? я занимаюсь js от силы 3 недели, через case пробовал, я не смог т.к там нельзя вводить операторы(<,>) . Не могли бы вы показать наглядно?

[edward_freedom]

spoT1, тут скорей логика, чем программирование. Ты сначала проводишь вычисления, а потом проверяешь

[spoT1]

edward_freedom, ну тут согласен

[ThunderCat]

spoT1, вычислить значения, затем отсвитчить.
И даже через иф - эти вложенные иф-елс вообще не нужны, у вас же все значения имеют граничные непересекающиеся условия. Достаточно только ифов.

Answer 3

[Alexandr]

Проверяйте корректность введенных данных в отдельном методе, сразу после их получения. А основная логика должна вычисляться только с корректными данными.

Answer 4

[SagePtr]

Вместо массы одинаковых вычислений, вынесите хотя бы для начала w/(h/100*h/100) в отдельную переменную - сразу станет на порядок проще и читабельнее)
Я уже молчу о том, что границы заданы абы как, что если окажется 24.95, попадающий между диапазонами [18.5, 24.9] и [25, 29.9].
И основное: сначала валидируем числа, а потом уже производим над ними арифметические операции.

Comments

[spoT1]

спасибо за совет