[Projeto] Será que o projeto está com boas práticas de programação?

Percebi que não preciso fazer a variável :

var numeroSorteado = sorteio ();

Que colocando sorteio (); ele chama a função e me retorna a string da condição a qual ele se iguala. Isso seria certo? Ou uma boa prática impor a variavel numeroSorteado = função sorteio?

<meta charset="UTF-8">

<script>

    function pulaLinha() {
        document.write("<br>");
        document.write("<br>");
    }

    function mostra(frase) {
        document.write(frase);
        pulaLinha();
    }

    function sorteio() {
        numeroSorteado = Math.round(Math.random() * 10);
        return numeroSorteado
   } 

    var nome = prompt("Qual o seu nome?"); 
    var chute = parseInt(prompt(nome + ", Tente adivinhar um número de 0 a 10!"));

    sorteio();      
    if(chute == numeroSorteado) {
        mostra("Você acertou! O número realmente era " + numeroSorteado);
        mostra("Atualize a página para jogar novamente!");
    } else {
        mostra("Você errou!O número sorteado foi " + numeroSorteado);
        mostra("Atualize a página para jogar novamente!");
    }

</script>