Problem when comparing hours

Question

When comparing timetables I have in the registered database the opening and closing hours of an establishment. I’m having a problem comparing the hours to see if the establishment is open or closed, it’s always appearing to me as open when it’s past time.

<?php
$horario = strtotime(date('H:i'));
$result_horario_estado=mysql_query("select * from horarios where id_mae='".$row->id."'");
$row_horario_estado=mysql_fetch_array($result_horario_estado);

$abertura   = strtotime($row_horario_estado['horario_abertura']);
$fechamento = strtotime($row_horario_estado['horario_fechamento']);

if($horario >= $abertura || $horario <= $fechamento) {
?>
    <div style="margin:-3px 0px 5px 0px; width:257px; font-family:Arial, Helvetica, sans-serif; color:#0C3; font-size:13px;">Aberto</div>
<?php
} else {
?>
    <div style="margin:-3px 0px 5px 0px; width:257px; font-family:Arial, Helvetica, sans-serif; color:#F00; font-size:13px;">Fechado Agora</div>
<?php
}
?>

Answer 1

Dude, analyze your code. You open at 14 and close at 2. You have to consider this. 14 is greater than 2.

<?php
$horario = strtotime('1:00');
$abertura = strtotime('22:00');
$fechamento = strtotime('7:00');

if ((
        $abertura < $fechamento && $horario >= $abertura && $horario <= $fechamento
        ) || (
        $abertura > $fechamento && ($horario >= $abertura || $horario <= $fechamento)
        )) {
    echo 'aberto';
} else {
    echo 'fechado';
}

Answer 2

You need both conditions to be met:

if($horario >= $abertura && $horario <= $fechamento){
            //  VEJA --- ^^

The && in place of || ensures this. In your code, one of the conditions is enough to enter the if, causing more events to display than you would like.