2
respostas

Resolvi o exercício sem utilizar o booleano

A quem interessar possa, o meu código ficou da seguinte forma e é uma outra opção de como resolver o exercício (achei a do professor muito melhor pensada, pra ser sincero):

<!-- bandeiras.html -->

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

<script>

function desenhaBandeiraBrasil() {
    var tela = document.querySelector('canvas');
    var pincel = tela.getContext('2d');

    pincel.fillStyle="green";
    pincel.fillRect(0, 0, 600, 400);

    pincel.fillStyle="yellow";
    pincel.beginPath();
    pincel.moveTo(300, 50);
    pincel.lineTo(50, 200);
    pincel.lineTo(550, 200);
    pincel.fill();

    pincel.beginPath();
    pincel.moveTo(50, 200);
    pincel.lineTo(300, 350);
    pincel.lineTo(550, 200);
    pincel.fill();

    pincel.fillStyle="darkblue";
    pincel.beginPath();

    pincel.arc(300, 200, 100, 0, 2* 3.14);
    pincel.fill();
}


function desenhaBandeiraAlemanha() {
    var tela = document.querySelector('canvas');
    var pincel = tela.getContext('2d');

    pincel.fillStyle = 'black';
    pincel.fillRect(0, 0, 600, 133);

    pincel.fillStyle = 'red';
    pincel.fillRect(0, 133, 600, 133);

    pincel.fillStyle = 'yellow';
    pincel.fillRect(0, 266, 600, 133);
}

var contador = 1;

function trocaBandeira(){

        if(contador == 1){

            desenhaBandeiraBrasil();

        }

        else if(contador == 2){

            desenhaBandeiraAlemanha();
        }

        else if(contador > 2){

            contador = 0;
        }

        contador++;
}

setInterval(trocaBandeira, 3000);
</script>
2 respostas

Fala Mike, tudo certinho?

Testando teu código notei que a Bandeira da Alemanha está sendo exibida por mais tempo que a bandeira do Brasil. Isso se dá pelo fato de que até o contador voltar a ter o valor 1 é realizado um novo ciclo no setInterval. Sugiro você remover o último else if e colocar o contador = 0 logo após a chamada da função desenhaBandeiraAlemanha, ficando a function desta forma:

unction trocaBandeira(){

        if(contador == 1){

            desenhaBandeiraBrasil();

        }

        else if(contador == 2){

            desenhaBandeiraAlemanha();
            contador = 0;
        }

        contador++;
}

Faz muito sentido, cara. Meu código estava dando mais uma processada sem nenhuma utilidade. Valeu pela dica!