1
resposta

Melhorando ainda mais o código

Boa noite pessoal,depois dessa aula de melhorar a manutenção do codigo o meu codigo de validação parou de detectar os erros mesmo eu colocando a altura > 3.00 ou < e o peso > 1000 ou < 0, ele continua colocando o resultado normal na tabela sem aparecer a mensagem de "Altura invalida ou peso invalido".

Onde eu estou errando ?

var titulo = document.querySelector(".titulo");
titulo.textContent = "Aparecida Nutricionista";

var pacientes = document.querySelectorAll(".paciente");

for (var i = 0; i < pacientes.length; i++) {

  var paciente = pacientes[i];

  var tdPeso = paciente.querySelector(".info-peso");
  var peso = tdPeso.textContent;

  var tdAltura = paciente.querySelector(".info-altura");
  var altura = tdAltura.textContent;

  var tdImc = paciente.querySelector(".info-imc");

  var pesoEhValido = true;
  var alturaEhValida = true;

  if (peso <= 0 || peso >= 1000) {
    console.log("Peso invalido");
    pesoEhValida = false;
    tdImc.textContent = "Peso Inválido!";
    paciente.classList.add("paciente-invalido")
  }

  if (altura <= 0 || altura >= 3.00) {
    console.log("Altura invalida")
    alturaEhValida = false;
    tdImc.textContent = "Altura Inválida!";
    paciente.classList.add("paciente-invalido")
  }

  if (alturaEhValida && pesoEhValido) {
    var imc = calculaImc(peso, altura);
    tdImc.textContent = imc;

  }
}

function calculaImc(peso,altura){
  var imc = 0;
  imc = peso / (altura * altura);

  return imc.toFixed(2);
}
var botaoAdicionar = document.querySelector("#adicionar-paciente");
botaoAdicionar.addEventListener("click", function(event) {
  event.preventDefault();

  var form = document.querySelector("#form-adiciona");
  // Estraindo infomações do paciente do form.
  var paciente = obtemPacienteDoFormulario(form);

  console.log(paciente);

  // Cria a tr e a td do paciente.
  var pacienteTr = montaTr(paciente);

  // Calcila o imc.
  var tabela = document.querySelector("#tabela-pacientes");

  tabela.appendChild(pacienteTr);

  form.reset();

});

function obtemPacienteDoFormulario(form) {

  var paciente = {
    nome: form.nome.value,
    peso: form.peso.value,
    altura: form.altura.value,
    gordura: form.gordura.value,
    imc: calculaImc(form.peso.value, form.altura.value)
  }
  return paciente;
}

function montaTr(paciente) {

  var pacienteTr = document.createElement("tr")
  pacienteTr.classList.add("paciente");


  pacienteTr.appendChild(montaTd(paciente.nome, "info-nome"));
  pacienteTr.appendChild(montaTd(paciente.peso, "info-peso"));
  pacienteTr.appendChild(montaTd(paciente.altura, "info-altura"));
  pacienteTr.appendChild(montaTd(paciente.gordura, "info-gordura"));
  pacienteTr.appendChild(montaTd(paciente.imc, "info-imc"));

  return pacienteTr

}

function montaTd(dado, classe) {
  var td = document.createElement("td");
  td.textContent = dado;
  td.classList.add(classe);

  return td;
}
1 resposta

Boa noite Ewerton!

Veja como você declarou a variável pesoEhValidoaqui:

var pesoEhValido = true;

E como você a utilizou aqui:

if (peso <= 0 || peso >= 1000) {
    console.log("Peso invalido");
    pesoEhValida = false;
    tdImc.textContent = "Peso Inválido!";
    paciente.classList.add("paciente-invalido")
  }

Na declaração ela está com final "o" e dentro do if com final "a".

Testa aí e depois posta se deu certo!

Abraços!!