Как продумать условие if / else?

Question

[wagwan]

Текст вопроса не может быть менее 30 и более 10 тысяч символов

Answer 1

[Stalker_RED]

Как правильней в условии разделить 6,5 часов на каждый час?

А зачем его делить? if ($hour > 17 || $hour == 00) {}

И вообще, есть такая штука как php.net/manual/ru/datetime.diff.php

Или по велосипедному:

$start = strtotime('today 17:30');
$end  = strtotime('tomorrow midnight');
$now = time();

if($start <= $now && $now <= $end) {
  // ...
}

И еще один момент, у вас в каждой ветке повторяется "вывожу файл X" - можно написать это один раз, в самом конце.

Comments

[Stalker_RED]

Иными словами, в этот промежуток времени нужно сделать только 5 запросов.

Может тогда стоит создать пять задач в кроне, например? Или сравнивать текущее время с временем последнего обновления кэша, и если прошел час, и время в нужном диапазоне - скачивать новый.

[Stalker_RED]

wagwan: php.net/manual/ru/function.filemtime.php

Answer 2

[maximw]

Смысла проверять содержимое нет, если у вас не сотни операций в секунду, когда производительность упрется в дисковый ввод/вывод, перезаписывайте просто и все.

if (такое-то время) {
  if (проверяю наличие файла X) {
      перезаписываю содержимым файла Y
  } else {
     создаю файл и записываю туда содержимое файла Y
  }
}
вывожу файл X