Discussion :

[3psilOn]

Bonjour,
Dans une table je stocke les états (workflow) de mes dossiers.
Je récupére via une requête les derniers états de mes dossiers comme suit :

Code :

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

1
2
3
4
5
6
7
 
	SELECT
	e.etat,
	e.id
	FROM etats AS e
	WHERE
	e.id = (SELECT MAX(id) FROM etats WHERE e.id = id)

La table contient bien évidemment beaucoup d'enregistrements de l'ordre des 50.000 et elle ne cesse d'évoluer...
Le "SELECT MAX(id) FROM etats WHERE e.id = id" me gêne du moment qu'elle est exécutée autant de fois que de résultats

Est ce qu'on peut pas faire mieux?

Merci

[KookieMonster]

Pourquoi ne pas mettre le MAX(id) directement dans le SELECT?

[3psilOn]

Parce que la requète

Code :

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

1
2
3
SELECT etat, MAX(id) AS LastID
FROM etats
GROUP BY id

donne bien le dernier id mais pas le dernier état, elle donne le premier état trouvé

[CinePhil]

Que représente id ? L'identifiant de l'état ou l'identifiant du dossier ?
Que cherches-tu à obtenir précisément ?

[3psilOn]

Voici la structure de ma table :
id : int (primary ,auto increment)
etat : varchar(50) (en cours, validé, clôturé)
idDossier : int

Je souhaite récupérer les états récents de mes dossiers, sachant que cette table enregistre tous les états d'un même dossier.

J'espère que c'est plus clair maintenant

[CinePhil]

Pour commencer, tu devrais revoir la structure de ta BDD.

Règle de gestion :
Un dossier a de un à plusieurs états et un état peut avoir plusieurs dossiers.

MCD :
dossier -1,n----avoir----0,n- etat

Tables :
dossier (dos_id, ...)
etat (eta_id, eta_libelle)
dos_avoir_eta (dae_id_dossier, dae_id_etat, dae_date...)

La requête avec ta structure et par étapes :

1) Quel est le dernier état de chaque dossier

Code :

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

1
2
3
SELECT idDossier, MAX(id) AS id_max
FROM etats
GROUP BY etat

2) Quel est le libellé de l'état correspondant pour chaque dossier ?

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT e.idDossier, e.etat
FROM etat e
INNER JOIN
(
	SELECT idDossier, MAX(id) AS id_max
	FROM etats
	GROUP BY etat
) tmp 
	ON tmp.idDossier = e.idDossier
	AND tmp.id_max = e.id

La sous-requête n'est exécutée qu'une fois et la jointure met en correspondance les données.

Ta requête était fausse puisque tu faisais correler id (l'identifiant de l'état) avec lui même.

[3psilOn]

J'ai modifié ma requête comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT e.idDossier, e.etat
FROM etat e
INNER JOIN
(
	SELECT idDossier, MAX(id) AS id_max
	FROM etats
	GROUP BY idDossier
) tmp 
	ON tmp.idDossier = e.idDossier

Avec GROUP BY sur "idDossier" au lieu de "etat" et une seule condition de jointure. Elle donne le même résultat que ma requête initiale, sauf qu'elle est 10 fois plus rapide...
Merci

[CinePhil]

Effectivement, je m'étais trompé sur la colonne du GROUP BY !

Si tu ne fais la jointure que sur idDossier, tu vas avoir tous les états de tous les dossiers. Autant ne pas chercher le dernier état de chaque dossier dans la sous-requête alors !

La seconde condition de jointure est nécessaire.

La sous requête donne l'identifiant du dernier état pour chaque dossier.
La requête principale joint cette sous requête à la table des états pour récupérer le libellé du dernier état de chaque dossier.
La condition de jointure est donc nécessairement double.

La bonne requête :

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT e.idDossier, e.etat
FROM etat e
INNER JOIN
(
	SELECT idDossier, MAX(id) AS id_max
	FROM etats
	GROUP BY idDossier
) tmp 
	ON tmp.idDossier = e.idDossier
	AND tmp.id_max = e.id

[3psilOn]

Avec "INNER JOIN" on peut se passer de la 2e condition de jointure, puisque par définition elle ne prend que les éléments qui ont une correspondance entre les 2 tables

[CinePhil]

Si j'ai bien compris, il y a plusieurs fois idDossier dans la table etat ?

Alors si tu enlèves la condition de jointure sur l'id_max de la sous-requête, tu auras tous les états d'un dossier et non pas seulement le dernier (le max_id du dossier).

[3psilOn]

Désolé je m'étais planté dans les noms des champs comme je les ai simplifié dans l'exemple

Voici ma requête qui marche et qui donne les mêmes résultats que ma première requête :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT e.idDossier, e.etat
FROM etat e
INNER JOIN
(
	SELECT MAX(id) AS id_max
	FROM etats
	GROUP BY idDossier
) tmp 
	ON tmp.id_max = e.id

Avec une jointure interne sur la clé primaire plutôt que idDossier
En effet, il y a plusieurs fois idDossier dans la table etat...

Exemple de données :

id idDossier etat
1 1 créé
2 2 créé
3 1 validé
4 3 créé
5 2 rejeté
6 1 clôturé

Merci encore une fois pour l'astuce