Visibilité variables globales

**exe2bin** · 29/01/2018, 18h12

Bonsoir à tous ,
je suis persuadé d'avoir compris la portabilité des variables en javascript , et pourtant ...
voici mon fichier html :

Code HTML :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
<!DOCTYPE html>
<html>
    <head></head>
<body>
    <img id="marteau" src="marteau.png"/>
    <script src="variables.js"></script>
</body>
</html>

Puis le fichier javascript afférent :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
var marteau = false;
var image = document.getElementsByTagName("img");
 
for (var i = 0; i < IMGElems.length; i++)
{
    IMGElems[i].onclick = handleClickEvent;
    IMGElems[i].isActive = true;
}
 
function handleClickEvent(e)
{
    if(!marteau)
    {
        alert("marteau = " + marteau);
        marteau = true;
    }
}

Comme vous l'aurez deviné l'alerte ne se produit jamais en réponse au click sur l'image ?!
Pourtant , si je ne m'abuse , "marteau" est pourtant une variable globale !!

**Beginner.** · 29/01/2018, 19h24

Salut,

Ah je vois que tu as adopté la solution de l'autre fil...

Pour ton code c'est normal : remplace var image = document.getElementsByTagName("img");...

par var IMGElems = document.getElementsByTagName("img");...

C'est probablement ce que tu voulais faire...

**Beginner.** · 29/01/2018, 19h30

Envoyé par Beginner.

Ah je vois que tu as adopté la solution de l'autre fil...

Sauf que tu voulais que le blocage ne se fasse que pour certaines images... Mais là ça bloque pour toute les images

Et justement l'instruction : IMGElems[i].isActive = true; que tu utilises ne sert à rien là...

On s'attendrait plutôt à un truc comme ça :

Code javascript :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function handleClickEvent(event) {
 
     var isThisImageActive = event.target.isActive;
 
     if (isThisImageActive) {
         alert("Oui elle est active ---> " + isThisImageActive);
         event.target.isActive = false;
     } else {
         alert("Non elle n'est pas active ---> " + isThisImageActive);
         event.target.isActive = true ;
     }
}

**exe2bin** · 29/01/2018, 21h26

Grand merci Beginner pour tes conseils mais mon problème est de comprendre pourquoi la variable "marteau" (censément globale ?) n'est pas visible dans
la fonction "handleClickEvent" ?!

**Beginner.** · 30/01/2018, 09h03

Elle est bel et bien visible, tu peux t'en rendre compte toi même en effectuant la modification que je t'ai indiquée.

--> Le problème n'est pas là : dans ton code la fonction handleClickEvent() n'est jamais lancée même lorsque tu cliques sur une image... Ce qui est normal puisque (contrairement à ce que tu a l'air de croire) cette fonction n'est pas associée à l’événement onClick des images..

Et oui cette instruction IMGElems[i].onclick = handleClickEvent; ne fait pas ce tu crois car "IMGElems" n'est pas défini. Et justement en faisant ce que je t'ai indiqué au message 2 il sera défini correctement....

Visibilité variables globales

JavaScript

Discussions similaires

Partager

Partager