Discussion :

[Devilju69]

Bonjour,

Je dois afficher un nombre de passage en fonction de l'heure et de la date.
Actuellement je mets la date dans la première colonne via une boucle for mais après je réalise les requetes sur chaque tranche horaire. C'est très long.
Quel est le meilleur moyen d'optimiser cela ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<td>
	<?php 
		$sql_COUNT1 = 'SELECT COUNT(date_passage) FROM passages WHERE MONTH(date_passage)="'.$mois.'" AND  YEAR(date_passage)="'.$annee.'" AND heure_passage>="08:00:00" AND heure_passage<="09:00:00" BETWEEN "'.$date_debut_en.'" AND "'.$date_fin_en.'"'; 
		$req_COUNT1 = mysql_query($sql_COUNT1) or die(mysql_error());;
		$COUNT1 = mysql_fetch_assoc($req_COUNT1);
		echo $COUNT1['COUNT(date_passage)'];
	?>
</td>
<td>
	<?php 
		$sql_COUNT2 = 'SELECT COUNT(date_passage) FROM passages WHERE MONTH(date_passage)="'.$mois.'" AND  YEAR(date_passage)="'.$annee.'" AND heure_passage>="09:00:00" AND heure_passage<="10:00:00" BETWEEN "'.$date_debut_en.'" AND "'.$date_fin_en.'"'; 
		$req_COUNT2 = mysql_query($sql_COUNT2) or die(mysql_error());;
		$COUNT2 = mysql_fetch_assoc($req_COUNT2);
		echo $COUNT2['COUNT(date_passage)'];
	?>
</td>

[mathieu]

C'est très long.

que trouvez vous long ? la taille de code ou le temps d'exécution des requêtes sql ?

[Devilju69]

C'est le temps d’exécution des requêtes SQL ! La page met énormément de temps à s'afficher car je fais une requête par heure et par mois

[Séb.]

Généralement on évite les boucles de requêtes et on fait plutôt une requête unique qui ramène l'ensemble des données nécessaires, puis on met en forme le résultat avec PHP.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$sql_COUNT1 = 'SELECT COUNT(date_passage) FROM passages WHERE MONTH(date_passage)="'.$mois.'" AND  YEAR(date_passage)="'.$annee.'" AND heure_passage>="08:00:00" AND heure_passage<="09:00:00" BETWEEN "'.$date_debut_en.'" AND "'.$date_fin_en.'"'; 
...
$sql_COUNT2 = 'SELECT COUNT(date_passage) FROM passages WHERE MONTH(date_passage)="'.$mois.'" AND  YEAR(date_passage)="'.$annee.'" AND heure_passage>="09:00:00" AND heure_passage<="10:00:00" BETWEEN "'.$date_debut_en.'" AND "'.$date_fin_en.'"';

Beaucoup de choses ne vont pas.
Il y a un problème de syntaxe avec ton SQL et la clause BETWEEN redondante avec MONTH() et YEAR(). Ton SQL mal formaté rend cela difficile à identifier, et le PHP mal utilisé vient obscurcir tout cela.
Autre chose, en SQL on délimite les chaînes avec ' et pas " (qui sert à délimiter les noms d'objet).
MONTH() retourne un numéro de mois sans zéro initial (ex, 9 et pas 09 pour septembre), ainsi je ne comprends pas les " (qui devraient être des ') autour de $mois.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

AND heure_passage<="10:00:00" BETWEEN "'.$date_debut_en.'" AND "'.$date_fin_en.'"';

Par ailleurs, avec ton système actuel les valeurs aux bornes seraient comptés 2 fois

Ton modèle auquel j'ajoute une colonne id :

passages
-------------
id
date_passage (date)
heure_passage (time)

Requête unique donnant le nombre de passage par tranche horaire sur une période donnée :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
$date_debut_en = '2024-01-08';
$date_fin_en = '2024-01-14';
 
$sql = <<<SQL
    SELECT ALL
        date_passage,
        TIME_FORMAT(heure_passage, '%H:00:00') AS heure_passage, -- On ne garde que la composante "heure" pour le groupement, ex. 09:49:23 => 09:00:00
        COUNT(*) AS volume
    FROM passages
    WHERE date_passage BETWEEN '{$date_debut_en}' AND '{$date_fin_en}'
    GROUP BY 1, 2 -- Groupement sur les colonnes numéro 1 et 2
    ORDER BY 1 ASC, 2 ASC -- Tri sur les colonnes numéro 1 et 2
    SQL;
 
$data = $pdo->query($sql)->fetchAll();

Cette requête unique donnera un tableau $data contenant des données de base comme :

date_passage heure_passage volume
2023-01-08   06:00:00      2
2023-01-08   08:00:00      6
2023-01-08   09:00:00      19
2023-01-08   12:00:00      17
...
2023-01-14   17:00:00      5
2023-01-14   20:00:00      2

Il ne reste plus qu'à boucler sur ton calendrier et/ou $data pour afficher les données. À ce niveau tu ne donnes pas suffisamment d'infos pour que j'en dise plus.

TIME_FORMAT() => https://dev.mysql.com/doc/refman/8.0...on_time-format