Discussion :

[madimane]

bonjour tout le monde
j'ai fait un trigger qui vérifie si un numéro de carte de crédit ou un numéro de paiement ont été déja utilisé et le cas échéant je dois supprimer un enregistrement dans une autre table.
mon code est le suivant:

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
34
35
36
37
38
39
40
41
42
CREATE OR REPLACE TRIGGER NO_CRT_CREDIT
BEFORE INSERT OR UPDATE ON PAIEMENTCREDIT
FOR EACH ROW
DECLARE
	Crt   NUMBER;
        Exist NUMBER;
        Aut   NUMBER;
BEGIN
        Crt   := 0;
        Exist := 0;
        Aut   := 0;
 
        SELECT count(nopaiement)
   	INTO Exist
   	FROM paiement
   	WHERE  paiement.nopaiement = :new.nopaiement ;
 
        IF (Exist = 0) THEN
            RAISE_APPLICATION_ERROR (-20005,'noPaiement invalide (inexistant dans la table Paiement)');
        end if;
 
	SELECT count(distinct noclient)
   	INTO Crt
   	FROM paiementcredit, paiement, detaillivraison, commande
   	WHERE  paiementcredit.nopaiement = paiement.nopaiement AND
        paiement.nolivraison = detaillivraison.nolivraison AND
        detaillivraison.nocommande = commande.nocommande AND
	nocarte = :new.nocarte;
 
        SELECT count(noautorisation)
   	INTO Aut
   	FROM paiementcredit
	where noautorisation = :new.noautorisation ;
 
	IF (Crt > 1) THEN
            DELETE FROM paiement WHERE nopaiement = :new.nopaiement;
	    RAISE_APPLICATION_ERROR (-20005,'Carte de credit deja utilisee par un autre client');
        elsif (Aut > 0) THEN
            DELETE FROM paiement WHERE nopaiement = :new.nopaiement;
            RAISE_APPLICATION_ERROR (-20005,'numero autorisation deja utilise pour un autre paiement');
        end if;
END;

le problème c'est qu'il affiche le message,ce qui veut dire qu'il rentre dans le if mais le delete ne se fait pas.
est ce que ca marche un delete dans un trigger??
merci pour votre aide

[pifor]

Oui, le DELETE dans un trigger doit marcher.

Est-ce que vous vérifiez dans la même connexion Oracle ou dans une autre connection ?
Etes-vous sûr que le code appelant fait COMMIT ? Si raise_application_error est exécuté, il sera récupéré par le premier gestionnaire d'exception dans la pile qui peut faire ROLLBACK.

[plaineR]

Oui, c'est votre raise_application_error qui annule le delete. Si vous voulez faire malgré tout le delete vous devez passer par une procédure autonome (autonomous_transaction) dans lequel vous committerez votre delete.

[madimane]

merci d'avoir porté attention à mon problème et je m'excuse car je suis vraiment un débutant dans cet immense domaine
j'ai écris une procédure autonome_transaction qui est la suivante:

Code :

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

1
2
3
4
5
6
CREATE OR REPLACE PROCEDURE SUPPRESSION( leNopaiement paiement.nopaiement%TYPE ) IS
PRAGMA AUTONOMOUS_TRANSACTION;
BEGIN
 DELETE FROM paiement WHERE nopaiement = leNopaiement;
 COMMIT;
END SUPPRESSION;

et j'ai fait l'appel à cette procédure dans le trigger comme suit

CREATE OR REPLACE TRIGGER NO_CRT_CREDIT
BEGIN
.....
....
elsif (Aut > 0) THEN
EXECUTE SUPPRESSION( :new.nopaiement)
RAISE_APPLICATION_ERROR (-20005,'numero autorisation deja utilise pour un autre paiement');
end if;
END;

mais il me sort les erreurs suivante:
LINE/COL ERROR
-------- -----------------------------------------------------------------
37/21 PLS-00103: Encountered the symbol "SUPPRESSION" when expecting
one of the following:
:= . ( @ % ; immediate
The symbol ":=" was substituted for "SUPPRESSION" to continue.
pouvez vous m'aider??

[bpprive]

Pas besoin de mettre le execute, juste le nom de la procédure suffit. Un ; aide toujours.

De plus, votre trigger ne fonctionnera pas. Vous faites un select sur la table de votre trigger, la table sera mutante et il y aura une erreur oracle...

Sr référer à l'article de Pomalaix pour voir une solution. Recherche "table mutante" dans la case de recherche.

De plus, nul besoin de supprimer explicitement la ligne insérée dans un trigger lorsqu'on détecte une erreur, le raise_application_error va faire un rollback implicite pour nous

Atention avec le autonomous_transaction, je ne pense pas que ce ne soit ce que vous désirez vraiment faire... Une étude plus approfondie s'impose je pense.

bpprive

[madimane]

merci infiniment pour ces explications assez précises.
effectivement j'ai enlevé le execute et j'ai rajouté mon fameux; et ca marche.
concernant la ligne ke je voulais supprimer est dans une autre table(la table mère[paiement]) et ma table mutante été la table(enfant[paiement_credit]).
merci infiniment pour votre aide
:o