Requête trop lente

Version imprimable

Requête trop lente

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

Code:

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

Bonjour,

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

Code:

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
 ");

27/10/2017, 08h24
patheo2

Bonjour jreaux62,
J'essaie et je te reviens. Merci
27/10/2017, 09h06
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 ?

Citation:

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.

Citation:

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:

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

27/10/2017, 11h23
Celira

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 :aie:

On peut voir le code qui fait l'affichage ?
27/10/2017, 11h49
CinePhil

Citation:

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).

Citation:

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 !

Citation:

Code:

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.

Citation:

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.

Citation:

On peut voir le code qui fait l'affichage ?

Voici le code complet :
la fonction:

Code:

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:

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();

Citation:

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

Les champ sont en VARCHAR

27/10/2017, 13h13
CinePhil

1)

Citation:

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

C'est combien, "trop de temps" ?

2)

Citation:

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. :mrgreen:

Comme dit dans mon précédent message, des jointures avec des colonnes en VARCHAR, c'est pas top !
27/10/2017, 13h35
patheo2

Citation:

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. :mrgreen:

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
27/10/2017, 14h14
Celira

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 :koi:
27/10/2017, 14h14
CinePhil

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); ?
27/10/2017, 14h18
Celira

Ah j'ai trouvé !

Code:

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.
27/10/2017, 14h57
patheo2

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