Discussion :

[Arnold59]

Bonjour,

Je suis en train de faire une refonte d'un site.

J'ai vu également sur le site la différence entre setFullYear et setUTCFullYear
Source http://javascript.developpez.com/faq...setUTCFullYear

Voici 2 scripts que j'avais il y a quelques temps.
Version 1 :

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
56
57
58
<script type = "text/javascript" >
        var d = new Date()
var j = d.getDate()
var m = d.getMonth() + 1
var y = d.getYear()
if (y < 999)
    y += 1900;
var h = d.getHours()
var mn = d.getMinutes()
var mois, jour;
var dayNames = new
        Array("dimanche", "lundi", "mardi", "mercredi", "jeudi", "vendredi", "samedi");
if (m == 1)
    mois = " janvier";
else if (m == 2)
    mois = " février";
else if (m == 3)
    mois = " mars";
else if (m == 4)
    mois = " avril";
else if (m == 5)
    mois = " mai";
else if (m == 6)
    mois = " juin";
else if (m == 7)
    mois = " juillet";
else if (m == 8)
    mois = " août";
else if (m == 9)
    mois = " septembre";
else if (m == 10)
    mois = " octobre";
else if (m == 11)
    mois = " novembre";
else if (m == 12)
    mois = " décembre";
if (j == 1)
    jour = "1er"
else
    jour = j;
// Test pour déterminer la formule à employer
if (h >= 18)
    document.write("Bonsoir");
else
    document.write("Bonjour");
document.write(" et bienvenue sur mon site. ")
document.write("Aujourd'hui nous sommes le " + dayNames[d.getDay()] + " " + jour + " " + mois + " " + y + ", ")
document.write("il est " + h)
if (h < 2)
    document.write(" heure ")
else
    document.write(" heures ")
document.write(mn)
if (mn < 2)
        document.write(" minute.")
        else
        document.write(" minutes.")
        </script>

Version n°2 :

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
 
<script type="text/javascript">
                var d = new Date()
                var j = d.getDate()
                var m = d.getMonth()
                var y = d.getFullYear()
                var h = d.getHours()
                var mn = d.getMinutes()
                var jour;
                var dayNames = new
                        Array("dimanche", "lundi", "mardi", "mercredi", "jeudi", "vendredi", "samedi");
                var mois = ["janvier", "février", "mars", "avril", "mai", "juin", "juillet", "aout", "septembre", "octobre", "novembre", "décembre"]
                if (j == 1)
                    jour = "1er"
                else
                    jour = j;
                // Test pour déterminer la formule à employer
                if (h >= 18)
                    document.write("Bonsoir");
                else
                    document.write("Bonjour");
                document.write(" et bienvenue sur mon site. ")
                document.write("Aujourd'hui nous sommes le " + dayNames[d.getDay()] + " " + jour + " " + mois[m] + " " + y + ", ")
                document.write("il est " + h)
                if (h < 2)
                    document.write(" heure ")
                else
                    document.write(" heures ")
                document.write(mn)
                if (mn < 2)
                    document.write(" minute.")
                else
                    document.write(" minutes.")
            </script>

Je souhaiterais savoir si le second script est optimisé, que faut-il ajouter ou modifier ?

D'avance merci.

[NoSmoking]

Bonjour,
pour commencer on va dire qu'il serait préférable que tu oublies le document.write : Comprendre document.write() en JavaScript.

Concernant l'initialisation du nom des jours autant utiliser la même méthode que celle que tu as utilisé pour les mois, il est préférable de rester homogène

Code :

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

1
2
var jours = ["dimanche", "lundi", "mardi", "mercredi", "jeudi", "vendredi", "samedi"];
var mois  = ["janvier", "février", "mars", "avril", "mai", "juin", "juillet", "aout", "septembre", "octobre", "novembre", "décembre"];

Il est aussi intéressant de construire la chaine à afficher et de l'afficher en une seule fois

Code :

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

1
2
3
4
5
6
7
8
9
10
var chaine = ''
// Test pour déterminer la formule à employer
if (h >= 18){
    chaine += 'Bonsoir';
}
else {
    chaine += 'Bonjour';
}
chaine += ' et bienvenue sur mon site. ';
chaine += ' Aujourd’hui nous sommes le ';

d'autres méthodes existent mais le plus simple serait, pour moi, d'oublier tout cela et de ne faire qu'un simple

Code :

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

document.getElementById('span_date').textContent = Date().toLocaleDateString(); // affichage de 12/05/2016

• Regarde également du coté de l'opérateur conditionnel ternaire

• Un autre rendez vous pourrait être les exercices sur DVP