Discussion :

[Marcel Chabot]

Bonjour je suis sous Oracle 10g.

Je veux faire la comparaison des des données de deux tables.
Ce sont deux tables identiques mais une qui servira d'historique.
La table histotique aura quatre colonnes supplémentaires.

J'ai penser faire une boucle imbriqué afin de lire chaque ligne des deux tables silmutanémment afin de voir s'il y a des différences dans une valeur de colonne des deux tables. Si oui je dois faire un DML.

Est-ce que c'est la bonne façon ou bien il y a de quoi de plus performant?

Merci!!!

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
21
22
23
24
25
26
27
28
29
30
31
32
33
 
DECLARE
V_DATE_DEBUT_OPER DATE;
 
CURSOR CUR_i_PAYS IS
   SELECT PAYS.CODE_PAYS, PAYS.CODE_REGION_MONDIALE, PAYS.NOM_LONG_PAYS, PAYS.NOM_PAYS
     FROM I_PAYS PAYS;
 
  REC_I_PAYS  CUR_I_PAYS%ROWTYPE;
 
CURSOR CUR_i_PAYS_H IS
   SELECT PAYS.CODE_PAYS, PAYS.CODE_REGION_MONDIALE, PAYS.NOM_LONG_PAYS, PAYS.NOM_PAYS,
               PAYS.NO_VERSION_H, PAYS.HORODATE_AJOUT_H, PAYS.HORODATE_DESACTIVATION_H, PAYS.INDIC_ACTIF_H
     FROM I_PAYS_H PAYS;
 
  REC_I_PAYS_H CUR_I_PAYS_H%ROWTYPE;
 
 
BEGIN
V_DATE_DEBUT_OPER := TRUNC(SYSDATE);
 
FOR REC_I_PAYS IN CUR_I_PAYS LOOP
   FOR REC_I_PAYS_H IN CUR_I_PAYS_H LOOP
    IF REC_I_PAYS_H.INDIC_ACTIF_H = 1 THEN
      IF REC_I_PAYS.CODE_PAYS <>  REC_I_PAYS_H.CODE_PAYS THEN
        UPDATE I_PAYS_H SET HORODATE_DESACTIVATION_H = V_DATE_DEBUT_OPER;
        UPDATE I_PAYS_H SET INDIC_ACTIF_H = 1;
      END IF;
    END IF;
     END LOOP;
 END LOOP;
 
 END;

[Invité]

Est-ce que c'est la bonne façon ou bien il y a de quoi de plus performant?

Y a bien plus performant en se passant des curseurs et en faisant une requête globale

Trois questions sur ce bout de code :

Code :

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

1
2
UPDATE I_PAYS_H SET HORODATE_DESACTIVATION_H = V_DATE_DEBUT_OPER;
UPDATE I_PAYS_H SET INDIC_ACTIF_H = 1;

Pourquoi faire deux update au lieu d'un?
Tu fais un UPDATE sur l'ensemble de la table, sans clause WHERE? C'est vraiment ce que tu veux??
Pas de COMMIT dans le code?

[mnitu]

La bonne façon de faire c'est en SQL.

[Marcel Chabot]

Je suis pas super bon en SQL ;-)

Certain que j'aimerais faire un update au lieu de deux!
Je vais fouiller pour voir comment ça se fait ;-)
Certain aussi que c'est pas sur l'ensemble de la table historique que je veux faire un update mais seulement sur les lignes qui ne répondent pas aux critères demandés.

J'utilise deux curseurs pour voir quelle valeur de la table source est différente de la table cible (Historique).
Ensuite je dois faire un update sur la table cible de la ligne qui n'est pas pareille à la table cible.

Tu as bien raison, je vais faire une clause where pour comparer les deux clés primaires de chaque table pour faire le update de cette ligne seulement et non de l'ensemble de la table.
Et un commit c'est sur.

Est-ce que c'est bien ça?

Il faudrait surement que je crée un curseur qui contient mes deux select.
Afin de mettre ma clause where pour comparer la même ligne dans chaque table ?

[Invité]

Avec le peu d'info que tu donnes, ça devrais ressembler à ça - j'ai pas testé:

Code :

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

1
2
3
4
5
6
7
UPDATE I_PAYS_H IPH
SET IPH.HORODATE_DESACTIVATION_H = TRUNC(SYSDATE),
	IPH.INDIC_ACTIF_H = 1
JOIN I_PAYS IP
	on IP.CODE_PAYS = IPH.CODE_PAYS -- clef étrangère des deux tables
where IPH.champ1 != IP.champ1
	or IPH.champ2 != IP.champ2

Il faudrait que tu te mettes sérieusement au SQL. Il y a plein de tutoriels sur ce site...

[mnitu]

Code :

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

1
2
3
4
5
 
Update ...
   Set ...
Join ...
Where ...

ça ne passe pas chez Oracle.

[Invité]

Code :

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

1
2
3
4
5
 
Update ...
   Set ...
Join ...
Where ...

ça ne passe pas chez Oracle.

M*** j'suis devenu rouillé en Oracle
Au moins un avantage avec slq server sur l'écriture de ce genre de chose...

[Jorafali]

Bonjour,
A la sauce Oracle ca donnerait quelque chose comme ca.

Code sql :

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

1
2
3
4
5
6
7
8
 
UPDATE I_PAYS_H IPH
SET IPH.HORODATE_DESACTIVATION_H = TRUNC(SYSDATE),
      IPH.INDIC_ACTIF_H = 1
WHERE EXISTS ( SELECT 1 FROM I_PAYS IP 
                        WHERE IP.CODE_PAYS = IPH.CODE_PAYS 
                        AND ( IPH.champ1 != IP.champ1 OR IPH.champ2 != IP.champ2 )
               )

Je ne l'ai pas teste mais au pire l'idee est la

edit : J'ai vraiment du mal a faire ma mise en forme avec ce forum ...