Seria essa uma má prática?

Eu vejo o tempo todo falar sobre "boas práticas". Antes de chegar nessa aula, decidi me arriscar para ver se conseguia criar a coluna do IMC e consegui. Mas seria essa forma que eu fiz um jeito "não recomendado", uma má prática? O melhor seria criar funções mesmo?

var botaoAdicionar = document.querySelector("#adicionar-paciente");
botaoAdicionar.addEventListener("click",function(event) {
    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 imc = form.peso.value/(form.altura.value*form.altura.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;
    imcTd.textContent = imc.toFixed(2);

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

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


});