Como ven, ¿podría mejorar el código o así esta bien?

<meta charset="UTF-8">
<h1>JUEGO LOTERIA (Adivina el numero)</h1>

<script>

    function saltoLinea() {

        document.write("<br>");

    }

    function imprimir(frase) {

        document.write(frase);
        saltoLinea();

    }

    function loteria() {

        return Math.round(Math.random()*numeroMaximo);
    }

    var numeroMaximo = parseInt(prompt("Ingrese valor maximo del numero para jugar"));
    var numeroPensado = loteria ();
    var contador = 1;
    var intentos = 3;


    while (contador<=intentos) {
    var numeroLanzado = parseInt(prompt("Ingrese un numero entre el 1 y el " + numeroMaximo));

    if (numeroPensado == numeroLanzado) {

        imprimir("Acertaste en el intento " + contador + " el numero es: " + numeroPensado + ".");
        break;

    }

    else {
        if (numeroPensado < numeroLanzado) {
            alert("Error, el numero es menor. Intento " + contador + ".");

        }
        else {
            alert("Error, el numero es mayor. Intento " + contador + ".");
        }

    }

    contador++;

    }

        if (numeroPensado < numeroLanzado){

            imprimir("Error, el numero pensado era menor que tu numero (era " + numeroPensado + ").");
        }

        if (numeroPensado > numeroLanzado)  {
            imprimir("Error, el numero pensado era mayor que tu numero (era " + numeroPensado + ").");
        }

    imprimir("Fin del juego");

</script>