1
resposta

ajuda!

alguém sabe o que posso melhorar nesse código?

<meta charset="UTF-8">

<header>

    <title>Jogo de Adivinhação</title>

</header>

<body>
<big>Insira números de 1 a 10!</big>
<br><br>
<big>Aposta: </big><input/>
<br><br>
<button>Confirma</button>

</body>

<script>

    function atualizar()
    {
        window.location.reload();
    }

    function sorteio()
    {
        return Math.floor(Math.random() * 10) + 1;
    }

    segredo = sorteio();
    console.log(segredo);
    var input = document.querySelector("input");



    function verifica()
    {
        if(input.value <= 10 && input.value >= 0)
        {

            if(input.value == segredo)
            {    
                alert("Você Acertou!");
                document.write("Recarregando Pagina...");
                setTimeout('atualizar()', 2500);

            } else
            {
                alert("Você Errou!");
                document.write("Recarregando Pagina...");
                setTimeout('atualizar()', 2500);
            }    
        }

        if(input.value >= 11)
        {
            alert("Por favor digitar números entre 1 e 10!");
            document.write("Recarregando Pagina...");
            setTimeout('atualizar()', 2500);
        }

        if(input.value < 0)
        {
            alert("Por favor digitar números entre 1 e 10!");
            document.write("Recarregando Pagina...");
            setTimeout('atualizar()', 2500);    
        }    
    }

    var button = document.querySelector("button");
    button.onclick = verifica;
</script>
1 resposta

Bom dia.

Notei constante repetição de código. Segue sugestão:

<meta charset="UTF-8">

<header>

    <title>Jogo de Adivinhação</title>

</header>

<body>
<big>Insira números de 1 a 10!</big>
<br><br>
<big>Aposta: </big><input/>
<br><br>
<button>Confirma</button>

</body>

<script>

    function atualizar()
    {
        window.location.reload();
    }

    function sorteio()
    {
        return Math.floor(Math.random() * 10) + 1;
    }

    segredo = sorteio();
    console.log(segredo);
    var input = document.querySelector("input");



    function verifica()
    {
        if(input.value <= 10 && input.value >= 0){
            if(input.value == segredo)
                alert("Você Acertou!");
            else
                alert("Você Errou!");            
        }else        
            alert("Por favor digitar números entre 1 e 10!");

        document.write("Recarregando Pagina...");
        setTimeout('atualizar()', 2500);
    }

    var button = document.querySelector("button");
    button.onclick = verifica;
</script>