Discussion :

[romca]

Bonjour,
J'ai créé un menu déroulant avec une tempo d'affichage du menu. Le menu s'affiche sur un onMouseOver() qui lance une fonction afficheMenu() qui affiche le menu. Dans cette fonction je lance une tempo avec setTimeout() qui lance à son tour une autre fonction cacheMenu() qui cache le menu après 3 secondes, mais voilà j'ai une erreur sur la page lorsqu le code doit disparaitre (inscrit en bas du navigateur "erreur sur la page" à cause de cette fonction).
Quand je met un alert() comme fonction dans setTimeout() cela fonctionne bien ?
j'utilise IE7. Le page se trouve dans un frame. La fonction est appelée dans un script javascript ".js"
Voici le code si quelqu'un peu m'aider merci.

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
//////////////////////////////////////////////////////////////////////////:
// affichage menu déroulant
//////////////////////////////////////////////////////////////////////////:
 
function afficheMenu(id){
  //alert("affiche "+id);
  document.getElementById(id).style.display='block';
  //setTimeout("alert('test setTimeout')",3000);
  setTimeout("cacheMenu(id)",3000);
 
  }
 
function cacheMenu(id){
    //alert("cache "+id);
  	document.getElementById(id).style.display='none';
	exit();
}

[herfrayg]

Je suppose que cela est peut-être du au fait que ta variable id n'est pas globale (et c'est mieux comme cela). Essaye de mettre ceci pour le timeout et dis nous ce que cela donne :

Code javascript :

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

1
2
 
setTimeout("cacheMenu(\""+id+"\")",3000);

Cordialement.

[Bovino]

Code :

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

setTimeout(function(){cacheMenu(id)},3000);

[romca]

Bonjour à vous 2 et merci !
J'ai essayé les 2 solutions et cela fonctionne !, mais j'aimerais comprendre:

- Pour herfrayg STP, pourquoi le fait de mettre le id de cette façon: \""+id+"\" va permettre de récupérer la variable id globale ?

- Pour Bovino STP, pourquoi il faut déclarer de nouveau la fonction "cacheMenu()" en fonction dans le Timout, je croyais que le setTimeout() prenait déjà en 1er paramètre une fonction... ?

merci.

A+

[Bovino]

Les deux réponses sont liées.

je croyais que le setTimeout() prenait déjà en 1er paramètre une fonction... ?

Non, pas tout à fait.
Le premier paramètre est soit une chaîne de caractères qui sera ensuite évaluée (méthode eval()) et dans ce cas, l'argument que tu lui passes ne sera pas considéré comme la variable id mais comme la chaîne "id", d'où l'erreur et la proposition de herfrayg, soit une fonction, qui elle appellera ta fonction cacheMenu avec son paramètre (déclaré comme étant la variable id), c'est la solution que je t'ai proposée.

A priori, ma solution est préférable en termes de performances et de qualité de code dans la mesure où eval() et ses dérivés sont considérés comme étant à éviter.

[romca]

Salut,
L'explication me va très bien. J'ai saisis maintenant.

merci encore à vous 2.

cdt.