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

Duvida no exercicio de mudar de cor

Olá

Fiz o exercício do meu jeito mas não estava dando certo, visualizei o do professor e só tem uma coisa de diferente. O erro se encontra na minha variável var cor = cores[i]; mas não sei o porque desse erro se essa variável recebe o valor do índice do array cores. Gostaria de saber o porque que meu código não funcionou. Agradeço desde já

<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);

    function mudaCor() {

        i++;

        if(i >= cores.length) {
            i = 0;
        }

        console.log('funcionou');
        return false;
    }

    function desenhaCirculo(evento) {

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

    var i = 0;
    var cores = ['blue', 'red', 'green'];
    var cor = cores[i];

    tela.onclick = desenhaCirculo;
    tela.oncontextmenu = mudaCor;

</script>
1 resposta
solução!

Olá, Caio, é uma ótima observação.

A diferença entre o seu código e o do instrutor é que ele usa o contador (indiceCorAtual) diretamente na função desenhaCirculo e você usa ele indiretamente, dentro da variável cor. Nessa função, não há uma instrução para atualizar a variável cor e isso não ocorre de maneira automática, como no uso direto de indiceCorAtual. Então, a variável cor continua tendo o índice 0 do array cores, conforme declarado inicialmente.

Agora, se chamarmos a variável cor dentro do bloco da função desenhaCirculo (o que, com a devida substituição, acabaria dando um resultado equivalente ao do instrutor), aí ela é atualizada, veja:

    function desenhaCirculo(evento) {

      cor = cores[i];

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

O código completo ficaria assim:

<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);

    function mudaCor() {

        i++;

        if(i >= cores.length) {
            i = 0;
        }

        console.log('funcionou');
        return false;
    }

    function desenhaCirculo(evento) {

        cor = cores[i];

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

    var i = 0;
    var cores = ['blue', 'red', 'green'];
    var cor = cores[i];

    tela.onclick = desenhaCirculo;
    tela.oncontextmenu = mudaCor;

</script>

Espero ter ajudado. Bons estudos!