Discussion :

[Auteur]

http://javascript.developpez.com/faq...e=date#Horloge

• Les variables du code ne sont pas déclarées (il manque le mot-clef var). Il y a également la présence d'une variable timerRunning dans la fonction. Je me suis demandé s'il fallait la supprimer car on peut se demander quel est son rôle.... Puis je l'ai déclarée globale et initialisée à false, je pense qu'ainsi on comprend mieux sa présence.
  
• Je me permets d'ajouter que l'autre syntaxe possible de setInterval() utilisé dans l'exemple est
  

  Code javascript :

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

  var timerID = setInterval("runClock()","1000");

  (avec des guillemets pour les arguments).
  
• Il y a également une petite faute d'orthographe dans le commentaire du code (il manque un "s" dans "le nombre de minute")
  Par ailleurs, est-ce que le terme timer prend un "s" au pluriel car c'est un mot anglais (j'ai toutefois laissé le "s" dans ma proposition de correction) ?
  Dans le commentaire du code il est écrit :

  on rajoute un 0

  , personnellement, je préfère le verbe "ajouter" à "rajouter" donc :

  on ajoute un 0

• Il faut également mettre à jour le code source correspondant :
  http://javascript.developpez.com/tel...-en-temps-reel

Voici ma proposition de correction :

Les timers JavaScript permettent d'exécuter des fonctions à des intervalles de temps donnés. Ainsi, pour notre horloge, nous allons utiliser setInterval qui se déclenchera toutes les 1000ms pour mettre à jour l'heure affichée sur l'horloge.

Voici la fonction pour afficher l'heure :

Code javascript :

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
 
var timerRunning = false;
function runClock() 
{
  var today   = new Date();
  var hours   = today.getHours();
  var minutes = today.getMinutes();
  var seconds = today.getSeconds();
  var timeValue = hours;
 
  // Les deux prochaines conditions ne servent que pour l'affichage.
  // Si le nombre de minutes est inférieur à 10, alors on ajoute un 0 devant...
 
  timeValue += ((minutes < 10) ? ":0" : ":") + minutes;
  timeValue += ((seconds < 10) ? ":0" : ":") + seconds;
  document.getElementById("time").value = timeValue;
  timerRunning = true;
}
 
var timerID = setInterval(runClock,1000);

Et voici le champ texte :

Code html :

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

<input type="text" id="time" size="10" />

Remarque :
Une autre syntaxe possible de la fonction setInterval() utilisée dans l'exemple est

Code javascript :

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

var timerID = setInterval("runClock()","1000");

(avec des guillemets pour les arguments).

[Bovino]

[list][*]Je me permets d'ajouter que l'autre syntaxe possible de setInterval() utilisé dans l'exemple est

Code javascript :

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

var timerID = setInterval("runClock()","1000");

(avec des guillemets pour les arguments).

Je ne suis pas sûr que ce soit très opportun : utiliser une chaine de caractère en premier argument est fortement déconseillé et à éviter au maximum...

[Auteur]

Je ne suis pas sûr que ce soit très opportun : utiliser une chaine de caractère en premier argument est fortement déconseillé et à éviter au maximum...

Ok dans ce cas, on retire la remarque que j'ai ajoutée.

[vermine]

Je vais attendre que toute la section sur les dates soit relue (c'est peut-être déjà le cas) afin de faire les modifications en une fois.

[Auteur]

Je vais attendre que toute la section sur les dates soit relue (c'est peut-être déjà le cas) afin de faire les modifications en une fois.

Pas encore. Je te préviendrai par MP quand j'aurai fini

[lanonyme]

Beau boulot Auteur

Juste une remarque dans la rédaction des Q/R, je pense qu'il faudrait garder une certaine homogénéité "dans l'aspect" quand nous écrivons en dehors du code le nom des fonctions/méthodes que nous utilisons.

Pour être plus clair :

getTime() => gras
et
SetInterval => identique au reste du texte

J'écris peut être pas au bon endroit, mais c'est relisant cette Q/R que j'ai remarqué ce petit détail