Estrutura do código

Quando fui fazer o exercício, estruturei o código dessa forma, não com no exemplo. É valido também, ou pode ser ruim com relação a boas práticas, ou então ter problemas com estruturas mais complexas?

// adicionando novos pacientes

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

//captando os dados

var nome = formulario.nome.value;
var peso = formulario.peso.value;
var altura = formulario.altura.value;
var gordura = formulario.gordura.value;


// botao adicionar

var botaoAdicionar = formulario.querySelector("#adicionar-paciente");


botaoAdicionar.addEventListener("click", function(event){

    event.preventDefault();

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

    var novoPacienteNome = document.createElement("td");
    novoPacienteNome.classList.add("info-nome");
    novoPacienteNome.textContent =  nome;

    var novoPacientePeso = document.createElement("td");
    novoPacientePeso.classList.add("info-peso");
    novoPacientePeso.textContent = peso;

    var novoPacienteAltura = document.createElement("td");
    novoPacienteAltura.classList.add("info-altura");
    novoPacienteAltura.textContent = altura;

    var novoPacienteGordura = document.createElement("td");
    novoPacienteGordura.classList.add("info-gordura");
    novoPacienteGordura.textContent = gordura;

    //colocando cada elemento html no seu respectivo pai

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

    tabela.appendChild(novoPaciente);
    novoPaciente.appendChild(novoPacienteNome);
    novoPaciente.appendChild(novoPacientePeso);
    novoPaciente.appendChild(novoPacienteAltura);
    novoPaciente.appendChild(novoPacienteGordura);

});