Skip to main content
conteúdo editado
Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

O problema, como já dito nos comentárioscomo 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.

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.

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.

Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

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.