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

Código falhando.

Desenvolvi esse código para solucionar o problema do gráfico de uma maneira bem simples. Mas quando eu chamo a função desenhaBarra, pela segunda vez, ela não funciona. Aparece apenas o ano, mas não aparece os retângulos. O que pode estar acontecendo?

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

<script>

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

    var serie2015 = [50, 25, 20, 5];
    var serie2016 = [65, 20, 13, 2];
    var cores = ['blue', 'green', 'yellow', 'red'];

    var contador = 0

    var posicao = 0;


    function desenhaRetangulo(x, y, largura, altura, cor) {

        pincel.fillStyle= cor;
        pincel.fillRect(x,y, largura, altura);
        pincel.strokeStyle='black';
        pincel.strokeRect(x,y, largura, altura);
    }

    function desenhaTexto(x, y, texto) {
        var tela = document.querySelector('canvas');
        var pincel = tela.getContext('2d');

        pincel.font='15px Georgia';
        pincel.fillStyle='black';
        pincel.fillText(texto, x, y);    
    }

    function desenhaBarra(x, y, serie, cores, texto) {


        while(contador < 4){

            desenhaRetangulo(x, y, 50, serie[posicao], cores[posicao]);

            y = y + serie[posicao];

            posicao++;
            contador++;


        }

        desenhaTexto(x, 40,texto);

    }


    desenhaBarra(50, 50, serie2015, cores, '2015');


    desenhaBarra(200, 50, serie2016, cores, '2016');




</script>
1 resposta
solução!

O problema no código está em deixar a variável "contador" fora do escopo da função desenhaBarra(). Deixe assim que irá funcionar:

function desenhaBarra(x, y, serie, cores, texto) {
        var contador = 0;
        var posicao = 0;

        while(contador < 4){

            desenhaRetangulo(x, y, 50, serie[posicao], cores[posicao]);

            y = y + serie[posicao];

            posicao++;
            contador++;
        }
        desenhaTexto(x, 40,texto);
    }

E tire a var contador e posição lá em cima, porque assim fica mais fácil ver onde é usada a variável e irá melhorar na execução, custando menos memória.

Obs: na função desenhaTexto() você não precisa definir novamente a var tela e var pincel, porque ja existe no escopo principal.