1
resposta

Críticas e opiniões.

Bom dia, tarde ou noite!

Após concluir o curso de Lógica de programação 1, eu criei este programa para facilitar a vida dos meus amigos. O programa sortea 10 pessoas em 2 times e designa sua função no jogo League of Legends.

<meta charset="UTF-8">

<h1> Insira o nome do jogador</h1>
<br>
<input/>
<button>ENTER</button>
<button type="button" onclick="finaliza()">SORTEAR</button>

<script>

    function pulaLinha() {

        document.write("<br>");
    }

    function show(string){

        document.write(string);
        pulaLinha();
    }

    function sorteia() {

       return Math.round(Math.random() * 10);

    }

    var ordem = [];

    var n = 0;

    while (n <= 9){

         var preOrdem = sorteia();
         var error = false;

         if (preOrdem != 10){

             for (var num = 0; num < ordem.length; num++){

                 if(ordem[num] == preOrdem){
                     error = true;
                     break;
                 }
             }    

             if(error ==false){

                 ordem.push(preOrdem);
                 n++
             }
         }


    }

    var players = [];
    var input = document.querySelector ("input");
    input.focus();

    function adiciona(){

        players.push(input.value);
        input.focus();
        input.value = "";
    }

    function finaliza(){

        show ("TIME 1");
        show ("TOP: " + players[ordem[0]]);
        show ("JUNGLE: " + players[ordem[1]]);
        show ("MID: " + players[ordem[2]]);
        show ("ADC: " + players[ordem[3]]);
        show ("SUP: " + players[ordem[4]]);
        pulaLinha();
        show ("TIME 2");
        show ("TOP: " + players[ordem[5]]);
        show ("JUNGLE: " + players[ordem[6]]);
        show ("MID: " + players[ordem[7]]);
        show ("ADC: " + players[ordem[8]]);
        show ("SUP: " + players[ordem[9]]);

    }

    var button = document.querySelector("button");
    button.onclick = adiciona;


</script>

Gostaria de saber opiniões e sugestões de pessoas mais experientes. Att. Ari Soares Ramos Júnior

1 resposta

Fala Ari, tudo bem? Espero que sim!

Parabéns pelo programa!

Ficou excelente!!!!

O que eu tenho a recomendar seria a nomeação das variáveis, tentar utilizar nomes mais esclarecedores, que dizem qual a função dela entende? Como por exemplo, nesse trecho de código:


    var ordem = [];

    var n = 0;

Não fica tão esclarecedor o que significa a variável n, assim fica mais dificil a compreensão do código. Então seria interessante dar uma olhadinha no código e ver quais nomes de veriáveis podem ser alteradas para que fiquem mais esclarecedoras!

Abraços e bons estudos :D