Solucionado (ver solução)
Solucionado
(ver solução)
1
resposta

Qual a melhor forma de organizar meu código?

Resolvi o exercício com o seguinte código:

<canvas width="600" height="400"></canvas>

<script>

    var tela = document.querySelector('canvas');
    var pincel = tela.getContext('2d');
    pincel.fillStyle = 'lightgray';
    pincel.fillRect(0, 0, 600, 400);

    function desenhaCirculo(x, y, raio, cor) {

        pincel.fillStyle = cor;
        pincel.beginPath();
        pincel.arc(x, y, raio, 0, 2 * Math.PI);
        pincel.fill();
    }

    function limpaTela() {

        pincel.clearRect(0, 0, 600, 400);
    }

    var raio = 20;
    var cresce = true

    function atualizaTela(){

        if(cresce){

            limpaTela();
            desenhaCirculo(50, 50, raio, "blue");
            raio++;

            if(raio>40){

                cresce = false;
            }
        }

        if(cresce==false){
            limpaTela();
            desenhaCirculo(50,50, raio, "green");
            raio--;

            if (raio<20){
                cresce=true;
            }
        }
    }


    setInterval(atualizaTela,10);

</script>

Porém vendo o tópico de outro aluno achei interessante a forma como ele organizou tudo e tentei seguir uma lógica similar, cortando os "if" e usando mais funções:

<canvas width="600" height="400"></canvas>

<script>

    var tela = document.querySelector('canvas');
    var pincel = tela.getContext('2d');
    pincel.fillStyle = 'lightgray';
    pincel.fillRect(0, 0, 600, 400);

    var raio = 20;
    var cresce = true

    function desenhaCirculo(x, y, raio, cor) {

        pincel.fillStyle = cor;
        pincel.beginPath();
        pincel.arc(x, y, raio, 0, 2 * Math.PI);
        pincel.fill();
    }

    function limpaTela() {

        pincel.clearRect(0, 0, 600, 400);
    }

    function check(){

        if(raio>40){

        cresce = false;
        } else if (raio<20){

        cresce=true;
        }
    }

    function inflador(){

        limpaTela();
        desenhaCirculo(50, 50, raio, "blue");
        raio++;
        check();
    }

    function desinflador(){

        limpaTela();
        desenhaCirculo(50,50, raio, "green");
        raio--;
        check();
    }








    function atualizaTela(){

        if(cresce){

            inflador();     
        } else {

            desinflador(); 
        }
    }


    setInterval(atualizaTela, 20);

</script>

Para quem quer trabalhar com programação qual dos 2 representa uma prática melhor?

Tenho receio de parecer aquelas pessoas que "complicam desnecessariamente as coisas" quando não precisa, algo que na área onde trabalho atualmente se torna um problema quando o assunto é trabalho em equipe.

1 resposta
solução!

Olá Alves, tudo bem com você?

Dado o exercício acredito que as duas formas são bem válidas, no mundo da programação nós temos uma tendência de fazer mais funções (igual a segunda forma), é uma prática melhor pois conseguímos reutilizar as funções, e o nosso código fica um pouco mais legível, então por exemplo:

if(cresce){
    inflaBolinha()
} else {
    desinflaBolinha()
}

É mais legível para nós, do que uma série de código:

if(cresce) {
    raio++
    [...]
} else {
    [...]
}

Então é bem comum você ver soluções que dividem o problema em mais funções

Agora a primeira forma também funciona muito bem, pois neste caso não iremos reutilizar as funções :)

Claro que podem existir times que preferem trabalhar com poucas funções, porém que resolvem o problema, mas acredito que separar em mais tende a auxiliar tanto quem escreve, quanto está lendo :)

Abraços e Bons Estudos!

Quer mergulhar em tecnologia e aprendizagem?

Receba a newsletter que o nosso CEO escreve pessoalmente, com insights do mercado de trabalho, ciência e desenvolvimento de software