Discussion :

[CLeBeR]

Bonjour,
je voudrais écrire un texte selon l'heure en PHP. J'utilise le code suivant mais il ne fonctionne pas, le script ne rentre pas dans le else ...

Code :

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

1
2
3
4
5
6
7
8
<?php
date_default_timezone_set('Europe/Paris');
$heure = date("H");
if ($heure > 07 AND $heure < 12 OR $heure > 13 AND $heure < 18): ?>
	<b><font color="green">Nous sommes acutellement ouvert.</font></b>
<?php else: ?>
	<font color="red"><b>Nous sommes acutellement fermé.</b></font>
<?php endif; ?>

Merci de votre aide.

[CosmoKnacki]

À quelle heure as-tu essayé?

Je viens de le tester et j'obtiens bien le texte du else. La seule chose un peu étrange dans ton code c'est l'utilisation des opérateurs logiques and et or ce qui n'est pas une erreur, mais généralement on utilise plutôt && et || et on se tourne vers les deux autres pour des subtilités de précédence.

Autre chose, la fonction date renvoie une chaîne, mais dés lors que tu fais une comparaison avec un nombre, PHP va la trans-typer en nombre pour effectuer la comparaison. Il n'est donc pas utile d'écrire $heure > 07 en mimant le format de la chaîne, $heure > 7 fera la même chose tout aussi bien.

[CLeBeR]

Maintenant mais j'ai modifié la plage horaire pour la mettre à partir de 00H.

[Invité]

Bonjour,
pour éviter toute erreur d'interprétation dans l'ordre des instructions, le mieux est encore de mettre des parenthèses :

Code :

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

if ( ($heure > 7 and $heure < 12) or ($heure > 13 and $heure < 18) ):

Voir : Les opérateurs logiques

La raison pour laquelle il existe deux types de "ET" et de "OU" est qu'ils ont des priorités différentes.
Voir le paragraphe précédence d'opérateurs.

Cela dit, "and" et "or" sont sémantiquement plus clairs.

[CLeBeR]

Hello,
je viens de tester à nouveau et bizarrement, le code fonctionne. J'ai rajouté les week-ends et les parenthèses comme tu me l'a conseillé pour être plus rigoureux.

Code :

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

1
2
3
4
5
6
7
8
9
<?php
date_default_timezone_set('Europe/Paris');
$heure = date("H");
$jour = date("D");
if (($heure > 07 AND $heure < 12) OR ($heure > 13 AND $heure < 18) AND ($jour <> "Sat" OR $jour <> "Sun")): ?>
	<b><font color="green">Nous sommes acutellement ouvert.</font></b>
<?php else: ?>
	<font color="red"><b>Nous sommes acutellement fermé.</b></font>
<?php endif; ?>

Je reviens vers vous si je constate des problèmes. Merci pour votre aide,
@+