web-dev-qa-db-fra.com

Afficher la position x et y de la souris avec javascript

La première version ne me montre pas x et y et je reçois l'erreur suivante: Uncaught TypeError: Cannot read property 'pageX' of undefined La deuxième version fonctionne mais son code est très similaire, je ne trouve pas le problème. 

PREMIÈRE VERSION (NE FONCTIONNE PAS)

    <form name ="show">
        <input type="text" name="mouseXField" value="0" size="6">Mouse X Position<br>
        <input type="text" name="mouseYField" value="0" size="6">Mouse Y Position<br>
    </form>

        <script type="text/javascript">

        var mie = (navigator.appName == "Microsoft Internet Explorer") ? true : false;

        if (!mie) {

            document.captureEvents(Event.MOUSEMOVE);
            document.captureEvents(Event.MOUSEDOWN);
        }


        document.onmousemove = mousePos();
        document.onmousedown = mouseClicked();

        var mouseClick;
        var keyClicked;

        var mouseX = 0;
        var mouseY = 0;

        function mousePos (e) {

            if (!mie) {
                mouseX = e.pageX; 
                mouseY = e.pageY;
            }
            else
                mouseX = event.clientX + document.body.scrollLeft;
                mouseY = event.clientX + document.body.scrollTop;


        document.show.mouseXField.value = mouseX;
        document.show.mouseYField.value = mouseY;

        return true;
        }

        </script> 

SECONDE VERSION (FONCTIONNEMENT)

      <form name="Show">
            <input type="text" name="MouseX" value="0" size="4"> X<br>
            <input type="text" name="MouseY" value="0" size="4"> Y<br>
      </form>


        <script type="text/javascript">

            var IE = document.all?true:false


            if (!IE) document.captureEvents(Event.MOUSEMOVE)


            document.onmousemove = getMouseXY;


            var tempX = 0
            var tempY = 0



            function getMouseXY(e) {
                if (IE) { 

                tempX = event.clientX + document.body.scrollLeft
                tempY = event.clientY + document.body.scrollTop
                } 
                else 
                {  
                 tempX = e.pageX
                 tempY = e.pageY
                }  


                document.Show.MouseX.value = tempX
                document.Show.MouseY.value = tempY

                return true
                }
        </script>     
9
user1010563

A première vue: clientX devrait être clientY et manquant {} dans votre else:

else {
   mouseX = event.clientX + document.body.scrollLeft;
   mouseY = event.clientY + document.body.scrollTop;
}

Aussi, vous devez envoyer l'événement à la fonction, 

document.onmousemove = function (e) {mousePos(e);};
...
function mousePos (e) {
...

vérifier ceci: http://jsfiddle.net/NLsdZ/1/

De plus, l'utilisation de captureEvents est obsolète, vous devriez plutôt utiliser addEventListener. Plus d'infos ici: http://forums.asp.net/t/1576872.aspx

18
NotGaeL

J'avais aussi remarqué que vous n'êtes pas en train de charger en dessous, ou du moins que vous le déclarez.

var mie = (navigator.appName == "Microsoft Internet Explorer") ? true : false;

    if (!mie) {

        document.captureEvents(Event.MOUSEMOVE);
        document.captureEvents(Event.MOUSEDOWN);
    }


    document.onmousemove = mousePos();
    document.onmousedown = mouseClicked();

Devrait être comme/ou finalement travaillé pour moi en tant que:

 window.onload = function () {
         mie = (navigator.appName == "Microsoft Internet Explorer") ? true : false;


    if (!mie) {
         document.captureEvents(Event.MOUSEMOVE);
         document.captureEvents(Event.MOUSEDOWN);
    }
         document.onmousemove = function (e) {mousePos(e);};
         document.onmousedown = function (e) {mouseClicked();};
 };
0
JSG