O problema, como já dito nos comentários, é ter usado a vírgula como separador decimal, quando o correto é usar o ponto. Ou seja, em vez de 18,5
deve ser 18.5
, o mesmo vale para todos os outros números.
Agora explicando o problema: a vírgula aí está funcionando como um operador. Basicamente, uma expressão como (a,b,c)
avalia todos os seus operandos (no caso, a
, b
e c
) da esquerda para a direita e retorna o valor do último (no caso, c
).
Isso quer dizer que o valor da expressão (imc < 18,5)
é 5
. Não acredita? Pode conferir:
let imc = 1000;
let x = (imc < 18,5);
console.log(x); // 5
Como a vírgula aí é um operador (e não o separador de casas decimais), você poderia inclusive escrever com espaços: (imc < 18, 5)
ou (imc < 18 , 5)
funcionariam da mesma forma. No caso, em (imc < 18,5)
temos duas expressões: a comparação imc < 18
e a expressão 5
, ambos separados pelo operador vírgula.
E o resultado desta expressão é o valor do último operando, que no caso é 5
.
Depois, o if
avalia o resultado desta expressão (que é 5
), e como o valor 5
é considerado "verdadeiro", ele sempre entra nesse if
, independente do valor do imc
. Veja:
let imc = 1000;
if (imc < 18,5)
console.log('ok'); // imprime "ok"
Embora a comparação imc < 18
seja falsa, o resultado de toda a expressão é 5
, que é considerado verdadeiro. Por isso sempre entra no if
.
Trocando para 18.5
funciona porque agora estamos usando o caractere correto para separar as casas decimais (ponto em vez de vírgula) e agora sim estaremos comparando o imc
com "dezoito e meio".
Outro detalhe é que não precisa repetir os valores mais de uma vez, poderia ser apenas:
let media = '';
if (imc < 18.5) {
media = 'Abaixo do peso';
} else if (imc < 24.9) {
media = 'com Peso normal';
} else if (imc < 29.9) {
media = 'com Sobrepeso';
} else if (imc < 34.9) {
media = 'com Obesidade grau I';
} else if (imc < 39.9) {
media = 'com Obesidade grau II';
} else {
media = 'com Obesidade Morbida';
}
Se não entrar no primeiro if
é porque o imc
com certeza é maior que 18.5
, então não precisa testar de novo no else if
seguinte. E se não entrar no segundo if
, é porque com certeza não é menor que 24.9
então não precisa testar isso no else if
seguinte. E assim por diante. Por fim, no último else
nem precisa testar nada, pois se chegou ali é porque não entrou em nenhuma das condições anteriores, então neste ponto eu sei que não é menor que 39.9
.
Repare também que 24.90
e 24.9
dá na mesma, então retirei esses zeros do final.
Claro que faltou validar se algum valor digitado é negativo, etc, mas aí você pode adicionar facilmente.