Trigger et table en Mutation

Version imprimable

Bonjour à vous avec mon trigger je voudrais que l’insertion d’une nouvelle opération concerner calcule le nouveau solde du compte en se basant sur le montant de l’opération et je voudrais aussi laisser les traces pour chaque opération effectué par l employé voici un début de code
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 create or replace TRIGGER DECLANCHE BEFORE UPDATE ON compte FOR EACH ROW DECLARE mttop operation.mtt_op%type; BEGIN SELECT mtt_op into mttop from OPERATION where num_op = num_op; if mttop > 0 THEN UPDATE COMPTE set avoir_cpt = solde_cpt + ( SELECT mtt_op FROM operation ) WHERE num_cpt = :new.num_cpt ; else UPDATE COMPTE set avoir_cpt = solde_cpt - ( SELECT mtt_op FROM operation ) WHERE num_cpt = :new.num_cpt ; End if ; EXCEPTION WHEN NO_DATA_FOUND THEN raise_application_error(-2001, 'numero de compte non valide non valide'); END;
voici mes tables
Agence (code_ag, nom_ag, ville_ag, #mat_emp) ;
Employe (mat_emp, nom_emp, date_embauche, numtel_emp, mt_salaire_emp, #code_ag, #code_ag_dir)
Client (code_cli, nom_cli, numtel_cli)
Operation (num_op, lib_op, date_op, mtt_op, #mat_emp)
Compte ( num_cpt, nature_cpt, solde_cpt, avoir_cpt, #code_ag, #mat_emp)
Concerner (#num_op, #num_cpt, sens)

16/07/2015, 14h42
McM

Voici un post du même genre http://www.developpez.net/forums/d15...e/#post8207145

Ok j ai vu le post cela ma permis de cerner la chose et par ailleurs j ai modifié mon code ,mais j ai un problème de mutation mais je vois pas comment le relever//
voici mon code Trigger
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 create or replace TRIGGER DECLANCHE AFTER INSERT ON CONCERNER FOR EACH ROW DECLARE mttop operation.mtt_op%type; sensop concerner.sens%type; BEGIN select sens into sensop from concerner where SENS =SENS; IF sensop = 1 THEN UPDATE COMPTE set avoir_cpt = solde_cpt + ( SELECT mtt_op FROM operation ) WHERE num_cpt = :new.num_cpt ; ELSIF sensop = 2 THEN UPDATE COMPTE set avoir_cpt = solde_cpt - ( SELECT mtt_op FROM operation ) WHERE num_cpt = :new.num_cpt ; End if ; EXCEPTION WHEN NO_DATA_FOUND THEN raise_application_error(-2001, 'numero du compte non valide'); END;
quand j insère la donnée

Code:

INSERT INTO CONCERNER (num_op, num_cpt, sens ) VALUES (5, '0084', 2);

j obtiens cet erreur
Erreur commençant à la ligne: 1 de la commande -
INSERT INTO CONCERNER (num_op, num_cpt, sens ) VALUES (5, '0084', 3)
Rapport d'erreur -
Erreur SQL : ORA-04091: table NGUIMBIS1.CONCERNER is mutating, trigger/function may not see it
ORA-06512: at "NGUIMBIS1.DECLANCHE", line 5
ORA-04088: error during execution of trigger 'NGUIMBIS1.DECLANCHE'
04091. 00000 - "table %s.%s is mutating, trigger/function may not see it"
*Cause: A trigger (or a user defined plsql function that is referenced in
this statement) attempted to look at (or modify) a table that was
in the middle of being modified by the statement which fired it.
*Action: Rewrite the trigger (or function) so it does not read that table.

17/07/2015, 09h31
Ptrous

Problème dans mon Trigger

je crois avoir résolu le problème de Trigger jai juste remplacé le

Code:

select sens into sensop from concerner where SENS =SENS;

par

Code:

sensop:= :new.sens;

mais j ai un autre message d erreur provenant du Trigger
Erreur commençant à la ligne: 1 de la commande -
INSERT INTO CONCERNER (num_op, num_cpt, sens) VALUES (5, '0084', 1)
Rapport d'erreur -
Erreur SQL : ORA-01427: single-row subquery returns more than one row
ORA-06512: at "NGUIMBIS1.DECLANCHE", line 8
ORA-04088: error during execution of trigger 'NGUIMBIS1.DECLANCHE'
01427. 00000 - "single-row subquery returns more than one row"
*Cause:
*Action:
17/07/2015, 10h39
al1_24

La réponse est dans le message d'erreur...
Combien de lignes retourne ta sous-requête SELECT mtt_op FROM operation ?
17/07/2015, 11h38
Ptrous

ok j ai pu remédier

Code:

SELECT mtt_op FROM operation P WHERE P.NUM_OP= :new.NUM_OP

ca marche mon problème est resolu
17/07/2015, 12h26
McM

Tu peux poster ton trigger parce que ce code n'est pas correct

Code:

select sens into sensop from concerner where SENS =SENS;

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
 
create or replace TRIGGER DECLANCHE
 
AFTER INSERT ON CONCERNER
FOR EACH ROW
 
DECLARE
mttop  operation.mtt_op%type;
sensop concerner.sens%type;
BEGIN
   sensop:= :new.sens; 
 
IF sensop = 1 THEN
     UPDATE COMPTE set avoir_cpt = solde_cpt + (   SELECT  mtt_op   FROM    operation P WHERE P.NUM_OP= :new.NUM_OP  )
WHERE num_cpt = :new.num_cpt  ;
     UPDATE COMPTE set  solde_cpt = avoir_cpt WHERE num_cpt = :new.num_cpt;
 
ELSIF  sensop = 2 THEN
UPDATE COMPTE set avoir_cpt = solde_cpt - (   SELECT  mtt_op   FROM    operation P WHERE P.NUM_OP= :new.NUM_OP   )
WHERE num_cpt = :new.num_cpt ;
UPDATE COMPTE set  solde_cpt = avoir_cpt WHERE num_cpt = :new.num_cpt;
End if ;
 
EXCEPTION
		WHEN NO_DATA_FOUND THEN 
				    raise_application_error(-2001, 'numero de compte non valide');
 
 
 
END;

Mon problème ici est la gestion des exception en principe je devrais avoir la possibilité de faire uniquement deux choix (sens=1 ou sens=2) à sens=3 aucune erreur ne surviens en plus je sais pas pourquoi mon SET SERVEROUTPUT ON génère une erreur je parviens pas à la mettre dans mon Trigger

17/07/2015, 13h19
Ptrous

Citation:

Envoyé par McM

Tu peux poster ton trigger parce que ce code n'est pas correct

Code:

select sens into sensop from concerner where SENS =SENS;

j ai modifié le code ...

1/ Si tu veux bloquer si le sens n'est pas 1 ou 2, soit tu crées une contrainte CHECK sur la colonne, soit tu le gères à la main dans le trigger avec un test et un raise_application_error

2/

Code:

RAISE_APPLICATION_ERROR(-2001, 'numero de compte non valide');

RAISE application_error doit avoir un code > -20000, donc pas -2001, mais -20001
Sinon erreur

Citation:

ORA-21000: error number argument to raise_application_error of -2001 is out of range

L'exception NO_DATA_FOUND n'est levée que sur un SELECT qui ne ramène aucune donnée.
Pas sur un Update. A aucun moment il n'y a un test d'existence de compte par un select, donc ce code ne sert à rien.
Si tu veux tester, soit tu testes que ton update modifie au moins une ligne (IF SQL%ROWCOUNT = 0 THEN RAISE_APPLICATION_ERROR...)
Soit tu fais un SELECT sur ta table de compte.

3/
Code:

1 2 UPDATE COMPTE set avoir_cpt = solde_cpt + ( SELECT mtt_op FROM operation P WHERE P.NUM_OP= :NEW.NUM_OP ) WHERE num_cpt = :NEW.num_cpt ; UPDATE COMPTE set solde_cpt = avoir_cpt WHERE num_cpt = :NEW.num_cpt;
2 updates : pas top.
Attention : Si la table operation n'a pas de ligne pour :new.num_op.. alors tu vas mettre NULL dans avoir_cpt.

Je ferais plutôt comme ceci
Code:

1 2 3 4 5 6 7 8 9 10 11 BEGIN SELECT mtt_op INTO v_mttop FROM operation P WHERE P.NUM_OP= :NEW.NUM_OP; IF :NEW.sens = 2 THEN v_mttop := - v_mttop; END IF; EXCEPTION WHEN NO_DATA_FOUND THEN .. -- a toi de voir ce qu'il faut faire END; IF :NEW.sens IN (1,2) THEN UPDATE COMPTE SET avoir_cpt = solde_cpt + v_mttop, solde_cpt = solde_cpt + v_mttop WHERE num_cpt = :NEW.num_cpt; IF SQL%ROWCOUNT = 0 THEN RAISE_APPLICATION_ERROR(-20001, 'numero de compte '|| :NEW.num_cpt ||' non valide'); END IF; END IF;

17/07/2015, 17h13
mnitu
Deux remarques:
1. Gérer des cumuls dans les trigger c'est une mauvaise "bonne solution". Souvent ces cumuls deviens "déphasés" par rapport aux écritures détaillés (voir 2).
2. Calculer des valeurs depuis une requête suivi d'une mise à jour d'une autre table en utilisant ces valeurs surtout pour gérer des cumuls n'est pas si anodine que cela semble et sans prendre beaucoup de soin ça risque de déphaser les cumuls de leurs valeurs réelles (voir 1).