Solucionado (ver solução)
Solucionado
(ver solução)
3
respostas

Porque não muda a cor?????

Alguém pode me explicar porque minha bolinha não muda de cor? Eu entendi a resolução do professor, mas queria entender se meu raciocínio está errado, se sim, onde; pra que eu aprenda.

<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 indiceCorAtual = 0;

    function desenhaCirculo(evento) {

        var x = evento.pageX - tela.offsetLeft;
        var y = evento.pageY - tela.offsetTop;

        pincel.fillStyle = cores[indiceCorAtual];
        pincel.beginPath();
        pincel.arc(x, y, 10, 0, 2 * 3.14);
        pincel.fill();
        console.log(x + ',' + y);
    }

    tela.onclick = desenhaCirculo;

    function mudaCor() {

        for (var i = 0; i < cores.length; i++) {
            alert('funcionou!!');
            return false;
        }
    }

    tela.oncontextmenu = mudaCor;
    desenhaCirculo(cores[i]);

</script>
3 respostas

Olá Amanda, tudo bem?

Para funcionar corretamente, dentro da function mudaCor() você precisa fazer um indiceCorAtual++; conforme exemplo abaixo:

    function mudaCor() {

        indiceCorAtual++;

        if(indiceCorAtual >= cores.length) {
            indiceCorAtual = 0; // volta para a primeira cor, azul
        }

        return false; // para não exibir o menu padrão do canvas
    }

Obrigada. Sabe me dizer porque o for não dá certo aqui? Eu segui o método do exercício do gráfico pra fazer esse, nele foi feito com o for.

solução!

No caso do for você estava iterando N vezes considerando o tamanho do array em cores.length, porém o indiceCorAtual não foi alterado em nenhum momento, e por isso quando o desenhaCirculo for chamado o pincel.fillStyle = cores[indiceCorAtual]; irá considerar sempre a mesma cor

for (var i = 0; i < cores.length; i++) {
            alert('funcionou!!');
            return false;
        }