Oi, Douglas, tudo bem?
Primeira coisa, o seu código funciona de acordo com o que é pedido. Mas eu mudaria algumas coisas:
A função mostra ela tem um objetivo diferente, que é o de mostrar uma mensagem ou qualquer outra informção. Então, o interessante dessa função é que ela tenha esse objetico para ser reutilizada quando você quiser nesse programa. A priori, você nesse programa não precisa, mas pensa que se fosse um programa mais complexo, essa função sendo mais neutra ela será reutilizável, assim como você fez, ela terá a função de mostrar os valores e o cálculo que você fez.
O meu código desse programinha ficou assim:
<script type="text/javascript">
function pulaLinha(){
document.write("<br>")
}
function mostra(frase){
document.write(frase)
pulaLinha();
}
var ano = 2019;
var anosPassaram = ano - 1500;
var mediaAnos = anosPassaram/28;
mostra("Passaram " + mediaAnos.toFixed(0) + "gerações!");
</script>
O mais interessante é você chamar a função mostra quando qusier mostrar algo na tela.
Espero ter te ajudado!