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

A mensagem de erro aparece, mas continua adicionando o valor inválido na tabela

Aparece a mensagem de erro, mas o paciente inválido continua sendo adicionado à tabela. Não consigo encontrar o erro

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

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


    // Extraindo informações do formulário
    var paciente = obtemPacienteDoFormulario(form);

    // Cria Tr e Td

    var pacienteTr = montaTr(paciente);

    var erro = validaPaciente(paciente);


    if(erro.length > 0) {
        var mensagemErro = document.querySelector("#mensagem-erro");
        mensagemErro.textContent = erro;
        return false;
    }

    // Adicionando

    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"));

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

    tabela.appendChild(pacienteTr);

    return pacienteTr;
};

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

function validaPaciente(paciente) {
    if(validaPeso(paciente.peso)) {
        return "";
    }else{
        return "O peso é inválido";
    }
}
var titulo = document.querySelector(".titulo");


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 imc = peso / (altura*altura);

tdImc.textContent = imc;

var pesoValido = validaPeso(peso);
var alturaValida = validaAltura(altura);

if (!pesoValido) {
    pesoValido = false
    tdImc.textContent = "Peso inválido";
};

if (!alturaValida) {
    alturaValida = false
    tdImc.textContent = "Altura inválida";
};

if (pesoValido == true && alturaValida == true) {
    var imc = peso / (altura*altura);

    tdImc.textContent = imc.toFixed(2);

};
}

function validaPeso(peso) {
    if(peso >= 0 && peso <= 1000) {
        return true;
    }
    else {
        return false;
    }
}

function validaAltura(altura) {
    if(altura >= 0 && altura <= 3.0) {
        return true;
    }else{
        return false;
    }
}

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

Arthur, poste a imagem do erro e também poste cole seu html aqui para poder testar

<!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 class="container">
    <h2 id="titulo-form">Adicionar novo paciente</h2>
    <span id="mensagem-erro"></span>
    <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>

Aparece a mensagem de erro, mas o paciente inválido é incorporado à tabela mesmo assim.

Insira aqui a descrição dessa imagem para ajudar na acessibilidade

Poste também o código que está nesse js:


<script src="js/form.js"></script>
solução!

Bom dia Arthur,

O código é lido de cima para baixo, na imagem 1, na linha 10, tem a sua variável "erro" que armazena o retorno da função validaPaciente, e logo abaixo tem a variável "pacienteTr", para depois tratar o erro, ou seja, você está inserindo o paciente na tela para só depois disso tratar o erro, mas ai já é tarde, pois o paciente já está na tela.

Para resolver o problema, basta você alterar o posicionamento da sua variável, conforme a imagem 2. Ele só vai inserir na tela depois de passar pelo seu if.

correcao

Deu certo. Muito obrigado!