Boa noite!
Tudo bem?
A máxima da programação que eu entendi é: "Se funcionou, está OK."
Porém, temos meios para otimizar o nosso código para deixá-lo mais enxuto, legível e "elegante". Você vai ouvir bastante isso... kkk
O código abaixo é o mesmo da nossa aula para você comparar com o seu.
A diferença que eu inseri o Math.round para arredondar os valores do IMC.
*** Moderadores, se houver mais algum comentário, agradeço, assim também aprenderei. Ficarei acompanhando. 
<meta charset="UTF-8">
<script>
    function pulaLinha() {
        document.write("<br>");
        document.write("<br>");
}
    function mostra(frase) {
        document.write(frase);
        pulaLinha();
}
   function calculaImc(altura, peso) {
    var imc = peso / (altura * altura);
    return Math.round(imc);
}
var imcFlavio = calculaImc(1.71, 73);
var imcAmigo = calculaImc(1.72, 68);
mostra(imcFlavio);
mostra(imcAmigo);
</script>
Sucesso no curso!