Ja que incluimos a linha else para validação dos dados do imc com uma mudança na mensagem, por que não foi sugerido colocar a mudança da cor do background dentro do else economizando assim linhas de código? Aqui funciou perfeitamente, existe algum motivo para fazermos nas categorias individuais?
Uma sugestão para uma futura reformulação do curso é o instrutor diminuir a velcidade da fala, chega a ser confuso muitas vezes e torna dificil acompanhar o raciocínio. A forma como o conteúdo do curso de HTML/CSS foi apresentada pelo Pedro Marins foi muito melhor.
if(peso <= 0 || peso >= 1000) {
console.log("Peso inválido!");
pesoEhVlido = false;
tdImc.textContent = "Peso inválido!";
}
if(altura <= 0 || altura >= 5){
console.log("Altura inválida!");
alturaEhValida = false;
tdImc.textContent = "Altura inválida";
}
if(pesoEhVlido && alturaEhValida){
var imc = peso/(altura*altura);
tdImc.textContent = imc.toFixed(2);
} else {
tdImc.textContent = "Peso e/ou altura invalido!";
paciente.classList.add("paciente-invalido");
}
}