Plantage de Mozilla firefox a cause d'un code Javascript

Version imprimable

Bonjour

J'utilise ce petit script Javascript pour faire défiler de la pub sur mon site (http://www.woneb.net):
Code:

1 2 3 4 5 6 7 8 9 10 11 <script language="JavaScript"> var pub=new Array(); pub[0]=... pub[1]=... function defiler_pub() { numpub = Math.round(Math.random()*(pub.length-1)); document.getElementById("script_pub").innerHTML = pub[numpub]; setInterval("defiler_pub()",20000); } </script>
Le tableaux pub [] contient mes differentes pub à afficher.

Mon body : <body onload="defiler_pub()">

L'affichage de ma pub : <div id="script_pub">....</div>

Mon probleme est que Mozilla Firefox (sous Linux) plante au bout d'un moment. Cela vient de ce script c'est sur a 100%.
Autre détail, avec le navigateur Opera cela ne plante pas.

Je ne suis pas trés doué en Javascript, est ce que vous pouvez m'aider a trouver ce qui géne Firefox et de trouver un code arlternatif qui ne plante pas.

Merci a vous
Thomas

JE n'ai pas de souci de plantage avec le script suivant

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<script type='text/javascript'>
var pub=new Array();
pub[0]="un"
pub[1]="deux"
pub[2]="trois"
pub[3]="quatre"
pub[4]="cinq"
pub[5]="six"
pub[6]="sept"
pub[7]="huit"
function defiler_pub()
{
	numpub = Math.round(Math.random()*(pub.length-1));
	document.getElementById("script_pub").innerHTML = pub[numpub];
	setInterval(function(){defiler_pub()},1000);
}
</script>
</head>
 
<body onload="defiler_pub()">
<div id="script_pub"></div>
</body>
 
</html>

16/10/2008, 11h18
Tgarfr

Non j'ai essayé sa fait toujour pareil.

Pour que sa plante il faut laissé la page pendant 2-3 minute. Il devient tous noir pour dire qu'il repond plus, je ne peu plus cliqué nul part et je suis obligé de fermer le processus.
16/10/2008, 11h59
SpaceFrog

rien à voir avec javascript ...
change de PC ...
17/10/2008, 02h34
Tgarfr

Lorsque je désactive javascript dans mozilla, cela ne plante pas.

Quand a mon PC core2duo 2.2 Ghz avec 2 Go de ram (dons seulement 200mo utilisé). Ya rien a craindre sa vient pas de là. Surtout que comme je l'ai dit plus haut avec Opera cela ne plante pas.

Sa vient de java script est uniquement dans mozilla, c'est sûr a 100%.
17/10/2008, 09h35
SpaceFrog

ben vu que le code que je t'ai donné plus haut fonctionne chez moi, je pencherait plutot pour un triangle des bermude insatllé autour de ton PC ...

as tu modifié ta balise script ?

La solution nous vient de BOVINO !!!

sors le setInterval de la focntion sinon ça entraine une satruation de la pile !

le setTimeout peut se lancer en fin de fonction de façon recursive, pas le setTimeout.

Ce qui devient :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 <script type='text/javascript'> var pub=new Array(); pub[0]="un" pub[1]="deux" pub[2]="trois" pub[3]="quatre" pub[4]="cinq" pub[5]="six" pub[6]="sept" pub[7]="huit" function defiler_pub() { numpub = Math.round(Math.random()*(pub.length-1)); document.getElementById("script_pub").innerHTML = pub[numpub]; } fcuntion foo(){ setInterval(function(){defiler_pub()},1000);} </script> </head> <body onload="foo()"> <div id="script_pub"></div> </body>
et du coup tu ne relancera pas un instance de setInterval à chaque instance de focntion defilpub ...

18/10/2008, 22h53
Tgarfr

Exelent Ca fonctionne.Merci beaucoup SpaceFrog et BOVINO.

ET c'était pas Le triangle des bermudes.:lol:

Pour ceux qui vienne pour copier le script, dans le code de Space frog y faut rempalcer "fcuntion foo()" par "function foo()". Petite erreur de frappe.

Encor Merci
19/10/2008, 01h03
Tgarfr

Encor un petit rajout pour ceux qui veulent le récupéré :
Avec le code modifié, il n'y a pas de pub pendant le premier labs de temps. Donc il faut rajouter defiler_pub() au debut de la fonction foo, comme ceci :

Citation:

function foo(){
defiler_pub();
setInterval(function(){defiler_pub()},1000);
}