Discussion :

[bstevy]

Bonjour à tous,


Voilà, je bloque sur une requête que j'aimerai simplifier, mais je n'arrive pas à bien m'y prendre.
Est ce que vous pourriez m'aider ?



Voici la requete initiale.

Code :

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

1
2
3
4
5
6
7
8
SELECT *
  FROM contract_versions CV
WHERE 1 = 1
   AND CV.cv_crc_code NOT IN (12, 13, 14, 15, 16, 102)
   AND cv_number = (SELECT MAX (cv_number)
             FROM contract_versions
            WHERE NOT (cv_status_code = '09' OR cv_status_code = '9')
              AND cv_con_number = CV.cv_con_number)

Comme vous le voyez, on lit deux fois la table contract_versions
et donc ce que je cherche à faire, c'est obtenir le même résultat en un seul passage.
Je suis sur que c'est faisable très facilement, mais j'arrive pas à m'en sortir avec les deux conditions :

D'un point de vue fonctionnel, ca donne cela
D'un coté, j'ai la liste des contrats valides : CV.cv_crc_code NOT IN (12, 13, 14, 15, 16, 102)
De l'autre, j'ai la liste des versions valides : NOT (cv_status_code = '09' OR cv_status_code = '9')
Et je cherche donc les infos (select *) pour la dernière version valide à laquelle le contrat est valide. (je ne ramène pas les contrats qui sont invalidés par leur dernière version)
->Un contract a pu par exemple ête invalidé par une version N, mais si cette version est justement invalidée elle aussi, alors le contrat reste valide.

Vous auriez une idée de comment je pourrais faire ca en évitant d'appeler deux fois la table ?


Pour info, l'idée sur laquelle je partais à la base était d'utiliser une fonction rank pour determiner ma ligne de version max.
Car après, j'ai besoin aussi des info de la version 1, donc le rank m'aurait permi d'avoir une condition du genre "rank = 1 (version max) OR version = 1"
Mais pour le moment, j'ai une couille dans le paté que j'arrive pas à enlever. C'est pas trop loin de ce que je cherche à faire, mais c'est toujours pas ça.

Code :

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

1
2
3
4
5
6
7
select	cv_con_number,  
	cv_number, 
	rank() over (partition by cv_con_number order by decode(cv_status_code, '09', 0, '9', 0, cv_number) desc),
	CV.*
FROM contract_versions CV
WHERE 1 = 1
AND CV.cv_crc_code NOT IN (12, 13, 14, 15, 16, 102)

Merci d'avance pour vos idées.


Steven




EDIT : Et je cherche donc les infos (select *) pour la dernière version valide à laquelle le contrat est valide. (je ne ramène pas les contrats qui sont invalidés par leur dernière version)
Je pense que c'est plus clair comme ca.

[bstevy]

Bon, j'annonce qu'a force d'en discuter, j'ai réussi à trouver une solution par moi même.
Néanmoins, j'aimerai bien avoir votre avis et connaitre votre facon de faire.
Et si personne n'a d'idée, je mettrais en résolu avec ma solution demain

[mnitu]

Cherchez sur ce forum First/Last dans leur version agrégée.

[bstevy]

Tu l'as mise en raccourci cette réponse ? genre ctrl+r, ca l'ecrit automatiquement ?

Merci, mais c'est pas du tout la réponse que j'attends... j'ai un problème sur la transformation de mes contions, pas sur la méthode de faire un group by -_-
si t'avais regardé ma deuxième requete, tu aurais peut etre vu qu'il y a qlq chose qui va pas avec les contions que j'utilise, et tu aurais pu m'aider



Au passage, pour aider ceux qui ont un problème avec leur first ... tu devrais mettre un lien vers un message ou y'a l'explication plutot que de demander une recherche qui renvoit sur tes demandes recherches ...

[mnitu]

Tu l'as mise en raccourci cette réponse ? genre ctrl+r, ca l'ecrit automatiquement ?

Merci, mais c'est pas du tout la réponse que j'attends... j'ai un problème sur la transformation de mes contions, pas sur la méthode de faire un group by -_-
si t'avais regardé ma deuxième requete, tu aurais peut etre vu qu'il y a qlq chose qui va pas avec les contions que j'utilise, et tu aurais pu m'aider



Au passage, pour aider ceux qui ont un problème avec leur first ... tu devrais mettre un lien vers un message ou y'a l'explication plutot que de demander une recherche qui renvoit sur tes demandes recherches ...

Et la solution de mnitu me convenait pas non plus car elle fait un group by, donc elle ne m'enlève pas les contrats dont je veux me debarasser...

Mais c'est quand même celle que j'ai prise

Bref c'est juste pour vous dire que j'avais déjà pris la décision de ne plus répondre à aucune de vos interventions. L'exception que j'avais fait ce matin c'était une erreur de ma part.
Bon courage pour la suite

[bstevy]

Bref c'est juste pour vous dire que j'avais déjà pris la décision de ne plus répondre à aucune de vos interventions. L'exception que j'avais fait ce matin c'était une erreur de ma part.
Bon courage pour la suite

Vous êtes un gentil, et vous êtes pas trop rancunié, c'est pas de votre faute si vous vous sentez obligé d'aider même ceux que vous n'aimez pas

Le problème que j'avais était sur l'utilisation de mes filtres, non pas sur le type de group by que je devais utiliser.
C'est cette partie que j'avais mal faite :

Code :

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

having max(CV_CRC_CODE) keep (dense_rank first order by CV_NUMBER desc) not IN (12, 13, 14, 15, 16, 102)

J'avais tendance à traiter ce cas dans mon max (ou rank) par un genre de case, ou une connerie dans le genre
J'avoue que je bloquais sur un problème de débutant, et c'est en expliquant mon problème à un collègue que j'ai trouvé ma réponse.

[bstevy]

Cherchez sur ce forum First/Last dans leur version agrégée.

Juste pour confirmer un truc quand même, ce que tu appelles FIRST/LAST en version agrégée, c'est ca :

Code :

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

min(id) keep(dense_rank first order by kind, id) id

?

[Waldar]

Je l'aurai écrit 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
with cte_max_cv_number as
(
  select cv_con_number
       , max(cv_number) as cv_number_max
    from contract_versions
   where cv_status_code not in ('09', '9')
group by cv_con_number
)
select cv.*
  from contract_versions cv
  join cte_max_cv_number cm
    on cm.cv_con_number = cv.cv_con_number
   and cm.cv_number_max = cv.cv_number
 where cv.cv_crc_code not in (12, 13, 14, 15, 16, 102);

[bstevy]

Je l'aurai écrit comme ceci :

ca donne juste le même plan que ce que j'ai avant... avec double lecture de la table, ce que je chercher à éviter.

Mais merci quand meme. La requete est jolie

Maintenant, ce qu'il faut que je test, c'est est ce qu'on peut faire du cte de cte...

[skuatamad]

Juste pour confirmer un truc quand même, ce que tu appelles FIRST/LAST en version agrégée, c'est ca :

Code :

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

min(id) keep(dense_rank first order by kind, id) id

?

Oui, un lien vers un exemple :
http://lalystar.developpez.com/fonct...tiques/#LIII-I

[bstevy]

Oui, un lien vers un exemple :
http://lalystar.developpez.com/fonct...tiques/#LIII-I

C'est gentil, mais comme je le disais, je sais comment utiliser cette fonction
C'est pas un problème de group by que j'ai XD