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

Problema ao calcular IMC, valor sempre vem zerado.

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

    var form = document.querySelector("#form-adiciona");

    //Extraindo informações do paciente do FORM
    var paciente = obterPacienteDoFormulario(form);

    //Criar TR e a TD do paciente
    var pacienteTr = montaTr(paciente); 

    console.log(pacienteTr);
    //Adicionando paciente na tabela
    var tabela = document.querySelector("#tabela-pacientes");
    tabela.appendChild(pacienteTr);

    //limpa formulário
    form.reset();
});

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

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

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

        return pacienteTr;
    }

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

        return td;
    }
//var titulo = document.querySelector(".titulo");
//            console.log(titulo.textContent);
//            titulo.textContent = "Foi alterado";
//var paciente = document.querySelector("#primeiro-paciente");
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 pesoEnvalido = true;
    var alturaEnvalido = true;

    if(peso < 0 || peso >= 300){
        pesoEnvalido = false;
        tdImc.textContent = "Peso inválido!";
        //paciente.style.backgroundColor = "lightcoral"; // alterando cor de fundo da linha. (Obs.:background-color esse padrão do CSS para o javaScript quando tem duas palavaras a união entre as daus deve ser feito com letra MAIÚSCULA).
        paciente.classList.add("paciente-invalido"); // classList.add: adiciona uma classe nova no meu html.
    }
    if(altura < 0 || altura >= 2.8){
        alturaEnvalido = false;
        tdImc.textContent = "Altura inválida!";
        //paciente.style.backgroundColor = "lightcoral";
        paciente.classList.add("paciente-invalido");
    }

    if(alturaEnvalido && pesoEnvalido){
        var imc = calcularImc(peso,altura);
        console.log(imc);
        tdImc.textContent = imc; 
    }

    function calcularImc(peso,altura){
        var imc = 0;
        imc = peso / (altura * altura);
        return imc.toFixed(2);// toFixed: limita o número de casas decimais.
    }

}
5 respostas

Oi Emerson,

O código JavaScript parece estar ok.

Poste aqui o seu código HTML, para que possamos tentar entender o que pode estar acontencendo.

<!DOCTYPE html>
<html lang="pt-br">
    <head>
        <meta charset="UTF-8">
        <title>Aparecida Nutrição</title>
        <link rel="icon" href="favicon.ico" type="image/x-icon">
        <link rel="stylesheet" type="text/css" href="css/reset.css">
        <link rel="stylesheet" type="text/css" href="css/index.css">
    </head>
    <body>

        <header>
            <div class="container">
                <h1 class="titulo">Aparecida Nutrição</h1>
            </div>
        </header>
        <main>
            <section class="container">
                <h2>Meus pacientes</h2>
                <table>
                    <thead>
                        <tr>
                            <th>Nome</th>
                            <th>Peso(kg)</th>
                            <th>Altura(m)</th>
                            <th>Gordura Corporal(%)</th>
                            <th>IMC</th>
                        </tr>
                    </thead>
                    <tbody id="tabela-pacientes">
                        <tr class="paciente" id="primeiro-paciente" >
                            <td class="info-nome">Paulo</td>
                            <td class="info-peso">100</td>
                            <td class="info-altura">2.00</td>
                            <td class="info-gordura">10</td>
                            <td class="info-imc">0</td>
                        </tr>

                        <tr class="paciente" >
                            <td class="info-nome">João</td>
                            <td class="info-peso">80</td>
                            <td class="info-altura">1.72</td>
                            <td class="info-gordura">40</td>
                            <td class="info-imc">0</td>
                        </tr>

                        <tr class="paciente" >
                            <td class="info-nome">Erica</td>
                            <td class="info-peso">-54</td>
                            <td class="info-altura">1.64</td>
                            <td class="info-gordura">14</td>
                            <td class="info-imc">0</td>
                        </tr>

                        <tr class="paciente">
                            <td class="info-nome">Douglas</td>
                            <td class="info-peso">85</td>
                            <td class="info-altura">-1.73</td>
                            <td class="info-gordura">24</td>
                            <td class="info-imc">0</td>
                        </tr>
                        <tr class="paciente" >
                            <td class="info-nome">Tatiana</td>
                            <td class="info-peso">46</td>
                            <td class="info-altura">1.55</td>
                            <td class="info-gordura">19</td>
                            <td class="info-imc">0</td>
                        </tr>
                    </tbody>
                </table>

            </section>
        </main>
        <!-- ... -->
    </section>
</main>
<section class="container">
    <h2 id="titulo-form">Adicionar novo paciente</h2>
    <ul id="mensagens-erro">

    </ul>
    <form id="form-adiciona">
        <div class="grupo">
            <label for="nome">Nome:</label>
            <input id="nome" name="nome" type="text" placeholder="digite o nome do seu paciente" class="campo">
        </div>
        <div class="grupo">
            <label for="peso">Peso:</label>
            <input id="peso" name="peso" type="text" placeholder="digite o peso do seu paciente" class="campo campo-medio">
        </div>
        <div class="grupo">
            <label for="altura">Altura:</label>
            <input id="altura" name="altura" type="text" placeholder="digite a altura do seu paciente" class="campo campo-medio">
        </div>
        <div class="grupo">
            <label for="gordura">% de Gordura:</label>
            <input id="gordura" type="text" placeholder="digite a porcentagem de gordura do seu paciente" class="campo campo-medio">
        </div>

        <button id="adicionar-paciente" class="botao bto-principal">Adicionar</button>
    </form>
</section>
        <script src="js/calcula-imc.js"></script>
        <script src="js/form.js"></script>
    </body>
</html>

Testei aqui e vi que ele calcula o IMC certinho dos registros que já estão na tabela(fixos no html), mas quando adiciona um novo paciente pelo formulário, a coluna IMC vem zerada.

Isso acontece porque o código JavaScript que calcula o IMC é chamado apenas uma vez, quando a página é carregada pelo navegador.

Pra resolver esse problema, temos que chamar o código que calcula o IMC também quando um novo paciente for adicionado no formulário.

Dá para fazer isso adicionando todo o código de caclulo de IMC em uma função:

function calcularIMCDeTodosOsPacientes() {
    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 pesoEnvalido = true;
        var alturaEnvalido = true;

        if(peso < 0 || peso >= 300){
            pesoEnvalido = false;
            tdImc.textContent = "Peso inválido!";
            //paciente.style.backgroundColor = "lightcoral"; // alterando cor de fundo da linha. (Obs.:background-color esse padrão do CSS para o javaScript quando tem duas palavaras a união entre as daus deve ser feito com letra MAIÚSCULA).
            paciente.classList.add("paciente-invalido"); // classList.add: adiciona uma classe nova no meu html.
        }
        if(altura < 0 || altura >= 2.8){
            alturaEnvalido = false;
            tdImc.textContent = "Altura inválida!";
            //paciente.style.backgroundColor = "lightcoral";
            paciente.classList.add("paciente-invalido");
        }

        if(alturaEnvalido && pesoEnvalido){
            var imc = calcularImc(peso,altura);
            console.log(imc);
            tdImc.textContent = imc; 
        }

        function calcularImc(peso,altura){
            var imc = 0;
            imc = peso / (altura * altura);
            return imc.toFixed(2);// toFixed: limita o número de casas decimais.
        }

    }
}
calcularIMCDeTodosOsPacientes();

E chamar essa função após o cadastro pelo formulário:

botaoAdicionar.addEventListener("click",function(event){
    //aqui continua o seu codigo da funcao...

  //chama a nova funcao para calcular o IMC dos pacientes:
  calcularIMCDeTodosOsPacientes();
});

Começou a dar outro erro agora: form.js:46 Uncaught ReferenceError: calcularImc is not defined at obterPacienteDoFormulario (form.js:46) at HTMLButtonElement. (form.js:8)

solução!

Consegui resolver o problema, eu estava passando os parâmetros do calcularImc errado, primeiro altura depois peso, fiz essa inversão e o problema foi sanado, não houve necessidade de fazer toda essa alteração falada pelo Rodrigo Ferreira, mas mesmo assim obrigado Rodrigo, valeu. imc: calcularImc(form.peso.value,form.altura.value)

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