Discussion :

[JCD21]

Bonjour.
Je précise que je suis débutant avec les triggers.
Voilà le problème:
J'ai une table RELTUBES qui contient un champ identifiant TUBE_ID créé avec une séquence.
Cette table contient aussi un champ TUBE_GVID et un champ TUBE_TRANCHEID.
Lorsque j'ajoute ou modifie une ligne dans RELTUBES (en particulier le champ TUBE_GVID), je veux mettre la valeur 1 (je simplifie pour l'instant) dans le champ TUBE_TRANCHEID lorsque TUBE_GVID n'est pas NULL.

J'ai donc tapé ceci:

Code :

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

1
2
3
4
5
6
7
8
 
create or replace trigger METAJOURTRANCHE
after insert or update of TUBE_GVID on RELTUBES
for each row when (new.TUBE_GVID is not null)
begin
update RELTUBES set TUBE_TRANCHEID=1 where TUBE_ID=:new.TUBE_ID;
end;
/

Mais j'ai le message d'erreur:
ORA-04091: la table UTILISATEURRELACHEMENT.RELTUBES est en mutation ; le déclencheur ou la fonction ne peut la voir

Comment faire?
Merci.

[Waldar]

L'erreur provient d'un simple problème de logique.

Votre trigger se déclenche APRÈS insertion ou mise à jour.
Mais pendant votre trigger, vous lancez une mise à jour, qui va donc exécuter le trigger, qui va donc refaire une mise à jour...

Il faut donc que votre trigger s'exécute avant insertion ou mise à jour, et il suffit de modifier la nouvelle valeur de TUBE_TRANCHEID (une simple assignation, pas d'update nécessaire).

Je vous laisse chercher un peu pour la syntaxe.

[Mohamed.Houri]

Bonjour.
Mais j'ai le message d'erreur:
ORA-04091: la table UTILISATEURRELACHEMENT.RELTUBES est en mutation ; le déclencheur ou la fonction ne peut la voir

Comment faire?
Merci.

Bonjour,

La "Mutating Table error" est là pour vous protéger. Elle est là pour vous dire que votre logique n'est pas bonne.

Pour votre gouverne, lorsque vous crééz un trigger sur une table T et que le contenu de ce trigger fait un select/update sur la table T, sachez alors que votre logique n'est pas bonne.

De plus, si tant est que votre trigger n'est pas en erreur, avez-vous pensé, lors de la création de ce trigger, à l'impact qu'aura ce trigger sur une application mutli-users mutli-concurrente.

Moralité : faire très attention à l'aspect mutli-user lors de l'utilisation des triggers

Bien à vous

Mohamed Houri

[JCD21]

J'ai esayé ce code:

Code :

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

1
2
3
4
5
6
7
8
 
create or replace trigger METAJOURTRANCHE
before insert or update of TUBE_GVID on RELTUBES
for each row when (new.TUBE_GVID is not null)
begin
select 1 into :new.TUBE_TRANCHEID from RELTUBES;
end;
/

mais j'ai cette erreur:
OCI_NO_DATA

Il n'est toujours pas bon mon trigger?

[Waldar]

Vous n'être plus très loin, il y a encore plus simple pour assigner une valeur, sans SELECT cette fois-ci.

[JCD21]

OK. Je n'ai pas trouvé...
Ce serait vraiment sympa de me donner la solution

[Waldar]

Code :

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

:new.TUBE_TRANCHEID := 1;

[JCD21]

évidemment.

[JCD21]

Maintenant un peu plus compliqué:
au lieu de mettre 1 dans :new.TUBE_TRANCHEID, je veux mettre

Code :

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

select GV_INTERVALLEENTRERGVID from RELGV where GV_ID=:new.TUBE_GVID

où RELGV est une table qui a pour identifiant GV_ID et qui contient le champ GV_INTERVALLEENTRERGVID.
Cette requête ne renvoie qu'une valeur.
J'ai donc essayé :

Code :

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

1
2
3
4
5
6
7
create or replace trigger METAJOURTRANCHE
before insert or update of TUBE_GVID on RELTUBES
for each row when (new.TUBE_GVID is not null)
begin
select (select GV_INTERVALLEENTRERGVID from RELGV where GV_ID=:new.TUBE_GVID) into :new.TUBE_TRANCHEID from RELTUBES;
end;
/

mais j'ai le message d'erreur OCI_NO_DATA
puis :

Code :

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

1
2
3
4
5
6
7
create or replace trigger METAJOURTRANCHE
before insert or update of TUBE_GVID on RELTUBES
for each row when (new.TUBE_GVID is not null)
begin
:new.TUBE_TRANCHEID := (select GV_INTERVALLEENTRERGVID from RELGV where GV_ID=:new.TUBE_GVID);
end;
/

mais ça ne passe pas à la compilation.
Que faut-il faire?

[Waldar]

Vous aimez bien faire compliqué, vous aviez déjà écrit la bonne syntaxe un peu plus haut :

Code :

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

1
2
3
SELECT GV_INTERVALLEENTRERGVID INTO :new.TUBE_TRANCHEID
  FROM RELGV
 WHERE GV_ID = :new.TUBE_GVID;

À mon avis ce que vous n'avez pas compris c'est que vous n'avez jamais besoin de faire référence à votre table RELTUBE dans votre trigger.
Toutes les colonnes existent, que ce soit les anciennes ou nouvelles valeurs avec les préfixes new et old.

[benj9025]

Bonjour

J'ai un trigger insert que je voudrais bien modifié pour en faire un update avez-vous une solution SVP ?
J'avoue que je suis perdu avec çà, j'ai même essayer de faire une table à coté. mais resultat néant. grr j'en perds mon latin c'est clair.
Merci d'avance pour votre aide.
=========================
create or replace
TRIGGER CAD_NUM_RUE
after
INSERT ON CAD_NUM_RUE
BEGIN

update CAD_NUM_RUE set CAD_NUM_RUE.adresse = CAD_NUM_RUE.num ||' '|| CAD_NUM_RUE.RUE;

END;
=========================