Discussion :

[Puppet_Master]

Bonjour.

J'ai besoin de collecter les 'id' de 'ma_table' sauf le dernier de chaque sous-ensemble d'un autre champ. Avec le code qui suit, cela devrait être plus clair:

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
create table ma_table (
id serial,
val char(1)
)
 
insert into ma_table ( val )
values 
('A'),
('B'),
('A'),
('B'),
('B'),
('A'),
('C'),
('C')
 
select * from ma_table

Retourne:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 id | val
----+-----
  1 | A
  2 | B
  3 | A
  4 | B
  5 | B
  6 | A
  7 | C
  8 | C
(8 lignes)

Donc je veux récupérer les id (1,2,3,4 et 7).

J'ai trouvé une solution, mais j'ai peur d'avoir fait trop compliqué, et comme cette solution doit intégrer un ensemble plus complexe, j'aimerai la simplifier au maximum pour ne pas accroitre inutilement la complexité du projet final.

Voici la solution:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
select B.id
from (
	select *
	     , max(A.rank) over ( partition by A.val ) as rank_max
	from ( select *
		    , rank() over ( partition by val order by id ) as rank 
		 from ma_table 
	     ) as A
     ) AS B
where not B.rank_max = B.rank

Les critiques ou autres solutions sont les bienvenus

[ced]

Bonjour,

Une écriture "plus simple" à mon sens (la notion de simplicité est toute relative) :

Code :

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

1
2
3
4
5
6
7
select id
from ma_table
except
select max(id) 
from ma_table
group by val
order by id;

Après, il faut aussi comparer les performances des différentes écritures...

@+

[Puppet_Master]

Bonjour ced.

Merci pour votre réponse.

Je trouve votre solution plus élégante que la mienne, car:

1. pas de sous-requêtes ni de fonction de fenêtrage
2. pas de création de nouvelles donnée pour répondre au problème
3. plus simple à lire

Je vois difficilement comment avoir une réponse plus simple a mon problème ( avis aux amateurs )

Je prends votre solution (encore merci) et je coche résolu !

[Waldar]

Un ajout à la solution de ced, faites un EXCEPT ALL au lieu d'un EXCEPT, vous économiserez un distinct.

La solution avec les fonctions de fenêtrage n'est pas à jeter à la poubelle, mais vous l'avez rédigée plus compliquée que nécessaire.
Une version plus simple serait :

Code :

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

1
2
3
4
5
6
7
8
with cte_matable (id, val, rn) as
(
select id, val, row_number() over(partition by val order by id desc)
  from matable
)
select id, val
  from cte_matable
 where rn > 1;

[Puppet_Master]

Bonjour Waldar.

Merci de votre réponse que je trouve maligne.

Par contre je me demande quel avantage y a-t-il à utiliser un 'with' plutôt qu'une sous-requête comme suit:

Code :

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

1
2
3
select A.id
  from ( select id, val, row_number() over(partition by val order by id desc) AS rn from ma_table ) as A
where A.rn > 1

Juste une question de lisibilité ou y a il un autre avantage ?

[Waldar]

C'est en effet plus lisible ; notamment quand plusieurs sous-requêtes s'imbriquent.
Mais surtout c'est réutilisable plusieurs fois - et construit une seule fois, encore que sur pg je n'en suis pas certain.

[ced]

Surtout, par rapport à ma requête, la requête avec CTE présente l'avantage (en termes de performances) de ne parcourir qu'une seule fois ma_table.
C'est donc plus performant dans le cas présent.

Les CTE sont effectivement utilisables plusieurs fois dans une même requête sous PostgreSQL.