Как правильно сравнить время в php и вывести true или false?

Question

[Денис Московский]

Есть время открытия магазина: 8:00 и время закрытия: 23:00, а также реальное время в данный момент.
Нужно выполнить php скрипт, который будет возвращать true или false через return(это важно).
true - если магазин открыт.
false - если магазин закрыт.

Пробую так:

<?php
    $start_work = "08:00";
    $end_work = "23:00";
    $currentTime = date("H:i:s");
    
    //текущее время с датой, даже если оно 12:00 все равно имеет дату!
    $currentDateTime = strtotime(date('Y-m-d')  ." ". $currentTime); 
    //Дата и время во сколько мы закрылись вчера
    $previousDayEnd;
    //Дата и время во сколько мы открылись сегодня
    $startDateTime;
    //Дата и время во сколько мы закрылись сегодня
    $endDateTime;
    
    $startDateTime = strtotime(date('Y-m-d')  ." ". $start_work);
    
    if (strtotime($start_work) <= strtotime($end_work)){
      $endDateTime = strtotime(date('Y-m-d')  ." ". $end_work);
      $previousDayEnd = strtotime(date('Y-m-d')  ." ". $end_work . "-1 days");
    }
    
    else{
     $endDateTime = strtotime(date('Y-m-d')  ." ". $end_work . "+1 days");
     $previousDayEnd = strtotime(date('Y-m-d')  ." ". $end_work );
    }
    
    return ($currentDateTime >= $startDateTime && $currentDateTime <= $endDateTime) ? true : false;
    return ($currentDateTime < $startDateTime && $currentDateTime < $previousDayEnd) ? true : false;
    ?>

Но не выходит. В php новичок, только основы изучаю, видимо допустил ошибку...
Помогите пожалуйста)

Answer 1

[Максим]

$now = new \DateTimeImmutable();
$startWork = new \DateTimeImmutable("08:00");
$endWork = new \DateTimeImmutable("23:00");

if ($startWork <= $now && $now <= $endWork) {
    return true;
} else {
    return false;
}

Comments

[Денис Московский]

Как ваш код сработает, если режим работы изменится и будет с 8:00 до 01:00 ?

[Максим]

Максим Морозов, добавил return. Меняете переменную$endWork.

[Денис Московский]

Максим, это понятно, но я изначально пробовал вашу схему.
Возникает проблема если любое время после полуночи, так как оно по сути меньше (от 0).

[Максим]

Максим Морозов, https://rextester.com/MSWP56927

[Максим]

Максим Морозов, https://rextester.com/OKMK36980

[Денис Московский]

Максим, ну вот же, если $endWork будет значение времени после полуночи, то close выводится при любых обстоятельствах.

[Максим]

Максим Морозов, ну Дак правильно)) вы же день то не учитываете)) 1 час это текущего дня. У вас магазин работает с 1:00 до 8:00 Тогда вам и дату нужно передавать. А если правильно указывать до часу, то надо указывать дату следующего дня. Так как после полуночи следующий день. Пример https://rextester.com/QJMW57701

Или передавайте дату вместе со временем, а не только время.

[Максим]

Максим Морозов

$now = new \DateTimeImmutable();
$startWork = new \DateTimeImmutable("13.06.2020 08:00");
$endWork = new \DateTimeImmutable("14.06.2020 01:00");

if ($startWork <= $now && $now <= $endWork) {
    return true;
} else {
    return false;
}

[Денис Московский]

Максим, Вот, это мне и нужно было, просто иногда нужно будет использовать оба варианта.

[Максим]

Максим Морозов, тогда храните ещё где-то дату) и передавайте её тоже, если она есть. Либо сразу хранить дату и время. Тут уж сами смотрите

[Денис Московский]

Максим, Понял, спасибо!

[Денис Московский]

Максим, извините, что беспокою, а Вы случайно не подскажите, как сделать тоже самое, только с днями недели. Я нашел инфу, но запутался, в итоге не выходит ничего. Пытаюсь сделать так: Если день недели - суббота или воскресенье, то true, если нет то false, ну или наоборот, мне неважно.

[Максим]

Максим Морозов,

$now = (new \DateTimeImmutable())->format('w');

if ($now === '6'|| $now === '0') {
    return true;
} else {
    return false;
}

[Денис Московский]

Максим, большое спасибо!

Answer 2

[Stalker_RED]

Вы сравниваете строки, это иногда работает, но не всегда. Лучше так:

function isOpen() {
  $now = new DateTime('now');
  $open = new DateTime('today 08:00');
  $closed = new DateTime('today 23:00');
  return ($now > $open && $now < $closed);
}

echo isOpen();

Comments

[Денис Московский]

А как получить не 1, а именно true? И что если $closed будет равен 01:00, возникнет ошибка?

[Stalker_RED]

Максим Морозов, там возвращается именно true, это уже echo конвертирует его в единицу, для читаемости.

Если напишете что "today 01:00", то будет всегда false. Нужно в таком случае само выражение поменять.
Если укажете полную дату-время, то посчитает правильно.