Discussion :

[bobo95]

Bonjour à tous,

Je me suis procuré un script JAVA qui permet d'afficher un message précis en fonction de l'heure.

Je souhaiterais ajouter un élément, les jours.

Le 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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
<SCRIPT LANGUAGE="JavaScript">
MaDate = new Date;
heure = MaDate.getHours();
function MessageParDate()
	{
	if (heure>=0 && heure<=7)
		{
		MessageB = '01';
		}
	if (heure>7 && heure<8)
		{
		MessageB = '02';
		}
	if (heure>=8 && heure<17)
		{
		MessageB = '03';
		}
	if (heure>=17 && heure<=18)
		{
		MessageB = '04';
		}
	if (heure>18 && heure<=19)
		{
		MessageB = '05';
		}
	if (heure>19 && heure<=23)
		{
		MessageB = '06';
		}
	return MessageB;
	}
</SCRIPT>

Ce que je souhaite par exemple c'est que du lundi au vendredi, le script fonctionne normalement en fonction des heures mais que le samedi dimanche le script affiche seulement un message fixe et reprend son rythme normal dès le lundi.

Avez vous une idée s'il vous plaît ?

Merci

[SpaceFrog]

Un minimum de recherche sur le forum s'impose...
Récupérer le jour de la semaine est une chose relativement simple.
Sinon faut aller demander à la source

[Bovino]

Je me suis procuré un script JAVA qui permet d'afficher un message précis en fonction de l'heure.

Java et JavaScript n'ont rien à voir...

Sinon, tu ferais bien d'éviter de récupérer des scripts qui sont manifestement aussi datés !

Tout d'abord, la syntaxe

Code html :

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

<SCRIPT LANGUAGE="JavaScript">

est vraiment dépassée. Je ne vais pas troller sur l'écriture des balises en majuscule, mais ce n'est vraiment plus d'actualité.
En revanche, l'attribut "language" est déprécié depuis HTML4 et à l'inverse, l'attribut "type" est désormais requis.
Et il semble que celui qui a écrit ce script ne soit pas encore au courant de la différence entre ces deux attributs, parce que LANGUAGE="JavaScript" n'a absolument aucun sens ni intérêt !
LANGUAGE sert à déterminer la version de JavaScript utilisée, donc mettre JavaScript comme valeur, ben, ça sert pas à grand chose

Sinon, au niveau des variables, je vois mal l'intérêt de les déclarer hors de la fonction (ce qui encombre inutilement l'objet global). Il serait préférable d'en déclarer une seule dans la fonction avec le mot clé var, histoire de respecter un peu la syntaxe JavaScript :

Code :

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

1
2
3
function MessageParDate(){
    var heure = new Date().getHours();
}

Enfin, la succession de conditions if est maladroite, ici, un switch serait plus approprié

Et puis des conditions du style

Code :

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

if (heure>7 && heure<8)

on peut s'en passer ! Je vois mal comment un chiffre pourrait être à la fois strictement supérieur à 7 et strictement inférieur à 8...

Donc un exemple de code correct pourrait être

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
function MessageParDate(){
    var heure = new Date().getHours();
    switch(true){
        case heure < 8:
            return '01';
        case heure < 17:
            return '03';
        case heure < 19:
            return '04';
        case heure < 20:
            return '05';
        default:
            return '06';
    }
}

(tu noteras aussi que le cas où l'heure soit entre 23 et 0 est désormais géré )