meu codigo -minha dúvida!

Olá, gostaria de saber se na parte da função dispara(evento) posso deixar um if com todos os parâmetros dentro ou isso não seria uma boa prática?

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

<script>

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

    pincel.fillStyle = 'lightgray';
    pincel.fillRect(0, 0, 600, 400);

    var raio = 10;

    function desenhaCirculo(x, y, raio, cor) {

        pincel.fillStyle = cor;
        pincel.beginPath();
        pincel.arc(x, y, raio, 0, 2 * Math.PI);
        pincel.fill();
    }

    desenhaCirculo(300,200, raio + 20, 'red'); // maior círculo
    desenhaCirculo(300,200, raio + 10, 'white');
    desenhaCirculo(300,200, raio, 'red'); // menor circulo

    function dispara(evento) {

        var x = evento.pageX - tela.offsetLeft;
        var y = evento.pageY - tela.offsetTop;
        if (x <=310 && x >=290 && y >=190 && y <=210)
        {
            tela. onmousedown = alert ("acertou") 

        }else{
            tela.onmousedown = alert ("foi quase");
        }

       // lógica de acerto?
    }

    tela.onclick = dispara;

</script>