Sélection avec gestion de niveau de priorité

**Aklain52** · 28/04/2020, 11h42

Bonjour à tous, je bute sur une requête SQL un peu complexe.

Pour l'instant, j'ai une requête très simple :

Code :

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

1
2
3
4
5
 
SELECT `Contenu`, `Type`, `Signature`
FROM `ressources`
WHERE `date`<= NOW()
AND `dateValidite` > NOW()

Il s'avère que ma table possède un champ intitulé "Priorité" qui contient un niveau de priorité de 1 à 4.

Ce que je voudrais via SQL c'est, dans les lignes sélectionnées par ma requête :

Si il y a des lignes contenant la priorité à 1 -> ne sélectionner que les éléments contenant la priorité à 1.
Si il y a des lignes contenant la priorité à 2 et/ou 3 (mais pas de 1) -> Sélectionner tous les éléments contenant la priorité 2 et 3 (exclusion du niveau de priorité 4).
Si il n'y a que des lignes contenant la priorité à 4 -> Sélectionner tout le monde.

Je me doute qu'il faut que j'utilise du CASE ou/et du EXISTS mais je n'arrive pas à formuler correctement ma requête.
Si une âme charitable pouvait me donner un coup de pouce...

PS : Je peux faire le tri via PHP mais je pense que ce serait bien plus rapide à l’exécution et instructif via SQL.

**tatayo** · 28/04/2020, 11h52

Bonjour,
Si tu reformules ton besoin, tu t'approche de la solution.
Tu veux:

Toutes les lignes de priorité 1
Toutes les lignes de priorité 2/3, pour lesquelles il n'en existe pas avec la priorité 1
Toutes les lignes de priorité 4, pour lesquelles il n'en existe pas avec une priorité inférieure à 4

Tu peux donc faire une union des 3 requêtes, et pour le test de non existence tu as plusieurs solution (not exists, jointure externe).

Tatayo.

**Aklain52** · 28/04/2020, 12h00

@tatayo

Merci, mais c'est plutot :
Si 1 existe, je ne sélectionne que 1.
Si 2 ou 3 existe, je sélectionne que 2 et 3.
Sinon je sélectionne 4.

En attendant, j'ai réussi à formuler ce code qui fonctionne :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
SELECT `Contenu`, `Type`, `Signature`, `Priorité`
FROM `ressources`
WHERE `date`<= NOW()
AND `dateValidite` > NOW()
AND (CASE
   WHEN EXISTS(SELECT * FROM `ressources` WHERE `date`<= NOW() AND `dateValidite` > NOW() AND `Priorité`=1) THEN `Priorité`=1
   WHEN EXISTS(SELECT * FROM `ressources` WHERE `date`<= NOW() AND `dateValidite` > NOW() AND `Priorité`=2 OR `Priorité`=3) THEN `Priorité`=2 OR `Priorité`=3
   ELSE True
END)

Il y a peut-être moyen d'optimiser.

**bib34690** · 28/04/2020, 12h10

Pourquoi ne pas faire un petit select pour recup PrioriteMin = MIN(Priorité)
puis ta requête qui filtre sur priorité avec un case PrioriteMin

**Aklain52** · 28/04/2020, 12h27

Envoyé par bib34690

Pourquoi ne pas faire un petit select pour recup PrioriteMin = MIN(Priorité)
puis ta requête qui filtre sur priorité avec un case PrioriteMin

J'ai fait 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
 
SELECT @PrioriteMin := MIN(`Priorité`) FROM `ressources`
WHERE `date`<= NOW()
AND `dateValidite` > NOW() OR `dateValidite`=Null;
 
SELECT `Contenu`, `Type`, `Signature`, `Priorité`
FROM `ressources`
WHERE `date`<= NOW()
AND `dateValidite` > NOW() OR `dateValidite`=Null
AND (CASE
    WHEN @PrioriteMin = 1 THEN `Priorité`=1
    WHEN @PrioriteMin <= 3 THEN `Priorité`=2 OR `Priorité`=3
    ELSE True
END)

Mais du coup, je récupère aussi la valeur de @PrioriteMin dans mon résultat.. ce qui ne m'est pas très utile.
Le code est plus court et il y a moins de répétition, c'est déjà ça.

**tatayo** · 28/04/2020, 13h44

Et pourtant ma requête répond bien au besoin.
La première renvois toutes les lignes avec pour priorité 1, donc elle ignore les autres lignes => Toutes les lignes de priorité 1.
La requête 2 renvoie les lignes de priorité 2 ou 3, s'il n'existe pas de ligne avec la priorité 1. Elle ignore donc les lignes avec la priorité 4, et ne renvoie rien s'il existe des lignes de priorité 1 => Toutes les lignes de priorité 2/3, pour lesquelles il n'en existe pas avec la priorité 1
La dernière renvoie les lignes de priorité 4, s'il n'en existe pas de priorité inférieur => Toutes les lignes de priorité 4, pour lesquelles il n'en existe pas avec une priorité inférieure à 4

Il faut faire 3 requêtes distincts, et faire une union des 3.

Je ne vois pas ce qui coince avec ma proposition.

Tatayo.

**vanagreg** · 28/04/2020, 13h53

Bonjour,

Pas possible d'utiliser des fonctions analytiques?

**Aklain52** · 28/04/2020, 15h21

Envoyé par tatayo

Et pourtant ma requête répond bien au besoin.
La première renvois toutes les lignes avec pour priorité 1, donc elle ignore les autres lignes => Toutes les lignes de priorité 1.
La requête 2 renvoie les lignes de priorité 2 ou 3, s'il n'existe pas de ligne avec la priorité 1. Elle ignore donc les lignes avec la priorité 4, et ne renvoie rien s'il existe des lignes de priorité 1 => Toutes les lignes de priorité 2/3, pour lesquelles il n'en existe pas avec la priorité 1
La dernière renvoie les lignes de priorité 4, s'il n'en existe pas de priorité inférieur => Toutes les lignes de priorité 4, pour lesquelles il n'en existe pas avec une priorité inférieure à 4

Il faut faire 3 requêtes distincts, et faire une union des 3.

Je ne vois pas ce qui coince avec ma proposition.

Tatayo.

D'abord, merci de tenter de m'aider.

Ce que je souhaite c'est n'avoir qu'une seule requête, pour des question de lisibilité et de facilité de maintenir mon code.
Ensuite, si la priorité 1 existe, je souhaite m'arrêter là et ne sélectionner que les lignes possédant cette priorité.

Si il n'y a pas de priorité 1 alors, si la priorité 2 ou 3 existe, je sélectionne les lignes de cette priorité et je m'arrête.

Enfin, si il n'y a ni priorité 1, 2 ou 3, je sélectionne la priorité 4.

Mon code fonctionne très bien.
Mais il y a des répétitions et ça ne me plait pas. Rapport au DRY.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
SELECT `Contenu`, `Type`, `Signature`, `Priorite`
FROM `ressources`
WHERE `date`<= NOW()
AND `dateValidite` > NOW() OR `dateValidite`=Null
AND (CASE
    WHEN EXISTS(SELECT * FROM `ressources` WHERE `date`<= NOW() AND `dateValidite` > NOW() OR `dateValidite`=Null AND `Priorite`=1) THEN `Priorite`=1
    WHEN EXISTS(SELECT * FROM `ressources` WHERE `date`<= NOW() AND `dateValidite` > NOW() OR `dateValidite`=Null AND `Priorite`=2 OR `Priorite`=3) THEN `Priorite`=2 OR `Priorite`=3
    ELSE True
END)

@vanagreg
Quelle fonction analytique te paraitrait intéressante ?

@tous
Si ce n'est pas possible de condenser ce code en évitant les répétitions, je le générerais en PHP pour éviter d'avoir à le modifier en plusieurs endroits...

**tatayo** · 28/04/2020, 15h53

Mais il s'agit d'une union de 3 requêtes, ce qui fait bien une seule requête au final.
Pour la priorité 1, c'est exactement ce que fait ma requête:

Code sql :

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
 
select LaTable.id,1
	from LaTable
	where priorite = 1
union
select LaTable.id,23
	from LaTable 
	left outer join LaTable as t2 
		on t2.id = LaTable.id and t2.priorite = 1
	where LaTable.priorite in (2,3) and  t2.id = null
union
select LaTable.id,4 
	from LaTable
	left outer join LaTable as t2 
		on t2.id = LaTable.id and t2.priorite < 4
	where LaTable.priorite = 4 and t2.id = null

Tu obtiens bien le bon résultat.
Tu peux ajouter les colonnes qui te manques dans chaque SELECT.

Bref pourquoi vouloir partir sur une requête compliquée si tu peux avoir le bon résultat avec une requête plus simple ?

Tatayo.

**aieeeuuuuu** · 28/04/2020, 16h26

Envoyé par vanagreg

Bonjour,

Pas possible d'utiliser des fonctions analytiques?

Si !

Un truc dans ce genre là

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
WITH CTE AS (
	SELECT 
			`Contenu`
		,	`Type`
		,	`Signature`
		,	RANK() OVER(ORDER BY CASE priorite
						WHEN 1 THEN 1
						WHEN 2 THEN 2
						WHEN 3 THEN 2
						WHEN 4 THEN 4
					END ) AS RK
	FROM `ressources`
	WHERE `date`<= NOW()
	AND `dateValidite` > NOW()
)
SELECT 
			`Contenu`
		,	`Type`
		,	`Signature`
FROM CTE
WHERE RK = 1

**bib34690** · 28/04/2020, 16h50

Envoyé par Aklain52

J'ai fait 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
 
SELECT @PrioriteMin := MIN(`Priorité`) FROM `ressources`
WHERE `date`<= NOW()
AND `dateValidite` > NOW() OR `dateValidite`=Null;
 
SELECT `Contenu`, `Type`, `Signature`, `Priorité`
FROM `ressources`
WHERE `date`<= NOW()
AND `dateValidite` > NOW() OR `dateValidite`=Null
AND (CASE
    WHEN @PrioriteMin = 1 THEN `Priorité`=1
    WHEN @PrioriteMin <= 3 THEN `Priorité`=2 OR `Priorité`=3
    ELSE True
END)

Mais du coup, je récupère aussi la valeur de @PrioriteMin dans mon résultat.. ce qui ne m'est pas très utile.
Le code est plus court et il y a moins de répétition, c'est déjà ça.

Le 1er select doit être fait ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
 
DECLARE @PrioriteMin TinyInt;
        SET @PrioriteMin = 
        (
           SELECT MIN(`Priorité`) 
           FROM `ressources`
           WHERE `date`<= NOW()
                   AND `dateValidite` > NOW() OR `dateValidite`=Null;
        )

**Aklain52** · 28/04/2020, 17h01

Merci à tous,

Quand je vois ce que vous m'envoyez, je me dit que je suis encore un gros débutant en SQL...

C'est le défaut de gérer de multiples code, on ne devient jamais vraiment bon dans l'un d'eux.

En tout cas, encore merci.

Je vais tester un peu les performances des solutions de @aieeeuuuuu et @tatayo pour garder le plus rapide et je passe ce sujet en résolu.

Sélection avec gestion de niveau de priorité

Langage SQL

Discussions similaires

Partager

Partager