Как оптимизировать код?

Question

[jazzman7]

Доброго времени суток. Была задача сделать график и отображать прямой данные из базы за 24 часа. Задачу реализовал, криво, да работало, перешёл сегодня с mysql 5 на mysql 8, страница начала падать, после этого перешёл естественно обратно, но заметил как страница быстро грузится без этого кода даже в единичном варианте, а он у меня к тому же 6 раз повторяется в разных вариациях, в общем в любом случае надо оптимизировать, попытался, что-то сделать, но ничего не вышло. В общем буду благодарен если кто-нибудь подскажет как это "чудо" оптимизировать

for($hour = 0; $hour< 24; $hour++){ 
$start_time = mktime($hour, 0, 0, date("m"), date("d"), date("Y")); 
$finish_time = mktime($hour+1, 0, 0, date("m"), date("d"), date("Y")); 


$host = $pdo->prepare("SELECT * FROM hosts WHERE time BETWEEN ? AND ? AND id_sait = ?");
$host->execute(array($start_time,$finish_time,$id));
$host = $host->rowCount();
;

if(strlen($hour) == 1) $hour = '0'.$hour; 
if(strlen($hour + 1) == 1) $f = '0'.($hour + 1); 
else $f = $f + 1; 
if ($hour<23){ $aa=',';} else {$aa='';}
echo ($host).$aa."";
}

Answer 1

[FanatPHP]

1. учить SQL и вместо этого циклического ужаса сделать один запрос

SELECT hour(time) hour, count(*) count FROM hosts WHERE id_sait = ? 
AND time BETWEEN unix_timestamp(curdate()) and unix_timestamp()
GROUP BY hour(time);

2. комбинированный индекс на id_sait, time

Comments

[FanatPHP]

если надо обязательно за 24 часа, то вручную добить нулями

[jazzman7]

Насчёт ужаса согласен и большое спасибо за код, если есть время расшифруйте пожалуйста его немного, особенно интересует эта часть BETWEEN unix_timestamp(curdate()) and unix_timestamp(), начало то понятно в принципе

[FanatPHP]

ну у тебя время хранится в unix timestamp
значит надо нормальное фремя переводить в этот формат
ты собираешь информацию за сегодня. начало дня - это curdate(), то есть дата без времени.
конец периода - это текущее время, то есть просто unix_timestamp

[jazzman7]

FanatPHP, благодарю

[jazzman7]

FanatPHP, почему-то не работает, выдает
#1140 - In aggregated query without GROUP BY, expression #1 of SELECT list contains nonaggregated column 'top.hosts.time'; this is incompatible with sql_mode=only_full_group_by

[FanatPHP]

ой. я забыл group by