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

fiquei em duvida

Fiquei em dúvida se meu código ficou simples de se entender!

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

<script>


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

    function desenhaRetangulo(x, y, largura, altura, cor) {
        var tela = document.querySelector('canvas');
        var pincel = tela.getContext('2d');

        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, porcentagem, cores, texto){
        var tela = document.querySelector('canvas');
        var pincel = tela.getContext('2d');

        desenhaTexto(x, y, texto);
        var somaY = 20;
        for(var posicao = 0;posicao < 4;posicao++){

            desenhaRetangulo(x, y + somaY , 50, porcentagem[posicao], cores[posicao]);
            somaY = somaY + porcentagem[posicao];
        }

    }

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

    desenhaBarra(50, 50, serie2015, cores, '2015');
    desenhaBarra(150, 50, serie2016, cores, '2016');

</script>

O que eu poderia mudar para deixar ele mais enxuto?

1 resposta
solução!

Oi Patrick, vc pode remover a repetição das var tela e pincel

Sugiro, também, vc usar const ou let no lugar do var


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

    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) { 
        pincel.font='15px Georgia';
        pincel.fillStyle='black';
        pincel.fillText(texto, x, y);    
    }

    function desenhaBarra(x, y, porcentagem, cores, texto){
        desenhaTexto(x, y, texto);
        var somaY = 20;
        for(var posicao = 0;posicao < 4;posicao++){
            desenhaRetangulo(x, y + somaY , 50, porcentagem[posicao], cores[posicao]);
            somaY = somaY + porcentagem[posicao];
        }
    }

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

    desenhaBarra(50, 50, serie2015, cores, '2015');
    desenhaBarra(150, 50, serie2016, cores, '2016');