Se podria poner mejor if(!pesovalido) en el else? de forma que solo si da false, automaticamente pase a esa funcion en vez de crearla?
else{ tdIMC.textContent = "Peso incorrecto"; paciente.classList.add("paciente-incorrecto"); }
y lo mismo para la altura?.
