É recomendável fazer dessa forma?

<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 posicao = 0;
    var raio = 10;

    function desenhaCirculo(evento){

        var x = evento.pageX - tela.offsetLeft;
        var y = evento.pageY - tela.offsetTop;
        pincel.fillStyle = cores[posicao];
        pincel.beginPath();

        if(evento.shiftKey){

            raio = raio + 10;

            while(raio > 40){            
                raio = raio - 10;
            }
        }

        if(evento.altKey){

            raio = raio - 5;

            while(raio < 10){

                raio = raio + 5;
            }
        }

        pincel.arc(x, y, raio, 0, 2 * 3.14);
        pincel.fill();
    }

    function mudaCor(){

        posicao++
        if(posicao  > 3){
            posicao = 0;
        }

        return false;                    
    }    

    tela.onclick = desenhaCirculo;

    tela.oncontextmenu = mudaCor;

</script>