Solucionado (ver solução)
Solucionado
(ver solução)
2
respostas

Botão adicionar adiciona 4 vezes os dados

Segui a ordem detalhadamente, como ensinada passo-a-passo pelo professor, e, quando clico em "Adicionar", ao invés de adicionar os valores digitados, essas informações são adicionadas 4(!) vezes! meu código é esse: var pacientes = document.querySelectorAll(".paciente");

for(var i = 0; i < pacientes.length ; i++){ var paciente = pacientes[i]; var tdPeso = paciente.querySelector(".info-peso"); var tdAltura = paciente.querySelector(".info-altura");

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

var peso = tdPeso.textContent;
var altura = tdAltura.textContent;

var alturaEhValida = true;
var pesoEhValido = true;
var botaoAdicionar = document.querySelector("#adicionar-paciente");

botaoAdicionar.addEventListener("click", function(){ event.preventDefault(); var form = document.querySelector("#form-adiciona"); var nome = form.nome.value; var peso = form.peso.value; var altura = form.altura.value; var gordura = form.gordura.value; var pacienteTr = document.createElement("tr");

var nomeTd = document.createElement("td");
var pesoTd = document.createElement("td");
var alturaTd = document.createElement("td");
var gorduraTd = document.createElement("td");
var imcTd = document.createElement("td");

nomeTd.textContent = nome;
pesoTd.textContent = peso;
alturaTd.textContent = altura;
gorduraTd.textContent = gordura;

pacienteTr.appendChild(nomeTd);
pacienteTr.appendChild(pesoTd);
pacienteTr.appendChild(alturaTd);
pacienteTr.appendChild(gorduraTd);

var tabela = document.querySelector("#tabela-pacientes");

tabela.appendChild(pacienteTr);

});

if(peso <= 0 || peso >= 1000){
    console.log("Peso inválido");
    tdImc.textContent = "Peso inválido!";
    pesoEhValido = false;

    paciente.classList.add("paciente-invalido");
}

if(altura <= 0 || altura >= 3.00){
    console.log("Altura inválida");
    tdImc.textContent = "Altura inválida!";
    alturaEhValida = false;

    paciente.classList.add("paciente-invalido");
}

if(pesoEhValido && alturaEhValida){
    var imc = peso / ( altura * altura);    
    tdImc.textContent = imc.toFixed(2);
}    

}

2 respostas

eu também estou com o mesmo problema, e ainda não conseguir resolver .

solução!

Alterei alguns códigos conforme assistindo a aula, e no meu caso resolveu dessa forma

alterando if:

}

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

}

E criando uma nova função, como mostra o vídeos do professor para ter uma boa pratica:

}

function calculaImc(peso, altura) {
        var imc = 0;
        imc = peso / (altura * altura);
        return imc.toFixed(2)

}