Discussion :

[lenoil]

Salut,

J'aimerais effectuer un update sur un champ d'une table à partir d'une deuxième table mais ma jointure est sur 3 champs

Voici ma syntaxe qui fonctionne

Code :

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

1
2
3
4
5
6
7
8
9
10
UPDATE TABLE1 t1 
 SET champ1table1 = ( 
 SELECT champtable2 
 FROM TABLE2 t2 
 WHERE t2.c1=  t1.c1 
 and t2.c2 =  t1.c2 
 and t2.c3 =  t1.c3 
  ) 
  where c4 = 'condition1' 
AND c5 = 'condition2'

J'aimerias savoir s'il y a un moyen d'optimiser cette requête par exemple en rajoutant les condition du where à la fin de ma requete mais cela ne fonctionne pas

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
UPDATE TABLE1 t1 
 SET champ1table1 = ( 
 SELECT champtable2 
 FROM TABLE2 t2 
 WHERE t2.c1=  t1.c1 
 and t2.c2 =  t1.c2 
 and t2.c3 =  t1.c3 
  ) 
  where c4 = 'condition1' 
AND c5 = 'condition2'
and t2.c1=  t1.c1 
 and t2.c2 =  t1.c2 
 and t2.c3 =  t1.c3

merci pour votre aide

[Waldar]

Votre première syntaxe est correcte et efficace.

Que se passe-t-il si vous ne trouvez pas de correspondance sur votre table t2, aujourd'hui vous remettez la colonne champ1table1 à null, est-ce bien votre souhait ?

[remi4444]

On rajoute rarement de l'éfficacité en compliquant une requête simple
Si la requête rame, c'est peut etre un pb d'index sur TABLE2 (c1,c2,c3) ou bien sur TABLE1 (c4,c5)

[remi4444]

Votre première syntaxe est correcte et efficace.

Que se passe-t-il si vous ne trouvez pas de correspondance sur votre table t2, aujourd'hui vous remettez la colonne champ1table1 à null, est-ce bien votre souhait ?

Effectivement il manque peut etre une clause de filtrage, mais ça va changer le truc fonctionnellement (rien faire plutot que de mettre des null quand y'a pas de correspondance)

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
UPDATE TABLE1 t1 
 SET champ1table1 = ( 
 SELECT champtable2 
 FROM TABLE2 t2 
 WHERE t2.c1=  t1.c1 
 AND t2.c2 =  t1.c2 
 AND t2.c3 =  t1.c3 
  ) 
  WHERE c4 = 'condition1' 
 AND c5 = 'condition2'
 AND EXISTS (
 select 1 from TABLE2 F 
where F.c1=  t1.c1 
 AND F.c2 =  t1.c2 
 AND F.c3 =  t1.c3
)

[lenoil]

Merci à tous

effectivement
"
Que se passe-t-il si vous ne trouvez pas de correspondance sur votre table t2, aujourd'hui vous remettez la colonne champ1table1 à null, est-ce bien votre souhait ?"

oui c bien mon souhait elle est mise à null si pas de correspondance.

Je voulais essayer avec des inner join mais je connais pas trop

je vais essayer le exist pour voir si je gangne du temps

[Waldar]

Si vous désirez mettre des nulls s'il n'y a pas pas de correspondance, alors votre première requête est très bien.
Si vous avez un problème de performance, voir du côté des index comme le suggère remi4444.

[remi4444]

je vais essayer le exist pour voir si je gangne du temps

Non attention! j'ai bien prévenu que le EXISTS changeait le fonctionnement à savoir que la non-correspondance ne mettra pas de null, les ligne ne seront plus affectées.

Si c'est votre souhait de mettre du null, alors ne changez rien à votre requete qui est très bien, il y a un pb d'index ou alors d'écriture de condition.

[lenoil]

ok je vais garder ma requete et regarder du coté des index

Avec le exist cela semble plus long ....

Dommage que je ne puisse pas traiter que eux qui m'intéresse dans ma table1

Merci en tout cas

[remi4444]

oui c bien mon souhait elle est mise à null si pas de correspondance.

Dommage que je ne puisse pas traiter que eux qui m'intéresse dans ma table1

Vous vous contredisez là...

de toutes manières, EXISTS ou pas, ce sont les même index qui seront pris en comtpe.

[lenoil]

je comprends pas

si j'ai 4000 enregistrements dans ma table 1 avec ma condition de départ

Code :

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

1
2
c4 = 'condition1' 
 AND c5 = 'condition2'

quand je fais ma requete j'ai par exemple 3000 enregistrements qui correspondent

or dans ma requete je parcours quand même les 4000

where c4 = 'condition1'
AND c5 = 'condition2'

j'aurais voulu éviter ceci ...
mais je me perds un peu

[remi4444]

quand je fais ma requete j'ai par exemple 3000 enregistrements qui correspondent

Qui correspondent à quel critère ?

[lenoil]

à ceux ci
t2.c1= t1.c1
AND t2.c2 = t1.c2
AND t2.c3 = t1.c3

[remi4444]

Donc, contrairement à ce que tu as dit tout à l'heure, il ne faut pas mettre des null dans les lignes ne correspondant pas, il faut ne rien faire, ce n'est pas pareil

C'est alors la requête avec EXISTS qui est la bonne, assure toi que tu as bien un index sur ton triplet TABLE2(c1,c2,c3) d'une part et sur une ou 2 de tes colonnes impliquée dans les conditions sur la table TABLE1. Attention, tout est important. j'espère que tu as mis la requete exacte, car un simple détail comme un to_date, to_number ou une conversion implicite et ça change tout.