Discussion :

[will74]

Bonjour à tous et bonne année.
Pour bien commencer celle-ci, rien de mieux qu'un petit assouplissement des neurones... ^^

Je suis actuellement en train de réaliser un minuteur en javascript...
J'appelle donc une fonction en lui passant le temps voulu, le pas, et le nom de la div ou afficher le compteur.... (valeurs en secondes)
Imaginons que je veuille un temps de dix secondes, et un pas de un...

Code :

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

minutTime(10, 1, 'minut');

Le résultat à atteindre est celui-ci :

10
9
8
7
6
5
4
3
2
1
'chaine vide'

Comme vous pourrez le remarquer dans le code, lorsque le minuteur arrive au zéro il affiche un 'blanc' à la place du chiffre.

Voici donc mon code actuel :

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
	/* fonction de minutage */
		function minutTime(time, pas, divName) {
			var myDiv = document.getElementById(divName);
			var i = time, j = 0;
 
			function decompt() {
				if (i == 0)
					myDiv.innerHTML='';
				else
					myDiv.innerHTML=i;
				i--;
				setTimeout("decompt()", pas*1000);
			}
			if (i >= 0) 
				decompt();
		}

Comme vous pouvez vous en douter, je fais appel à vous car ile ne marche pas, et que je dois faire une bête erreur que je ne voie désespérément pas.
Merci d'avance à ceux qui pourront m'éclairer.

PS : Etant donné que mon code est plus simpliste et ne produit pas tout à fait le même résultat que les divers comptes à rebours résolus sur le forum, je me permet de créer tout de même un nouveau fil.

wILL

[SpaceFrog]

setInterval est plus approprié dans ce cas de figure... et un clearInterval en fin de minuterie ...

[will74]

Bonsoir SpaceFrog.

J'avais lu que setIntervall posais des problèmes de compatibilité en comparaison avec setTimeout.
Je vais essaye ce que tu me dit.... dans la même idée, toujours avec mes timeout, set et clear, j'ai fais cela en attendant une réponse :

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
		function minutTime(time, divName) {
			count = time;
			doTimer(divName);
 
			function doTimer(divName) {
				if (!timerStatut) {
				  timerStatut = 1;
				  timedCount();
				}
			}
 
			function timedCount(divName) {
				if (count == 0) {
					document.getElementById(divName).innerHTML = '';
					stopCount();
				}
				else {
					document.getElementById(divName).innerHTML = count;
					count--;
					t = setTimeout("timedCount(divName)", 1000);
				}
			}
 
			function stopCount() {
				clearTimeout(t);
				timerStatut = 0;
			}
		}

Seulette voilà, je n'ai pas trouvé d'autres moyen pour faire fonctionner ce code de déclarer mes variables count, t et timerStatut de façon globale.

De plus, mon code plante si je l'écrit comme ceci... ou disons plutôt que sa ne marche pas.

Je suis convaincu que le problème vient du passage du nom de la div, via la variable divName....
Pourtant je fais bien un appel correct :

Code :

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

minutTime(5, 'minut');

Merci d'avance

wILL

[will74]

Je reviens après ce que SpaceFrog m'a conseillé.
Voici donc une seconde version de mon premier 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
		function minutTime(time, pas, divName) {
			var myDiv = document.getElementById(divName);
			var i = time;
            var t;
 
			var setTimer = function() {
				if (i == 0) {
					myDiv.innerHTML='';
                    clearInterval(t);
				}
				else
					myDiv.innerHTML=i;
					i--;
					t = setInterval("setTimer()", pas*1000);
			}
			setTimer();
		}

A mon avis, le problème vient de la place ou je déclare mes fonctions...
Seulement je ne veut pas de variables globales, et le passage de paramètres ne semble pas marcher.

Merci d'avance pour vos conseils.

wILL