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

Simplifiquei tanto que não tenho certeza se está aceitável

Boa tarde, senhores(a)! Criei somente dois intervalos para a geração das bandeiras. Elas aparecem sem problema algum, ficam por 3 segundos e ocorre a troca, mas fiquei um pouco incerto se dessa forma está um código viável. Ele funciona, mas não tenho certeza se é o ideal a ser conquistado.

<!-- 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);
}
    function limpaTela() {

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

    setInterval( desenhaBandeiraAlemanha, 3000)
    setInterval( desenhaBandeiraBrasil, 6000)
</script>
1 resposta
solução!

Fala Rafael, tudo bem?

Veja, como você colocou setInterval diretamente sem atribuir a uma function, ambas as linhas que desenham as bandeiras do Brasil e da Alemanha serão executadas. Seria interessante criar uma função "trocaBandeira", exemplo:

var mostraBrasil = true;

function trocaBandeira() {

    if(mostraBrasil == true) {
        desenhaBandeiraBrasil();
        mostraBrasil = false;
    } else {
        desenhaBandeiraAlemanha();
        mostraBrasil = true;
    }
}

setInterval(trocaBandeira, 3000); // a cada 3 segundos, chama trocaBandeira 

Pela lógica ficou muito bom, você elaborou outra solução para o problema, mas de uma perspectiva de boas práticas de código daria para evoluir =)

Abraço!