SETTIMEOUT(); en javascript

**Sawyer4** · 24/07/2007, 19h02

Slt � tous,

J'ai un petit probl�me avec une fonction "compte a rebour" :

Entre HEAD et \HEAD j'ai mit :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
<SCRIPT LANGUAGE="JavaScript">
	 function compte_a_rebour(temps){
	 var i=0;
	 for (i = 0, i<temps, i++)
	 {
	  window.setTimeout("document.getElementById('temps').innerHTML='"+i+"';",i+"000")
	 }
	 }
     </script>

Et dans ma page il y a :

Code :

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

1
2
3
<body onLoad="compte_a_rebour('30');">
 
<div id="temps"></div>

Je comprends pas : le temps ne s'affiche pas dynamiquement, j'ai peut �tre fais des erreurs de syntaxe ( le javascript je m y fais pas ).

Merci de me r�pondre au plus vite ^^

**sharrascript** · 24/07/2007, 20h59

bonsoir,

Bon voici un petit code tap� � la vol�, je sais pas si c'est ce que tu souhaitais faire:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<SCRIPT language=javascript>
var i= 30;
function compte_a_rebour(){
	i--;
	if (i <= 0){
		clearsetInterval(tempo);
	}
	document.getElementById("temps").innerHTML= i+";";
}
tempo= setInterval('compte_a_rebour()',2000);
</SCRIPT>

Bizarrement, si tu passe en dessus des 2000 c'est pas terrible.

Bon, c'est juste un petit compte � rebour, je sais pas si c'est ce que tu souhaitais.

A savoir que .setTimeout retardera l'effet d'une fonction � tant de millisecondes, setInterval agira toutes les X millisecondes

Voil� ++

**SpaceFrog** · 24/07/2007, 21h02

Code :

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

i+"000"

Code :

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

i*1000

**Sawyer4** · 26/07/2007, 20h46

Merci bcp ^^

**Shinuza** · 27/07/2007, 07h09

On ne passe PAS de string � setTimeout, on lui passe une r�f�rence � une fonction ou au pire une fonction anonyme

@Cruzty : Ton code qui est cens� corrig� des erreurs en am�ne pas mal d'autres :

-Tu ne d�clare pas i, donc tu cr�es une variable globale.
-Tu passe une string � setTimeout est c'est mauvais
-Tu utilises deux fois le mot temps pour deux �lements totalement diff�rents
-Tu appelles x fois la fonction document.getElementById pour faire une action sur le m�me element (

)
-Tu appelles window.setTimeout ce qui est totalement inutile puisque la m�thode setTimeout fais partie du namespace global.
-Accessoirement (-1*i)+temps et i*-1+temps font exactement la m�me chose, r�gle �lementaire d'arith�m�tique. D'ailleurs cette deuxi�me �criture est autrement plus claire.

ET NOM D'UNE PIPE, on n'utilise PAS l'attribut language pour un script javascript;

Code :

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

<script type="text/javascript"></script>

**Cruzty** · 27/07/2007, 20h31

J'ai simplement corrig� son code. Si j'avais a faire un compte a rebours, il est certain que je ne ferai absolument pas comme ca, en balancant x fois la fonction setTimeout avec des d�lais diff�rents. J'utiliserai setInterval pour d�cr�menter une variable en secondes, puis d�composer le tout en heures, minutes, secondes, etc...

Je vais quand meme r�pondre pour etre plus clair :

- i est la variable que j'utilise tout le temps dans mes boucles, et rien d'autre, je ne vois donc pas l'utilit� de la d�lcarer.
- je reprend son code ( setTimeout(code,millisec[,lang]), pour moi, c'est bien une fonction qui est rentr�e, sauf qu'elle fait qu'une ligne)
- je reprend son code
- une fois de plus, je reprend son code. je ne ferai pas comme ca de toute facon, si on lance un compte a rebours de 10heures, j'imagine pas le carnage au chargement.
- je reprend toujours son code, et ca ne change rien au fonctionnement
- ce sont des habitudes de programmation, pour moi c'est plus clair avec les parentheses, pour toi non, chacun son style.
- et j'ai bien utilis� l'attribut type.

Il aurait �t� plus utile de ta part d'utiliser mes erreurs pour faire quelquechose de global, ainsi ajouter une solution pratique � ce probleme de compte a rebours. Et je trouve inutile de remanier le code fourni par quelqu'un, pour simplement montrer que l'on sait mieux faire. C'est pour cela aussi que j'ai utilis� la base de son code, sans vouloir faire � ma facon.

Cordialement,
Florian

**Cruzty** · 24/07/2007, 21h02

Bonjour !

Comme tu as pu le deviner, il y a deja pas mal d'erreurs de syntaxe, effectivement. Pour ne pas te 'lacher' le code correct inutilement, je vais plutot �num�rer les quelques petites erreurs d'�tourderies que tu as faites, et qu'on a tous faites.. au passage

Allons-y...

Code :

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

onLoad="compte_a_rebour('30');"

Ici tu appelles ta fonction compte_a_rebour (rebours

) et tu passes en param�tre une valeur chaine, c'est a dire, du texte... ok jusque l�...

Code :

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

1
2
function compte_a_rebour(temps){
	 for (i = 0, i<temps, i++)

Mais ici, gros probl�me... tu essaies d'utiliser une valeur chaine en tant que nombre... et ca, pas bien

Il faut, soit passer en param�tre un nombre (ce qui est le plus simple), soit, utiliser Number(temps) a CHAQUE fois

Code :

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

i+"000")

De m�me ici, tu m�langes texte et nombres... seul un nombre est accept� dans ce param�tre, qui est le d�lai... multiplie tout simplement i par 1000

- Derni�re remarque, si tu souhaites que ce soit un compte a rebours, (30 � 0) il faut inverser les conditions dans ta boucle for {}...

J'esp�re que je t'aurai aid� a cerner les petites erreurs que tu as faites, et voici le code fonctionnel :

Code :

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

1
2
3
4
5
6
7
8
<script type="text/javascript">
function compte_a_rebour(temps){
	for (i = temps; i>=0; i--)
	{
		window.setTimeout("document.getElementById('temps').innerHTML='"+i+"';",((-1*i)+temps)*1000);
	}
}
</script>