Ainda não tem acesso? Estude com a gente! Matricule-se
Ainda não tem acesso? Estude com a gente! Matricule-se

Solucionado (ver solução)

Exercício - Trocando a cor das bolinhas

Olá pessoal,

Fiz o exercício de trocar a cor da bolinha ao clicar com o botão direto do mouse, porém percebi que está repetindo a cor verde. Alguém pode me ajudar a encontrar o erro? A ideia é a cada clique alterar a cor da bolinha, alternando entre as três cores (blue, red e green).

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

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

    var cores = ["blue", "red", "green"];
    var alteraCor = 0;

    function desenhaCirculo(evento) {

        var x = evento.pageX - tela.offsetLeft;
        var y = evento.pageY - tela.offsetTop;
        pincel.fillStyle = cores[alteraCor];
        pincel.beginPath();
        pincel.arc(x, y, 10, 0, 2 * 3.14);
        pincel.fill();
        console.log(x + ',' + y);
    }

    tela.onclick = desenhaCirculo;

    function mudaCor() {

        if(alteraCor >= 3){

            return alteraCor = 0;
        } else{
        return alteraCor++;
    }
    }

    tela.oncontextmenu = mudaCor;

</script>
2 respostas

Robert, bom dia!

O problema está na lina abaixo

return alteraCor++;

Quando a variável corEscolhida chega em 2 ela passa pelo primeiro if por não ser maior nem igual a 3

if(alteraCor >= 3){

            return alteraCor = 0;
        }

logo depois ela é incrementada no else abaixo, fazendo com que ela fique igual a 3. Só que a array só vai até dois, então ele repete o último valor da array, que é 'green'. Para corrigir isso tente alterar o if, como no exemplo:

  function mudaCor() {
        if(alteraCor >=  2){
            return alteraCor = 0;
        } else{
          return alteraCor++;
        }
    }
solução

Oi Robert esse erro é muito comum, acontece que a contagem do indíce do vetor inicial em 0, logo um vetor de 3 posições tem as posições { 0, 1 ,2 } ele não possui a posição 3, logo quando na lógica temos:

alteraCor >= 3

Ele demora duas vezes para sair do verde e reiniciar pro vermelho. Basta ajustar a lógica para ir até o 2 que é a última posição do nosso vetor de 3 posições.

if(alteraCor >= 2){

Espero ter ajudado e bons estudos.