conseils pour fonction récurcive

**dominos** · 13/07/2010, 10h09

Bonjour,

Le script ci-dessous tente de faire suivre plusieurs fichiers swf en gérant la durée de chacun, les boutons permettent donc de stoper celle en cours et de lancer celle désignée à l'aide du paramètre select. Le soucis c'est que la valeur null semble non réinitialisé et l'animation passe de l'une à l'autre sans respecter le time_swf désigné par i ...

swf_viewer.js

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
var url_files = 'http://www.domaine.com/swf/';
 
var swf_tab = new Array();
swf_tab[1] = "fichier1.swf";
swf_tab[2] = "fichier2.swf";
 
var time_swf = new Array();
time_swf[1] = "30000";
time_swf[2] = "15000";
 
var desc_swf = new Array();
desc_swf[1] = "decritption fichier 1";
desc_swf[2] = "description fichier 2";
 
var i = 0;
 
function change_swf(select) {
 
	document.getElementById('bt_swf1').src = 'images/bt01.png';
	document.getElementById('bt_swf2').src = 'images/bt02.png';
 
	if (select == null) {
		i++;
		if (i >= swf_tab.length)
			i = 0;
		document.getElementById('bt_swf' + i).src = 'images/bt0' + i + '.png';
	} else {
		i = select;
		document.getElementById('bt_swf' + i).src = 'images/bt0' + i + '_on.png';
	}
 
	document.getElementById('swf_desc').innerHTML = desc_swf[i];
 
	var so = new SWFObject(url_files + swf_tab[i], "mymovie", "500", "250", "8");
	so.addParam("wmode", "transparent");
	so.write("swf_content");
 
//	if (select != null) // réinitialise
//		i = 0;
 
	setTimeout("change_swf(" + null + ")", time_swf[i]); // null => passe à la suivante
 
}

test.html

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
<html>
<head>
 <script type="text/javascript" src="library/swfobject.js"></script>
 <script type="text/javascript" src="swf_viewer.js"></script>
 <link rel="stylesheet" type="text/css" href="css/swf_viewer.css" />
</head>
 
<body onload="change_swf(null);">
 
<div id="swf_content"></div>
<div id="swf_desc"></div>
 <div id="swf_buttons">
  <a href="javascript:change_swf(1);"><img id="bt_swf1" src="" border="0"></a>
  <a href="javascript:change_swf(2);"><img id="bt_swf2" src="" border="0"></a>
 </div>
 
</body>
</html>

swf_viewer.css

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
#swf_content {
	width: 500px;
	height: 250px;
}
 
#swf_desc {
	float: left;
	padding-right: 10px;
	color: white;
}
 
#swf_buttons {
	float: left;
	padding-top: 2px;
}

Merci pour vos éclaircissements.

**tusssss** · 15/07/2010, 11h17

ton problème vient possiblement, de la comparaison entre une chaine de caractère "null" et la valeur null...

Le mieux serait de regarder les types de variables que tu compares.

Ou alors, tu te simplifies la vie en renommant ta fonction :

Code :

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

1
2
 
function change_swf(select=null)

Comme cela quand tu veux appeler ta fonction avec la valeur null pour le select, tu auras juste à appeler ta fonction de la sorte :

Code :

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

1
2
 
change_swf();

Puisque ton paramètre prendra par défaut la valeur null si tu ne lui en communiques pas une autre.

P.S : Pas certain que cette valeur par défaut fonctionne en javascript, ça existe en PHP donc ca doit bien exister en javascript.

bon courage.

**sedryk** · 15/07/2010, 11h26

Hello Dominos,

Je n'ai pas trop compris ton problème de "valeur null qui ne se réinitialise pas".

Cependant tu peux déjà simplifier ton appel à setTimeout en retirant la concaténation

Code :

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

setTimeout("change_swf(null)", time_swf[i])

**sedryk** · 15/07/2010, 11h29

Envoyé par tusssss

P.S : Pas certain que cette valeur par défaut fonctionne en javascript, ça existe en PHP donc ca doit bien exister en javascript.

En faite, le javascript connait null et undefined. Null doit être explicitement affecté alors que undefined est la valeur d'une variable non affectée.

**tusssss** · 15/07/2010, 11h55

Envoyé par sedryk

En faite, le javascript connait null et undefined. Null doit être explicitement affecté alors que undefined est la valeur d'une variable non affectée.

Merci de la précision, j'avais oublié ce petit détail.

Cependant mon questionnement était tout autre :

Quand on écrit une fonction de ce type :

Code :

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

1
2
3
4
 
function test(attribut="zero") {
     alert(attribut);
}

Si on appelle la fonction dans notre page, de la manière suivante :

Code :

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

1
2
 
test();

La fonction va t-elle fonctionner en faisant apparaitre un alert("zero") ? Ou ne va t-elle pas fonctionner du tout?

Edit : J'ai ma réponse, ça ne fonctionne pas du tout...

**gwyohm** · 15/07/2010, 14h39

Code :

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

1
2
3
4
 
function test(attribut="zero") {
     alert(attribut);
}

Il y a quand même des façons de définir des valeurs par defaut :

Code :

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

1
2
3
4
5
 
function test(attribut) {
  // si attribut est undefined, il sera aussi "faux", tu peux donc faire
  attribut = attribut || "zero"
}

Maintenant, un peu plus propre et pour gérer le cas ou la valeur par defaut doit être false:

Code :

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

1
2
3
function test(attribut) {
  attribut = (typeof(attribut) == "undefined") ? false :attribut;
}

après il peut y avoir à gérer le cas du null...

Code :

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

1
2
3
function test(attribut) {
  attribut = (typeof(attribut) == "undefined" || attribut == null) ? false :attribut;
}

bref tout dépend de tes besoins

**sedryk** · 15/07/2010, 17h24

Au passage,

Code :

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

typeof(attribut) == "undefined"

peut être écris plus simplement

Code :

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

attribut === undefined

Ceci dit tant que dominos ne nous répond pas ce n'est que pure rhétorique de ma part

**dominos** · 16/07/2010, 09h38

Envoyé par sedryk

Hello Dominos,

Je n'ai pas trop compris ton problème de "valeur null qui ne se réinitialise pas".

Cependant tu peux déjà simplifier ton appel à setTimeout en retirant la concaténation

Code :

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

setTimeout("change_swf(null)", time_swf[i])

Merci, sinon j'ai ajouté

Code :

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

1
2
	clearTimeout(t);
	t=setTimeout("change_swf(null)", time_swf[i]);

mais je ne sais pas si ça change quelque chose..

conseils pour fonction récurcive

JavaScript

Discussions similaires

Partager

Partager