Mon code javascript fait planter IE

**ovcrash** · 22/05/2009, 16h01

Bonjour,

Voici mon code :

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
<html>
<head>
<SCRIPT LANGUAGE="JavaScript" TYPE="text/javascript">
<!--
// Script de Fenetre vers l'exterieur
gSlideshowInterval = 30; // En seconde
gNumberOfImages = 4; // Nombre d'image dans le array
 
gImages = new Array(gNumberOfImages);
gImages[0] = "http://www.pixelweb.net/image.jpg";
gImages[1] = "http://216.137.100.129:81/IMAGE.JPG";
gImages[2] = "http://webcam.northco.net/montreal/montreal1.jpg";
gImages[3] = "http://webcam.northco.net/montreal/montreal2.jpg";
 
// Valide si le browser prend en charge les fonctions
function canManipulateImages() {
	if (document.images)
		return true;
	else
		return false;
}
 
// Load les images de l'array
function loadSlide(imageURL) {
	if (gImageCapableBrowser) {
		tmp = new Date();
		tmp = "?" + tmp.getTime();
		document.slide.src = imageURL + tmp;
		return false;
	}
	else {
		return true;
	}
}
 
// Change l'image dans le slide
function nextSlide() {
	gCurrentImage = (gCurrentImage + 1) % gNumberOfImages;
	loadSlide(gImages[gCurrentImage]);
	setInterval("loadSlide(gImages[gCurrentImage])", 5000); // Fait l'update de l'image en cours au 5 secondes
}
 
gImageCapableBrowser = canManipulateImages();
gCurrentImage = 0;
setInterval("nextSlide()",gSlideshowInterval * 1000); // Change l'image du slide au temp indiquer dans gSlideshowInterval
// -->
</SCRIPT>
</head>
 
<body>
 
<IMG SRC="http://webcam.northco.net/montreal/montreal1.jpg" NAME="slide" width="373" height="324">
 
</body>
</html>

Apr�s un certain temp, IE ferme tout seul. Je ne comprend pas pourquoi, pourtant le code fonctionne bien.

J'ai essayer de changer mes setInterval par setTimeout mais ca ne fonctionne pas dans ce que je veux faire, vue que le setTimeout c'est juste une fois que ca se produit et non a r�pition comme moi je le veux.

De l'aide serait appr�cier! Merci!

**Bovino** · 22/05/2009, 17h23

Oui, tu as un setInterval qui a chaque seconde, lance une fonction qui g�n�re un nouveau setInterval... Je te laisse calculer le nombre total de setInterval au bout de quelques minutes... et de ce fait, la charge m�moire associ�e !

**ovcrash** · 22/05/2009, 19h33

Merci de mettre en evidence ce que je savais d�j�.

Mais je voulais plus savoir une solution au mon probl�me.
Parceque j'ai essayer de changer mes setInterval par des setTimeout et ca ne fonctionne pas, dison que ca ne fait plus ce que je veux faire.

Est-ce qu'il aurait une solution?

Une r�ponse clair et pr�cise serait des plus appr�cier.
Merci de votre temps et aide.

**Bovino** · 22/05/2009, 19h42

Envoy� par ovcrash

Merci de mettre en evidence ce que je savais d�j�.

D�sol�, c'est pas ce que semblait indiquer :

Apr�s un certain temp, IE ferme tout seul. Je ne comprend pas pourquoi, pourtant le code fonctionne bien.

Mais bon...

Est-ce qu'il aurait une solution?

Une r�ponse clair et pr�cise serait des plus appr�cier.
Merci de votre temps et aide.

Il suffit � priori de mettre un setTimeout dans la fonction nextSlide() :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
function nextSlide() {
	gCurrentImage = (gCurrentImage + 1) % gNumberOfImages;
	loadSlide(gImages[gCurrentImage]);
	setTimeout("loadSlide(gImages[gCurrentImage])", 5000); // Fait l'update de l'image en cours au 5 secondes
}
 
gImageCapableBrowser = canManipulateImages();
gCurrentImage = 0;
setInterval("nextSlide()",gSlideshowInterval * 1000);

**ovcrash** · 23/05/2009, 03h28

Merci de la r�ponse.

Mais en mettant un setTimeout l'image ne se rafraichit plus au 5 secondes, mais seulement au 2 minutes.

Dans le fond, faut que j'aille deux timer qui fonctionne simultanemant.

**Bovino** · 23/05/2009, 08h56

OK, j'avais survol� ton code trop rapidement et j'avais cru comprendre que le setInterval que j'ai modifi� �tait un pr�chargement d'images...
Donc, si j'ai d�soramis bien compris, tu as plusieurs sources d'images type webcam.
Tu souhaites changer de source sur un setInterval et mettre � jour la vue dans un autre ?
Si c'est le cas, je te conseille de lancer les deux ensemble comme ceci :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
// Tu n'as plus besoin de cette fonction
/*function nextSlide() {
	gCurrentImage = (gCurrentImage + 1) % gNumberOfImages;
	loadSlide(gImages[gCurrentImage]);
	setInterval("loadSlide(gImages[gCurrentImage])", 5000); // Fait l'update de l'image en cours au 5 secondes
}*/
 
gImageCapableBrowser = canManipulateImages();
gCurrentImage = 0;
setInterval(function(){
    gCurrentImage = (gCurrentImage + 1) % gNumberOfImages;
},gSlideshowInterval * 1000);
setInterval(function(){loadSlide(gImages[gCurrentImage])}, 5000);

Mon code javascript fait planter IE

JavaScript

Vue hybride

Discussions similaires

Partager

Partager