Contrôler UPDATE d'une table [Résolu]

[sat83]

Bonjour à tous,
J'ai un programme (C++) qui réalise des requêtes UPDATE cycliquement sur une table.

Le problème, c'est que je ne dois pas effectuer la mise à jour de l'enregistrement si la valeur du champ NOM_DU_CHAMP est inchangée.

Juste avant d'effectuer la requête UPDATE, j'effectue un SELECT pour récupérer l’enregistrement existant, et je n'effectue l'UPDATE uniquement si la valeur de NOM_DU_CHAMP est différente.

Ne serait-il pas mieux d'effectuer les UPDATE dans mon programme C++ sans me poser de question, et d'ajouter un TRIGGER BEFORE UPDATE à ma table qui annulerait l'UPDATE si nécessaire.

Quelle solution vous semble la plus judicieuse? Si c'est celle du Trigger, comment annuler l'UPDATE si l'ancienne et la nouvelle valeur du champ sont identique?

Merci d'avance pour vos conseils.

PS: A noter qu'il y a plus d'UPDATE annulé qu'effectué.

[Dajon]

Bonjour,

Pourquoi ne pas inclure le test dans ton update, cela éviterait de perdre du temps à récupérer la valeur d'abord et cela me semble plus "naturel" (enfin bon c'est personnel )

Code :

1
2
3
4
UPDATE matable
SET champ=nouvelle_valeur
WHERE id = id_a_updater
AND champ != nouvelle_valeur

Ainsi l'update n'est réalisé que si les valeurs sont différentes, pas besoin de select l'ancienne valeur, et pas de trigger.

[Edit] Je pense que c'est plus judicieux de faire comme cela car:
- Pas de logique superflue dans ton code
- Pas de trigger, j'évite au maximum d'en utiliser car je trouve que leur coût en terme de maintenance est souvent élevé (tout le monde les oublie, et on perd a chaque fois du temps à chercher à droite a gauche l'origine d'un problème, avant de penser aux triggers)
- Je pense qu'un update réalisé comme cela est globalement moins coûteux qu'un select systématique ou un trigger systématique pour vérifier les valeurs. Je n'ai pas vérifié ce point.

[Waldar]

Attention à ne pas oublier les nulls (ce n'est pas précisé, donc c'est peut-être inutile) :

Code :

1
2
3
4
5
UPDATE matable
   SET champ = nouvelle_valeur
 WHERE id = id_a_updater
   AND (champ <> nouvelle_valeur 
    OR  champ IS NULL);

[sat83]

Merci, je vais voir pour adapter mon code en ce sens...

[Mohamed.Houri]

Envoyé par sat83

Bonjour à tous,
J'ai un programme (C++) qui réalise des requêtes UPDATE cycliquement sur une table.

Attention aussi aux ''bind variables''. Pensez à mettre votre update dans une procédure PL/QL stockée afin que le soucis des ''binds variables'' ne se posera pas pour vous.

Pensez aussi à l'appel de cette procédure. Il faudrait que vous l'appelliez en utilisant des ''binds variables'' si elle contient des paramètres. Pour cela vous devez utilisez OCI interface OCIStmtPrepare2 et OCIDefineByPos. Je ne suis pas un spécialiste de ce qui précède mais vous pouvez trouvez des détails sur ceci dans le livre de Chistian Antognini:

http://www.amazon.fr/Troubleshooting-Oracle-Performance-Christian-Antognini/dp/1590599179
Si vous ne le faites pas correctement vous allez remplir la mémoire (v$sql) de vos appels à la procédure.

http://hourim.wordpress.com/2011/06/...ing-parameter/

[Dajon]

Utiliser des bind variable est une bonne pratique, mais ce n'est pas toujours possible (Risque de régression dans une appli existante, pas le temps, ou choix projet: "nan, on veut pas appeler de procédure et pis c'est tout").

Dans ce cas, il y a aussi le paramètre CURSOR_SHARING, qui peut être intéressant il me semble.

Mais s'il n'y a pas de problème de performance relatif à ces updates, je ne sais pas si sat83 à besoin de se poser ces questions

[Mohamed.Houri]

[QUOTE=Dans ce cas, il y a aussi le paramètre CURSOR_SHARING, qui peut être intéressant il me semble.
QUOTE]

Dans ce cas je pense que vous n'avez pas lu le lien que j'ai proposé.

[mnitu]

Envoyé par sat83

...
Quelle solution vous semble la plus judicieuse? ...

Merge

[Dajon]

@Mohamed: Oui, j’avoue, je n'ai pas pris le temps de lire votre article dans le détail mea maxima culpa. Je ne savais pas que le cursor sharing à force ne fonctionnait pas pour des appel à du PL depuis une application tierce. Merci

[sat83]

Envoyé par mnitu

Merge

En faite j'utilise effectivement un MERGE.
Si l'enregistrement existe j'effectue un UPDATE, et si il n'existe pas encore j'effectue un INSERT.

Maintenant je souhaiterais ajouter la nuance: si l'enregistrement existe et que 3 champs sont identiques, je ne fais rien.

J'aurais pensé utiliser une clause WHERE dans le UPDATE du MERGE, mais apparemment ce n'est pas possible. J'avou ne pas être trop à l'aise avec le MERGE.

Est ce que ca vous semble possible comme fonctionnement avec un MERGE de faire soit un INSERT, soit un UPDATE, soit RIEN?

[Waldar]

Envoyé par sat83

J'aurais pensé utiliser une clause WHERE dans le UPDATE du MERGE, mais apparemment ce n'est pas possible. J'avoue ne pas être trop à l'aise avec le MERGE.

Vous pouvez !
Mettez votre clause WHERE avant la clause WHEN NOT MATCHED THEN INSERT.

[sat83]

Envoyé par Waldar

Mettez votre clause WHERE avant la clause WHEN NOT MATCHED THEN INSERT.

Malheureusement ça ne fonctionne pas...
Mon MERGE fonctionne bien, mais j'ai une erreur lorsque j'ajoute une clause WHERE dans l'UPDATE:

Code :

1
2
3
4
5
6
[...]
WHEN MATCHED THEN UPDATE SET
[...]
WHERE NOM_DU_CHAMP = '1500'  -- ça fonctionne sans cette ligne
WHEN NOT MATCHED THEN INSERT (
[...]

[skuatamad]

Quelle version d'oracle ?
merge enhancements in 10g

[sat83]

ORACLE 9.i
Je pense que ça doit venir de là... Ca ne doit fonctionner qu'avec la version 10.

Je pense que dans mon programme C++ je vais remplacer mon MERGE par un SELECT suivi d'un INSERT, d'un UPDATE ou de RIEN selon le cas...

[mnitu]

Envoyé par sat83

...
Maintenant je souhaiterais ajouter la nuance: si l'enregistrement existe et que 3 champs sont identiques, je ne fais rien.
...

Ne sélectionnez pas ces enregistrements.

[sat83]

Je pensais que si je ne sélectionnais pas ces enregistrements, la requête MERGE effectuerait un INSERT, non?

[mnitu]

Pour un ensemble des enregistrements sélectionnés le merge fait soit insert soit update. Si parmi ces enregistrements ne figure pas celles qui existent déjà dans la table à mettre à jour et qui respectent certaines critères alors il n’y aurait pas de problème.

[sat83]

Voici une version simplifiée de ma requête MERGE:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
MERGE INTO NOM_DE_LA_TABLE T1
USING ( SELECT 
'vvv'    NOM_CHAMP_1 ,
'www'    NOM_CHAMP_2 , 
'xxx'    NOM_CHAMP_3 , 
'yyy'    NOM_CHAMP_4 , 
'zzz'    NOM_CHAMP_5 , 
 
 FROM DUAL ) REF_FIELDS
--CONDITIONS DE CORRESPONDANCES
ON ( 
      T1.NOM_CHAMP_1 = REF_FIELDS.NOM_CHAMP_1  AND
      T1.NOM_CHAMP_2 = REF_FIELDS.NOM_CHAMP_2  AND 
      T1.NOM_CHAMP_3 = REF_FIELDS.NOM_CHAMP_3  AND 
      T1.NOM_CHAMP_4 = REF_FIELDS.NOM_CHAMP_4  AND 
      T1.NOM_CHAMP_5 = REF_FIELDS.NOM_CHAMP_5       )
--SI CORRESPONDANCE => UPDATE
WHEN MATCHED THEN UPDATE SET
--[...] Liste des champs mis à jour
 
--SINON => INSERT
WHEN NOT MATCHED THEN INSERT ( /*[...] */ ) 
VALUES ( /*[...] */ )

A quel niveau est-ce que je dois placer le critère? (dans mon exemple, le critère porte sur 3 champs qui ne sont pas les champs 1 à 5 )...
J'aimerais que si l'enregistrement existe ET si les 3 champs (6 à 8 par exmeple) son identiques à ceux existant ALORS je ne fais RIEN (ni INSERT ni UPDATE)
Merci d'avance pour vos éclaircissement!

[Mohamed.Houri]

Vous pouvez ajouter une clause where à votre update comme dans l'exemple ci-dessous

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
mhouri > DROP TABLE t_emp purge;
 
TABLE dropped.
 
mhouri > CREATE TABLE t_emp AS SELECT * FROM emp;
 
TABLE created.
 
mhouri > MERGE INTO t_emp
  2  USING (SELECT
  3           b.empno
  4          ,b.ename
  5          ,b.job
  6          ,b.mgr
  7          ,b.hiredate
  8          ,b.sal
  9          ,b.comm
 10          ,b.deptno
 11        FROM
 12          emp b) e
 13  ON    (e.empno = t_emp.empno)
 14  WHEN MATCHED THEN
 15       UPDATE
 16       SET    t_emp.sal   =  e.sal ,
 17              t_emp.comm  =  e.comm
 18       WHERE
 19             t_emp.deptno  != e.deptno
 20       AND   t_emp.mgr      != e.mgr
 21       AND   t_emp.hiredate != e.hiredate
 22  ;
 
0 rows merged.
 
mhouri > UPDATE t_emp
  2  SET mgr = 7698
  3  , hiredate = sysdate
  4  , deptno  = 30
  5  WHERE empno = 7369;
 
1 row updated.
 
mhouri > MERGE INTO t_emp
  2  USING (SELECT
  3           b.empno
  4          ,b.ename
  5          ,b.job
  6          ,b.mgr
  7          ,b.hiredate
  8          ,b.sal
  9          ,b.comm
 10          ,b.deptno
 11        FROM
 12          emp b) e
 13  ON    (e.empno = t_emp.empno)
 14  WHEN MATCHED THEN
 15       UPDATE
 16       SET    t_emp.sal   =  e.sal ,
 17              t_emp.comm  =  e.comm
 18       WHERE
 19             t_emp.deptno  != e.deptno
 20       AND   t_emp.mgr      != e.mgr
 21       AND   t_emp.hiredate != e.hiredate
 22  ;
 
1 row merged.

[sat83]

Malheureusement, comme expliqué plus haut, la clause WHERE ne fonctionne pas dans un MERGE sous ORACLE 9.i