[REQUETE][IMBRIQUEE] Bien ou pas bien?

**fabszn** · 18/04/2005, 10h50

Bonjour,

Je suis en train d'ecrire un batch permettant de faire des mise à jour sur une base de donnée.

Je dois sélectionner des dossiers selon certains critères. Sur chaque dossier ramené, je dois mettre à jour des champs...

Est ce que l'utilisation de requetes imbriquées est recommandée?

c'est à dire :

Code :

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

1
2
3
 
 
update T_MATABLE m set m.MONCHAMP where 'condition de selection'  and exits(select ... from ... where ...);

Est ce que cette requete vous sembles absurde?

Merci d'avance,

@+

Fabszn

**trotters213** · 18/04/2005, 11h24

bè non je vois pas où il peut y avoir un problème, c'est le but du UPDATE, seulement soit sur de ce que te renvoit ta sous-requête sinon ça risque de foutre un brave bordel dans ta base

Perso j'aurais fait comme ça en tout cas, c'est le plus simple à mes yeux.

**BenoitDenis** · 18/04/2005, 11h28

entierement d'accord avec trotter

execute ta sous requete seul poour voir si le résultat renvoyé et bien celui que tu recherche et si oui, a toi les updates!!

**fabszn** · 18/04/2005, 11h29

Merci pour ta réponse,

Envoyé par trotters213

bè non je vois pas où il peut y avoir un problème, c'est le but du UPDATE, seulement soit sur de ce que te renvoit ta sous-requête sinon ça risque de foutre un brave bordel dans ta base

Perso j'aurais fait comme ça en tout cas, c'est le plus simple à mes yeux.

Aussi, quand tu dis qu'il faut faire attention à ce que ramene la sous requete, quelles genre de precaution je dois prendre?

Merci d'avance,

@+

Fabszn

**BenoitDenis** · 18/04/2005, 11h30

entierement d'accord avec trotter

execute ta sous requete seul poour voir si le résultat renvoyé et bien celui que tu recherche et si oui, a toi les updates!!

**annedjomo** · 18/04/2005, 11h31

salut.

Perso je t'aurai proposé ceci

Code :

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

update T_MATABLE m set m.MONCHAMP = Valeur where 'condition de selection'  and exits(select ... from ... where ...);

**fabszn** · 18/04/2005, 11h34

Envoyé par annedjomo

salut.

Perso je t'aurai proposé ceci

Code :

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

update T_MATABLE m set m.MONCHAMP = Valeur where 'condition de selection'  and exits(select ... from ... where ...);

Oui... j'avais oublier en ecrivant la requete sur le forum la valeur de mis à jour...

@+

Fabszn

**Jaouad** · 18/04/2005, 11h41

Envoyé par trotters213

bè non je vois pas où il peut y avoir un problème, c'est le but du UPDATE, seulement soit sur de ce que te renvoit ta sous-requête sinon ça risque de foutre un brave bordel dans ta base

Perso j'aurais fait comme ça en tout cas, c'est le plus simple à mes yeux.

j'ajouterai à cela de ne pas oublier de vérifier que ta sus reqûete est bien optimisé ( notamment qu'il a un index dessus ...) afin de ne pas dégrader les preformances de ton update .

**trotters213** · 18/04/2005, 11h54

Envoyé par jaouad

j'ajouterai à cela de ne pas oublier de vérifier que ta sus reqûete est bien optimisé ( notamment qu'il a un index dessus ...) afin de ne pas dégrader les preformances de ton update .

+1.
Sinon ça va ramer comme il faut, tu peux nous la montrer ta requête en entier pour voir.

Envoyé par fabszn

Aussi, quand tu dis qu'il faut faire attention à ce que ramene la sous requete, quelles genre de precaution je dois prendre?

Fais attention qu'elle te renvoit toujours ce que tu veux exactement et non pas faire un test en ce disant "c'est bon", soit sur de ta requête sinon ça va rien faire ou pire, te modifier des champs que tu ne voulais pas touché.

**fabszn** · 18/04/2005, 12h02

Hello,

Voila la requete :

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
 
update tds_ctu ctus                                            
	set	ctus.CDDET      = 'NN',                                 
        	ctus.DAMAJ      = sysdate                               
                where  ctus.ANCAMP 	= ?1                                    
                and    ctus.NUMSESSION = ?2                                    
                and    ctus.CDSUI 		= 'RE'                          
                and    ctus.CDDET 		= 'CL'                          
                and    ctus.CODREC 	= '6'                                   
                and    not exists (select * from   td_pst pst_           
                        	where  pst_.ANCAMP		 = ?1               
                        		and	   pst_.NUMSESSION	 = ?2               
                        		and	   pst_.CODREC		 = '6'              
                        		and	   pst_.NUMETAB		 = ctus.NUMETAB     
                        		and	   pst_.NUMEMP		 = ctus.NUMEMP      
                        		and	   pst_.NUMECAN_NOMM = ctus.NUMECAN)    
		and    not exists (select * from td_pst pst__       
                        		where pst__.ANCAMP		= ?1                
					and	  pst__.NUMSESSION	= ?2                
                        		and	  pst__.NUMECAN_NOMM = ctus.NUMECAN)

Je ne sais pas si vous avez besoin de détail fonctionnels?

Merci encore pour votre aide!!

@+

Fabszn

**Jaouad** · 18/04/2005, 12h08

quel SGBDR et quel version ?

**fabszn** · 18/04/2005, 12h15

oupppss!! desolé j'ai oublié...

Oracle 8i

**Jaouad** · 18/04/2005, 13h57

Envoyé par fabszn

oupppss!! desolé j'ai oublié...

Oracle 8i

et que donne le plan d'execution

**trotters213** · 18/04/2005, 14h14

2 trucs :

si ça c'est une jointure :

Code :

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

pst_.NUMETAB = ctus.NUMETAB

c'est pas bon. C'est jamais dans le WHERE les jointures mais dans le FROM avec un opérateur normalisé JOIN :

Code :

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

SELECT * FROM td_pst pst_ INNER JOIN tds_ctu ctus ON pst_.NUMETAB = ctus.NUMETAB

au risque de passer pour un con (j'ai jamais vu Oracle) c'est quoi ?1, ?2, ... bref le ? dans tes comparaison

:

**fabszn** · 18/04/2005, 14h52

Envoyé par trotters213

2 trucs :

si ça c'est une jointure :

Code :

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

pst_.NUMETAB = ctus.NUMETAB

c'est pas bon. C'est jamais dans le WHERE les jointures mais dans le FROM avec un opérateur normalisé JOIN :

Code :

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

SELECT * FROM td_pst pst_ INNER JOIN tds_ctu ctus ON pst_.NUMETAB = ctus.NUMETAB

au risque de passer pour un con (j'ai jamais vu Oracle) c'est quoi ?1, ?2, ... bref le ? dans tes comparaison

:

Concernant INNER JOIN cela permet de faire des jointures? cela remplace ce type de jointure :matable.monchamp=tatable.tonchamp?
C plus optimisé?
Cela veut dire que je peux l'utiliser dans n'importe quelle requete?

En fait , cette requete est executé dans un prog Java... les ?X correspondent parametre de la requete (rien à voir avec Oracle)..

Merci d'avance,

@+

Fabszn

**trotters213** · 18/04/2005, 15h03

oui le JOIN bien mieux. Ici tu y trouveras toutes les infos nécessaire et indispensable pour les utiliser (et les comprendre)

**fabszn** · 18/04/2005, 15h12

Merci bcp pour cette info!!!

Je vais regardé le lien... et changer ma facon de faire mes jointures...!!!

**Jaouad** · 18/04/2005, 16h22

je te conseille quand même de travailler avec l'explain plan

**fabszn** · 19/04/2005, 16h09

Envoyé par trotters213

2 trucs :

si ça c'est une jointure :

Code :

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

pst_.NUMETAB = ctus.NUMETAB

c'est pas bon. C'est jamais dans le WHERE les jointures mais dans le FROM avec un opérateur normalisé JOIN :

Code :

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

SELECT * FROM td_pst pst_ INNER JOIN tds_ctu ctus ON pst_.NUMETAB = ctus.NUMETAB

au risque de passer pour un con (j'ai jamais vu Oracle) c'est quoi ?1, ?2, ... bref le ? dans tes comparaison

:

Bonjour,

J'essaye d'utiliser la methode jointure entre deux tables comme tu me l'as expliquer dans ton post..

J'utilise Toad comme requeteur... mais celui ci ne reconnait pas le mot clef INNER JOIN...voila ma requete :

Code :

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

1
2
3
4
 
 
SELECT * FROM TDS_CTU ctus INNER JOIN TD_CTU ctu ON ctus.ANCAMPCTU=ctu.ANCAMPCTU,ctus.NUMECAN=ctu.NUMECAN
WHERE ctus.CDSUI='AQ'

Est ce que le probleme vient de toad qui ne reconnais pas cette syntaxe, ou est ce que j'ai fait une erreur dans l'ecriture de la requete.

Merci d'avance,

@+

Fabszn

**Quentin** · 19/04/2005, 16h25

Envoyé par fabszn

Code :

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

1
2
SELECT * FROM TDS_CTU ctus INNER JOIN TD_CTU ctu ON ctus.ANCAMPCTU=ctu.ANCAMPCTU,ctus.NUMECAN=ctu.NUMECAN
WHERE ctus.CDSUI='AQ'

Il faut que tu sépares les différentes clauses de ton INNER JOIN par des AND et non des virgules

Code :

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

... ON ctus.ANCAMPCTU=ctu.ANCAMPCTU AND ctus.NUMECAN=ctu.NUMECAN

[REQUETE][IMBRIQUEE] Bien ou pas bien?

Langage SQL

Discussions similaires

Partager

Partager