Discussion :

[Bichette001]

Bonjour

j'ai besoin d'additionner les lignes de 2 requetes de la manière suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
select * from D_offre_com o
left join d_contrat_permanent p
on p.numofr_init = o.numofr or p.numofr_der = o.numofr
where (
(p.datsor is null
and o.codsitofr is null
or o.codsitofr in ('54','57','59'))
OR
(o.datstaofr <= add_months('05/01/2023',36)
and p.datsor is not null))
+
select *from D_offre_com where o.numofr not in (p.numofr_init union p.numofr_der)

je ne sais pas comment on peut faire pour additionner la 2è partie après le + ?

merci ;-)

[tatayo]

Bonjour,
qu'est-ce que tu entends par "additionner" ?
Si tu veux que le résultat contienne les lignes des deux requêtes, il faut utilise une UNION:

Code sql :

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

1
2
3
4
 
select ...
UNION
select ...

Et si tu ne veux pas supprimer les doublons, un UNION ALL sera (un peu) plus performant si les deux requêtes renvoient beaucoup de lignes.
Par contre il faut remplacer les SELECT * par la liste des colonnes à renvoyer dans les deux requêtes, et que les deux listes soient cohérentes. Dans le cas contraire l'union ne fonctionnera pas.

Si tu veux une addition au sens mathématique, il nous faut plus d'infos (le schéma des tables, un jeu de test et le résultat attendu).

Tatayo.

[escartefigue]

Attention aussi aux combinaisons de AND et OR sans parenthèses : le résultat obtenu ne sera pas forcément celui attendu.
Ne serait-ce que pour des raisons de lisibilité et de maintenance, il est préférable d'ajouter des parenthèses.

[vanagreg]

OMG, une date passée en tant que chaines de caractères
Quand tu passes une valeur date, il faut utiliser TO_DATE pour éviter une conversion implicite.

Dans ton exemple, si le format nls date par défaut est mm/dd/yyyy, alors il s'agira du 1er mai 2023. Donc si tu veux que ce soit le 5 janvier 2023, précise-le explicitement:

Code :

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

add_months(to_date('05/01/2023', 'dd/mm/yyyy'),36)

Sinon pour ton besoin, j'ai l'impression que tu peux juste ajouter une condition.

[Bichette001]

Merci pour vos réponses

oui c'était pour ajouter les lignes


par contre, je me rends compte que ma requête tourne très longtemps
est-ce qu'on peut écrire cette requête de manière plus performante d'après vous ?
rappel ci-dessous de la requête où j'ai modifié un OR par un AND correspondant à ce que je cherche

Code :

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

1
2
3
4
5
6
7
8
9
10
11
select * from D_offre_com o
left join d_contrat_permanent p
on p.numofr_init = o.numofr or p.numofr_der = o.numofr
where (
(p.datsor is null
and o.codsitofr is null
or o.codsitofr in ('54','57','59'))
and 
(o.datstaofr <= add_months('05/01/2023',36)
and p.datsor is not null)
);

[tatayo]

Sans connaitre la structure des tables, les éventuels indexes, sans le plan d'exécution, il est difficile pour nous de répondre.

Tatayo.

[Bichette001]

comment svp utilise-t-on les indexes ?

[tatayo]

Tu as toutes les infos ici.

Tatayo

[Bichette001]

merci
j'ai compris que ça accélérait mais pas comment je l'écrit en requête

[tatayo]

Bonjour,
On n'utilise pas un index dans une requête, c'est le moteur (ici Oracle) qui va décider ou pas d'utiliser tel ou tel index, en fonction de la requête (colonnes renvoyées, jointures, colonnes filtrées, type de filtre...).
Ici comme on ne connait ni la structure des tables, ni les indexes existants, on ne peut pas te dire comment optimiser ta requête.

Accessoirement comme l'indiquait escartefigue, tu as une combinaison de AND et de OR sans parenthèse, donc il existe un risque que le résultat renvoyé ne soit pas celui attendu:

Code sql :

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

1
2
3
4
 
p.datsor is null
and o.codsitofr is null
or o.codsitofr in ('54','57','59')

Est-ce qu'il faut compendre:

Code sql :

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

1
2
3
4
 
(p.datsor is null
and o.codsitofr is null)
or o.codsitofr in ('54','57','59')

Ou:

Code :

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

1
2
3
4
 
p.datsor is null
and
(o.codsitofr is null or o.codsitofr in ('54','57','59'))

Tatayo.

[Bichette001]

bonjour
oui merci j'ai bien mis les () comme suit de mon côté

Code :

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

1
2
3
p.datsor is null
and
(o.codsitofr is null or o.codsitofr in ('54','57','59'))

index de la d_offre_com :
REF IDX_REF_D_OFFRE_COM Unique TRAORI, NUMOFR, DEBVLE N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 169600k next 1m minextents 1 maxextents unlimited )
REF PK_REF_D_OFFRE_COM Unique IDTOFR N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 26432k next 1m minextents 1 maxextents unlimited )
REF PK_REF_D_OFFRE_COM_NUMCON Normal NUMCON N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 34m next 1m minextents 1 maxextents unlimited )

et d_contrat_permanent :

REF IDX_CONTP_1 Normal NUMCON, FAMIFP, IDTPRT, DATTRM N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 370240k next 1m minextents 1 maxextents unlimited )
REF IDX_CONTP_2 Normal NUMCON, IDTPRT, FAMIFP N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 344000k next 1m minextents 1 maxextents unlimited )
REF IDX_CONTP_3 Normal NUMCON, FAMIFP, IDTPRT, DATEFF N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 424m next 1m minextents 1 maxextents unlimited )
REF IDX_NUMAGTREA Normal NUMAGT_AGTREA, FLG_GENERENVCLI N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 240m next 1m minextents 1 maxextents unlimited )
REF PK_REF_D_CONTRAT_PERMANENT Unique NUMCON N N tablespace ref_indx pctfree 10 initrans 2 maxtrans 255 storage ( initial 256960k next 1m minextents 1 maxextents unlimited )

[tatayo]

Du coup je me demande bien comment ta requête peut renvoyer des données:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
select * from D_offre_com o
left join d_contrat_permanent p
on p.numofr_init = o.numofr or p.numofr_der = o.numofr
where (
(p.datsor is null
and (o.codsitofr is null
or o.codsitofr in ('54','57','59')))
and 
(o.datstaofr <= add_months('05/01/2023',36)
and p.datsor is not null)
);

Si je supprime les parenthèses "inutiles" du point de vue logique booléenne ( A and (B and C) équivaut à A AND B AND C et a (A and B) and C) et que je change l'ordre des filtres (le AND est commutatif), on a la restriction suivante:

Code sql :

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

1
2
3
4
5
6
7
8
9
10
 
select * from D_offre_com o
left join d_contrat_permanent p
on p.numofr_init = o.numofr or p.numofr_der = o.numofr
where (
p.datsor is null
and p.datsor is not null
and o.datstaofr <= add_months('05/01/2023',36)
and (o.codsitofr is null
or o.codsitofr in ('54','57','59'))

p.datsor doit être null et non null en même temps, ça ne peut pas fonctionner !

Tatayo.

[JeitEmgie]

A première vue la condition du WHERE telle qu'écrite est équivalente à

Code :

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

1
2
3
o.codsitofr in ('54','57','59')
and o.datstaofr <= add_months('05/01/2023',36)
and p.datsor is not null

changer la première partie en

Code :

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

1
2
3
((p.datsor is null
and o.codsitofr is null)
or o.codsitofr in ('54','57','59'))

ne changerait donc rien du tout.

[tatayo]

Je viens de voir la description des indexes:

• Pour la table D_offre_com, aucun n'a comme première colonne codsitofr ou datstaofr qui sont utilisées comme filtre ou numofr utilisée pour la jointure.
• Pour la table d_contrat_permanent, aucun n'a comme première colonne numofr_der, qui est utilisé pour la jointure

Donc aucun index n'est utilisable ici, que ce soit pour filtrer les tables ou pour la jointure.
De fait Oracle est obligé de lire toutes les lignes des deux tables pour retrouver ses petits.
Enfin un SELECT * empêche d'utiliser un éventuel index couvrant.

Je ne connais pas trop Oracle, mais tu devrais regarder le plan d'exécution de la requête, peut-être qu'il t'indiquera quel indexes devraient être ajoutés sur les deux tables pour améliorer le temps d'exécution de la requête.

Tatayo.

[escartefigue]

@Bichette001

Comme je le disais plus haut, en l'absence de parenthèses, les combinaisons d'opérateurs AND et OR ne produisent pas toujours les résultats attendus, dans votre nouvelle réponse n° 5, on a toujours ce même problème.

A and B or C

Ne produit pas le même résultat que

A and (B or C)

[Bichette001]

merci pour ta remarque

pour les indexes si les champs que tu cites avaient été des index=> pas de modif pour le script de ma requête , c'est ce que tu dis ?
où peut-on trouver le plan d'exécution de requête?

pour les (), si je mets A and (B or C) sous-entend A et (codsitofr est null ou = 54 ou = 57 ou = 59)
tu l'écrirais comment ?

[tatayo]

Pour le plan d'exécution, une recherche rapide dans Google m'a donné ceci.

Concernant l'écriture de la requête (parenthèses et autre), sans jeu de test et résultat attendu, la seule réponse que tu peux avoir est: aucune idée.

Tatayo.

[escartefigue]

Cette condition qui présente des parenthèses inutiles alors que d'autres parenthèses manquent :

Code SQL :

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

1
2
3
4
5
6
7
8
where (    (    p.datsor is null
            and o.codsitofr is null
            or o.codsitofr in ('54','57','59'))
            and 
           ( o.datstaofr <= add_months('05/01/2023',36)
              and p.datsor is not null)
      )
;

est à remplacer par :

Code :

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

1
2
3
4
5
6
7
where p.datsor is null
  and (    o.codsitofr is null
       or  o.codsitofr in ('54','57','59')
      )
  and o.datstaofr <= add_months('05/01/2023',36)
  and p.datsor is not null
;

Le plan d'exécution peut être estimé grâce à un EXPLAIN ou vérifié avec DBMS_XPLAN.
Attention : une estimation (EXPLAIN) ne vaut que pour la plate-forme d'exécution sur laquelle elle est calculée. Comme le calcul se base essentiellement sur les statistiques et les index, deux plates formes différentes peuvent produire des estimations différentes (les index installés, les cardinalités, les dates de dernières réorg... peuvent être différents entre par exemple une production et un environnement de recette).

[Bichette001]

le explain semble intéressant
je dois l'écrire comment ?

Code :

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

 explain select * from d_offre_com;

ou

Code :

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

 explain plan select * from d_offre_com;

ces 2 scripts ne fonctionnent pas ..🤨

[JeitEmgie]

Code :

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

1
2
3
4
5
explain plan for
select ...
;
 
SELECT * FROM table(DBMS_XPLAN.DISPLAY);

ou

Code :

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

1
2
3
 
select * from dbms_xplan.display_cursor(null, null, 'ALLSTATS LAST') ;
-- il faut le "SELECT privilege on V$SESSION"