Requête trop lente

**patheo2** · 26/10/2017, 21h33

Bonsoir à tous,
Cette requête met trop de temps à s'exécuter.

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
17
18
19
20
21
22
23
24
25
26
function recherche_cessation ($connexion)
{
 
$matricule = $_POST['matricule'];
 
$cessations = $connexion->prepare("SELECT *  
FROM absences A
JOIN employes E, type_conge T 
WHERE E.num_matricule = A.matri_employe
AND A.id_type = T.id_type_conge
AND A.motif ='Congés'  AND A.matri_employe = :matricule
 ");
 
$cessations->bindParam(':matricule', $matricule, PDO::PARAM_STR); 
$cessations->execute();
 
return $cessations->fetch(PDO::FETCH_OBJ);
}
 
$lesCessation=recherche_cessation ($connexion);
 
while( $lesCessations ) // on récupère les produits
{
	//j'affiche les enregistrements
 
}$cessations->closeCursor();

Pourrais-je avoir de l'aide pour résoudre le problème?
Merci

Invité · 27/10/2017, 07h36

Bonjour,

1- bonne utilisation de JOIN
2- SELECT * : as-tu besoin de tous les champs ? Sinon, il faut nommer ceux utiles

Code :

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

1
2
3
4
5
6
7
8
9
$cessations = $connexion->prepare("SELECT A.champ1, A.champ2, E.champ3, T.champ4, ......
FROM absences A
JOIN employes E 
  ON E.num_matricule = A.matri_employe
JOIN type_conge T 
  ON A.id_type = T.id_type_conge
WHERE A.motif ='Congés'  
AND A.matri_employe = :matricule
 ");

**patheo2** · 27/10/2017, 08h24

Bonjour jreaux62,
J'essaie et je te reviens. Merci

**CinePhil** · 27/10/2017, 09h06

3) Les colonnes sur lesquelles portent les conditions de jointure sont-elles indexées ?
4) Les colonnes sur lesquelles porte la restriction (clause WHERE) sont-elles indexées ?
5) Combien y de t-il de lignes dans les tables ?

**patheo2** · 27/10/2017, 11h05

Envoyé par CinePhil

3) Les colonnes sur lesquelles portent les conditions de jointure sont-elles indexées ?
4) Les colonnes sur lesquelles porte la restriction (clause WHERE) sont-elles indexées ?
5) Combien y de t-il de lignes dans les tables ?

Non aucune des colonnes n'est indexée.
Il y a 13 lignes maximum dans chacune des tables.

1- bonne utilisation de JOIN
2- SELECT * : as-tu besoin de tous les champs ? Sinon, il faut nommer ceux utiles

J'ai repris la requête comme ceci:

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
17
18
19
$cessations = $connexion->prepare("SELECT 
A.matri_employe AS matricule,
E.nom AS nom,
E.prenom AS prenom,
A.numerodecision AS numerodecision,
A.date_decision AS date_decision,
A.tranche AS tranche,
A.date_debut AS date_debut,
A.date_fin AS date_fin,
T.designation_conge AS designation_conge,
T.nbre_jour AS nbre_jour
FROM absences A
JOIN employes E 
  ON E.num_matricule = A.matri_employe
JOIN type_conge T 
  ON A.id_type = T.id_type_conge
WHERE A.motif ='Congés'  
AND A.matri_employe = :matricule
 "

Mais j'ai l'impression que mon while boucle à l'infini. En effet, quand je vérifie dans la table absences, seulement six ligne répondent à la requête mais la requête m'affiche plus d'une dizaine de ligne à l'écran et le navigateur plante

**Celira** · 27/10/2017, 11h23

Ta requête a l'air correcte (en tous cas, beaucoup plus que la 1ere version

) Tu as exécuté la requête directement sur la base ? (via PhpMyAdmin ou équivalent) Tu obtiens le bon résultat ?

13 lignes dans chacune des 3 tables, si tu te loupes sur les jointures dans la requête et que ça remonte le produit cartésien de la totalité des lignes, tu peux potentiellement récupérer 2000 et quelques résultats

On peut voir le code qui fait l'affichage ?

**CinePhil** · 27/10/2017, 11h49

Non aucune des colonnes n'est indexée.

Arf ! Chose à corriger !
Toutes les clés étrangères doivent être indexées.
Les colonnes sur lesquelles portent les recherches les plus probables doivent l'être aussi, sauf si elle ne sont pas discriminantes (exemple : on n'indexe pas une colonne booléenne parce qu'il n'y aura au plus que deux valeurs, même avec des millions de lignes. L'index est inutile et ne sera probablement jamais utilisé par le SGBD).

Il y a 13 lignes maximum dans chacune des tables.

Avec une aussi faible volumétrie, la requête devrait s'exécuter en une fraction de seconde, même sans index !

Code SQL :

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

ON E.num_matricule = A.matri_employe

De quel types sont les colonnes présentes dans cette jointure ?
D'une manière générale, on ne fait les jointures que sur des identifiants de type entier parce que c'est beaucoup plus rapide que sur des colonnes de type alphanumérique.

**patheo2** · 27/10/2017, 12h27

Tu as exécuté la requête directement sur la base ? (via PhpMyAdmin ou équivalent) Tu obtiens le bon résultat ?

Oui exécuté directement sur la base ça marche bien.

On peut voir le code qui fait l'affichage ?

Voici le code complet :
la fonction:

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
17
18
19
20
21
22
23
24
25
26
27
function recherche_cessation ($connexion,$matricule)
{
$cessations = $connexion->prepare("SELECT 
A.matri_employe AS matricule,
E.nom AS nom,
E.prenom AS prenom,
A.numerodecision AS numerodecision,
A.date_decision AS date_decision,
A.tranche AS tranche,
A.date_debut AS date_debut,
A.date_fin AS date_fin,
T.designation_conge AS designation_conge,
T.nbre_jour AS nbre_jour
FROM absences A
JOIN employes E 
  ON E.num_matricule = A.matri_employe
JOIN type_conge T 
  ON A.id_type = T.id_type_conge
WHERE A.motif ='Congés'  
AND A.matri_employe = :matricule
 ");
 
$cessations->bindParam(':matricule', $matricule, PDO::PARAM_STR); 
$cessations->execute();
 
return $cessations->fetch(PDO::FETCH_OBJ);
}

l'affichage :

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
17
$matricule = $_POST['matricule'];
 
$lesCessations=recherche_cessation ($connexion,$matricule);
 
while( $lesCessations ) // on récupère les produits
{
		echo "<tr> ";
 
        echo '<td align="center">'.$lesCessations->matricule.'</td>';
		echo '<td align="center">'.$lesCessations->nom .' '.$lesCessations->prenom.'</td>';
		echo '<td align="left" style ="padding-left:10px">'.$lesCessations->numerodecision.' du '.date("d/m/Y",strtotime($lesCessations->date_decision)).'</td>';
		echo '<td align="center">'; if($lesCessations->tranche=="1") echo 'Première'; else echo "Deuxième"; echo'</td>';
		echo "<td align=\"center\"> du ".date("d/m/Y",strtotime($lesCessations->date_debut))." au ".date("d/m/Y",strtotime($lesCessations->date_fin))."</td>";
		echo '<td align="center"><a href="imp_cessation.php?tranche='.$lesCessations->tranche.'&matricule='.$matricule.'&date_decision='.date("d/m/Y",strtotime($lesCessations->date_decision)).'&numerodecision='.$lesCessations->numerodecision.'&type='.$lesCessations->designation_conge.'&nbrejr='.$lesCessations->nbre_jour.'">Imprimer</a></td>';
 
 
}$cessations->closeCursor();

De quel types sont les colonnes présentes dans cette jointure ?

Les champ sont en VARCHAR

**CinePhil** · 27/10/2017, 13h13

1)

Cette requête met trop de temps à s'exécuter

C'est combien, "trop de temps" ?

2)

Les champ sont en VARCHAR

Les champs sont à la campagne ou dans les formulaires, pas dans les tables SQL qui ne sont composées que de colonnes et de lignes.

Comme dit dans mon précédent message, des jointures avec des colonnes en VARCHAR, c'est pas top !

**patheo2** · 27/10/2017, 13h35

Envoyé par CinePhil

1)
C'est combien, "trop de temps" ?

2)
Les champs sont à la campagne ou dans les formulaires, pas dans les tables SQL qui ne sont composées que de colonnes et de lignes.

Comme dit dans mon précédent message, des jointures avec des colonnes en VARCHAR, c'est pas top !

1) Ça met plus de trois minutes pour afficher les lignes alors que directement sur la base c'est en quelques fractions de secondes.
Par ailleurs, comme je l'ai dit plus haut, ma boucle While semble bouclé à l'infini et fait planter le navigateur (et la machine avec).

2) Compris pour les colonnes en VARCHAR

**Celira** · 27/10/2017, 14h14

Ya un truc bizarre dans ton histoire : la fonction qui exécute la requête fait un seul appel à fetch, donc ne récupère que la 1ere ligne de résultat. Donc tu ne peux pas avoir plus d'une seule ligne affichée

**CinePhil** · 27/10/2017, 14h14

Le problème est donc dans le PHP.

Pourquoi faire une boucle while alors que vous n'avez ramené qu'une ligne avec return $cessations->fetch(PDO::FETCH_OBJ); ?

**Celira** · 27/10/2017, 14h18

Ah j'ai trouvé !

Code :

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

while( $lesCessations ) // on récupère les produits

Ya pas de modification de la condition ! Donc en fait, tu affiches la même ligne en boucle infinie !

Il faut que tu remplaces le fetch par un fetchAll et le while par un foreach.

**patheo2** · 27/10/2017, 14h57

Coool! Ca marche très bien. Merci bien. Bénédictions!

Requête trop lente

PHP & Base de données

Discussions similaires

Partager

Partager