Discussion :

[Fred_34]

Bonjour à tous,

J'ai deux tables identiques (tableCourante et tableArchive) , et j'aimerai récupérer le ou les enregistrements qui ont la meilleure note.

Cette requête fonctionne, mais elle ne me parait pas trop optimisée :

Code :

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

1
2
3
4
5
6
7
8
 
 
SELECT  * FROM 
    (SELECT  * from tableCourante union SELECT  * FROM tableArchive) A
    where note= 
       (select max(note) from 
           (SELECT  * FROM tableCourante union SELECT  * FROM tableArchive) B
       )

Il y a t'il une meilleure façon de faire ?

Merci pour votre aide.

[Fred_34]

Entre temps, j'ai modifié cette requête par celle-ci. Je ne sais plus exactement pourquoi, mais elle devait probablement être plus performante.

Code :

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

1
2
3
4
5
6
7
 
SELECT * FROM 
  (SELECT * FROM tableCourante UNION ALL SELECT * FROM tableArchive) AS A
  WHERE A.note= 
    (SELECT max(note) As note FROM tableCourante
    UNION ALL SELECT max(note) As note FROM tableArchive
    ORDER BY note DESC LIMIT 1);

Pour récupérer la valeur min j'ai du la modifier un petit peu :

Code :

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

1
2
3
4
5
6
7
 
SELECT * FROM 
  (SELECT * FROM tableCourante UNION ALL SELECT * FROM tableArchive) AS A
  WHERE A.note= 
    (SELECT min(note) As note FROM tableCourante
    UNION ALL SELECT min(note) As note FROM tableArchive
    ORDER BY (CASE WHEN note IS NULL THEN 1 ELSE 0 END),note ASC LIMIT 1);

Maintenant, si j'ajoute une condition, par exemple récupérer tous les enregistrements ayant la note minimum ou l'UV est 2, ça donne ceci :

Code :

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

1
2
3
4
5
6
7
8
 
SELECT * FROM 
  (SELECT * FROM tableCourante UNION ALL SELECT * FROM tableArchive) AS A 
  WHERE A.note =
    (SELECT min(note) As note FROM tableCourante WHERE uv=2
    UNION ALL SELECT min(note) As note FROM tableArchive WHERE uv=2
    ORDER BY  (CASE WHEN note IS NULL THEN 1 ELSE 0 END),note ASC LIMIT 1)
  AND A.uv=2;

J'ai un peu l'impression de faire n'importe quoi...
Si quelqu'un pouvait me rassurer ou me donner des pistes sur la bonne façon de faire, ça m’arrangerait bien.
Merci.

[CinePhil]

Il faut éviter de relancer la guerre des étoiles !

Quelle est la note maxi des deux tables ?

Code :

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

1
2
3
4
5
6
7
8
9
SELECT MAX(note) AS note_max
FROM
(
	SELECT note
	FROM tableCourante
	UNION 
	SELECT note
	FROM tableArchive
) u1

Comme on ne fait l'union que sur les notes et pas sur toutes les colonnes, il y aura probablement moins de lignes, UNION supprimant les doublons.

Maintenant, récupérons les lignes de deux tables qui ont cette note_max :

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
SELECT --les colonnes nécessaires et pas étoile !
FROM
(
	SELECT --les colonnes nécessaires et pas étoile !
	FROM tableCourante
	UNION ALL
	SELECT --les colonnes nécessaires et pas étoile !
	FROM tableArchive
) u2
WHERE note = 
(
	SELECT MAX(note) AS note_max
	FROM
	(
		SELECT note
		FROM tableCourante
		UNION 
		SELECT note
		FROM tableArchive
	) u1
)

Cette fois pour récupérer toutes les lignes ayant la note MAX, il faut bien utiliser UNION ALL pour éviter le dédoublonnage.
Si tu as besoin de savoir de quelle table provient la ligne, tu ajoutes une colonne fictive :

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
SELECT --les colonnes nécessaires et pas étoile !
FROM
(
	SELECT 'tableCourante' AS Nom_table, --les colonnes nécessaires et pas étoile !
	FROM tableCourante
	UNION ALL
	SELECT 'tableArchive', --les colonnes nécessaires et pas étoile !
	FROM tableArchive
) u2
WHERE note = 
(
	SELECT MAX(note) AS note_max
	FROM
	(
		SELECT note
		FROM tableCourante
		UNION 
		SELECT note
		FROM tableArchive
	) u1
)

[Fred_34]

Merci pour ta réponse.

Si je modifie ta requête pour ajouter la condition uv=2, ça me donne:

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
SELECT --les colonnes nécessaires et pas étoile !
FROM
(
	SELECT --les colonnes nécessaires et pas étoile !
	FROM tableCourante
	UNION ALL
	SELECT --les colonnes nécessaires et pas étoile !
	FROM tableArchive
) u2
WHERE note = 
(
	SELECT MAX(note) AS note_max
	FROM
	(
		SELECT note
		FROM tableCourante
		WHERE uv=2
		UNION 
		SELECT note
		FROM tableArchive
		WHERE uv=2
	) u1
)
AND uv=2;

J'ai ajouté "uv=2" à 3 endroits

Est-ce la bonne façon de faire ? Faut-il plutôt utiliser une jointure?

Avec une jointure je suis arrivé à cette requête :

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
SELECT --les colonnes nécessaires et pas étoile !
FROM
(
	SELECT --les colonnes nécessaires et pas étoile !
	FROM tableCourante
	UNION ALL
	SELECT --les colonnes nécessaires et pas étoile !
	FROM tableArchive
) u1
RIGHT JOIN 
(
 
SELECT idtable,uv,MAX(note) AS note
	FROM
	(
		SELECT idtable,uv,note
		FROM tableCourante
		UNION 
		SELECT idtable,uv,note
		FROM tableArchive
	) u2  
  WHERE uv=2
)
u3
USING( idtable,note,uv);

Le résultat a l'air correct, mais comme d'habitude comment savoir si c'est la bonne façon de faire...

Il faut éviter de relancer la guerre des étoiles !

Vaste débat...

Dans certain cas l'emploi de l'étoile est une erreur, par exemple :

Code :

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

1
2
3
4
5
6
SELECT  * FROM 
    (SELECT  * FROM tableCourante union SELECT  * FROM tableArchive) A
    WHERE note= 
       (SELECT max(note) FROM 
           (SELECT  * FROM tableCourante UNION SELECT  * FROM tableArchive) B
       )

Doit effectivement être corrigée par :

Code :

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

1
2
3
4
5
6
SELECT  * FROM 
    (SELECT  * FROM tableCourante union SELECT  * FROM tableArchive) A
    WHERE note= 
       (SELECT max(note) FROM 
           (SELECT  note FROM tableCourante UNION SELECT  note FROM tableArchive) B
       )

Par contre, dans un forum, je trouve que remplacer les étoiles restantes par le nom des colonnes que l'on souhaite récupérer alourdirait la requête et donc la compréhension du problème initial.
De plus, dans mon cas particulier, je "map" les enregistrements dans des objets. Comme je ne sais pas à l'avance ce que l'application va utiliser, il faut tout récupérer...
Accessoirement ça me permet aussi de vérifier que la structure d'une table est identique à celle d'un objet.