Ya estoy inscrito ¿Todavía no tienes acceso? Nuestros Planes
Ya estoy inscrito ¿Todavía no tienes acceso? Nuestros Planes
Solucionado (ver solución)
Solucionado
(ver solución)
3
respuestas

No me da Alert ..

buenas gente.. mire millones de veces y no encuentro el error..

cuando le pego a la Diana no me da sale el pop up.. alguien puede decirme que no estoy viendo..

desde ya muchas gracias.

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

<script>
    var pantalla = document.querySelector("canvas");
    var pincel = pantalla.getContext("2d");    
    pincel.fillStyle = "lightgrey";
    pincel.fillRect(0,0,600,400); 

    var radio = 10;
    var xAleatorio;
    var yAleatorio;

    function disenharCircunferencia(x,y,radio,color) {
        pincel.fillStyle = color,
        pincel.beginPath();
        pincel.arc(x,y,radio,0,2*Math.PI);
        pincel.fill();
    }

    function limpiarPantalla(){

        pincel.clearRect(0,0,600,400);
    }

    var x = 0;



    function disenharObjetivo(x,y) {

        disenharCircunferencia(x,y,radio+20,"red");
        disenharCircunferencia(x,y,radio+10,"white");
        disenharCircunferencia(x,y,radio,"red");
    }

    function sortearPosicion(maximo) {

        return Math.floor(Math.random()*maximo); //floor redonde para abajo

    }

    function actualizarPantalla() {

        limpiarPantalla();
        var xAleatorio = sortearPosicion(600);
        var yAleatorio = sortearPosicion(400);
        disenharObjetivo(xAleatorio, yAleatorio);
        x++;
    }    

    setInterval(actualizarPantalla,1500);

    function disparar(evento){

        var x = evento.pageX - pantalla.offsetLeft;
        var y = evento.pageY - pantalla.offsetTop;

        if ((x < xAleatorio + radio) &&
            (x > xAleatorio - radio) &&
            (y < yAleatorio + radio) &&
            (y > yAleatorio - radio) ) {

            alert("tiro certero");

        }

    }

    pantalla.onclick = disparar;

</script> 
3 respuestas
solución!

Tienes un error en esta parte de tu código

function actualizarPantalla() {

        limpiarPantalla();
        var xAleatorio = sortearPosicion(600);
        var yAleatorio = sortearPosicion(400);
        disenharObjetivo(xAleatorio, yAleatorio);
        x++;
    }  

tus variales xAleatorio y yAleatorio, las estás volviendo a declarar en tu función y no estás usando las ya declaradas. Simplemente elimina la palabra var que tienes en esas variables dentro del método, y todo funcionará perfectamente.

si esta respuesta te ayudo, marcala como solucion

Exactamente. Vuelves a declarar las variables y por el scope no toma los antiguos valores.

gracias genteeee.. que dificil es ver cuando estas cegado.. buenas noches