header.php problem

[misheck]

I am trying to create a header.php that echoes Good morning/afternoon/evening depending on the time and day. I need to add it to my login page and the page that the user is directed to when they login. This is what I have tried to enter on my header.php

<div align="center"><font size="4">Welcome to my review site</font>
<br>
<?php
echo "Today is ";
echo date("F d");
echo ", ";
echo date("Y");
?>
<br>
<font size="2">
<?php
if ((date("G") >= 5) AND (date("G") <= 11 )) echo "Good Morning!";
if ((date("G") >= 12) AND (date("G") <= 18 )) echo "Good Afternoon!";
if ((date("G") >= 19) AND (date("G") <= 4 )) echo "Good Evening!";
?>
</font>
</div>
But its not working. It only displays todays date.

[jib0]

Your last "if" is wrong, the integer number (hour) cannot be greater than 18 and smaller than 4. Try this:

<div align="center"><font size="4">Welcome to my review site</font>
<br>
<?php
  $now = time();
  echo 'Today is ', date('F d', $now), ', ', date('Y', $now);
?>
<br>
<font size="2">
<?php
  $hour = date('G', $now);
  if ($hour > 4  && $hour <= 11)
    echo 'Good Morning!';
  elseif ($hour > 11 && $hour <= 18)
    echo 'Good Afternoon!';
  else
    echo 'Good Evening!';
?>
</font>
</div>

The date function without the timestamp parameter returns the current value of time so it is better to get the current timestamp once at the beginning of the script and to use it in the date calls.
We can also use "elseif" instead of "if" because it is not necessary to evaluate the 3 "if" when one of them has already been evaluated to true.

[misheck]

Thanks for the reply its only that my book had provided me with the wrong answer and I did not know where to place the if statements.

Thanks for your help.