Как правильно сформировать логическое условие в php?

Question

[Анастасия Foxman]

Здравствуйте! Пишу компонент для одной CMS и я столкнулась с такой проблемой - никак не получается правильно сформировать логическое И ИЛИ, чтобы скрипт правильно работал.

Вот в чём суть:

Есть следующие переменные:

$work_time = '09:00'; // Время открытия компании
$work_end_time = '18:00'; // Время закрытия компании
$dinner_time = '13:00'; // Время начала обеда
$dinner_end_time = '14:00'; //Время окончания обеда

Как понятно из содержимого переменных - нужно выстроить логику показа определенного сообщения в зависимости от текущего времени. Например, если до открытия остается меньше часа, то выводить сообщение - откроется через N мин. Если открыто в данный промежуток времени - то выводить фразу "Сейчас открыто". Ну и если время совпадает с обедом - то выводить фразу "Сейчас обед".

Вот что я смогла сделать с временем открытия и закрытия:

<?php echo $work_time != $work_end_time && (date('H:i') > $work_end_time || date('H:i') < $work_time) ? 'Закрыто' : 'Открыто' ?>

Данное условие работает, однако, если например установить время открытия - 01:00, а время закрытия - 03:00, то логика почему-то нарушается.

Помогите реализовать данное условие. Заранее всем спасибо!

Comments

[AUser0]

date('H:i') > $work_end_time - это так не работает, переписывайте.

[Дмитрий]

Ох уж эти однострочники. Чтоб разобраться, напишите в столбик ваши условия, отладте, потом собирайте в строку

Answer 1

[ThunderCat]

Во первых - вы сравниваете строки, что уже хреново.
Во вторых - в правилах выставления времени уже должны быть ограничения, например что время закрытия больше времени открытия, так же с обедом, и еще надо проверить что обед попадает между закрытием и открытием.
Ну и когда вы приведете все условия в нормальный вид можно думать о логике, которая по факту проще пареной репы... ну и $work_time по уму должно быть $work_start_time, то же самое с $dinner_time.
Если все еще не ясна логика:

$work_time = [
  'work_start_time'=>'09:30',
  'work_end_time'=>'18:20',
  'dinner_start_time'=>'13:30',
  'dinner_end_time'=>'14:00',
];
$answer = 'Closed';
$date = str_replace(':','',$work_time);
$now = date('Hi');
if( $date['work_start_time'] <= $now && $now <= $date['work_end_time'] ) $answer = 'Now we open';
if( $date['dinner_start_time'] <= $now && $now <= $date['dinner_end_time'] ) $answer = 'Now we on break';
echo $answer;

Comments

[Анастасия Foxman]

ThunderCat спасибо, попробую ваше решение. Я довольно много прочитала литературы по поводу работы с датой и временем, и честно говоря пока мне это тяжеловато дается. По вашему решению отпишусь через некоторое время, как буду у рабочего ПК.

[ThunderCat]

Анастасия Лисиченко, тут не нужна работа с датой, и с временим тоже. Задача кажется связанной с датой, но по факту речь о сравнении 4 чисел.

[Анастасия Foxman]

ThunderCat, благодарю, кажется это то, что мне нужно. Просто проблема стояла в том, что это не маленький скрипт со статичными данными - данные передаются из компонента админки. Админка компонента выглядит вот так:



Помимо проверки на время работы есть еще куча других проверок - выходной или нет, закрыто ли по другим причинам и тд.

Но ваше решение кажется мне идеальным - большое спасибо, отмечаю решением!

Answer 2

[psiklop]

Проверяйте

$work_time = '09:00'; 
$work_end_time = '18:00'; 
$dinner_time = '13:00';
$dinner_end_time = '14:00';
$current = intval(date("G"));
echo ($current<intval($work_time) && (intval($work_time)-$current)==1)?"Откроется через ".(60-intval(date("i")))." минут":(($current==intval($dinner_time))?"Сейчас обед":(($current+1>intval($work_end_time) || $current<intval($work_time))?"Закрыто":"Сейчас открыто"));

Comments

[Анастасия Foxman]

К сожалению не работает. Всегда "Открыто", при любых значениях. Для примера выставила у себя в скрипте режим работы с 01:00 до 03:00 ночи

[ThunderCat]

ага, давайте попробуем теперь

$work_time = '09:30'; 
echo intval($work_time);

, интересно, ЧТО ЖЕ МЫ ПОЛУЧИМ?

[Анастасия Foxman]

ThunderCat, 9 получим.

[psiklop]

Анастасия Лисиченко, та не, чтобы проверить тоже надо скилл иметь)). Про ваш скрипт ниче не знаю, достаточно менять $current прямо в notepad+

[psiklop]

ThunderCat, согласен, решение хреновое. Только с целыми числами. Я вначале не понял задания, думал это константы.

Answer 3

[Михаил Ливач]

Если минуты не нужны , то можно их отбросить и работать только с часами, как с числами. Только нужно не "09", а 9.
Если минуты нужны, то можно перевести время в минуты ( например, "9:30" -> 9*60+30) и снова работать с числами.