Discussion :

[RSI06]

Bonjour à tous et d'avance merci à eux qui pourront m'aider.
Je vous explique mon problème :
J'ai 2 tables :
- 1 table T_Fournissseurs
- 1 table T_Factures

La table T_Fournisseurs a 1 colonne "nb_factures"

Ma question est simple : comment puis-je compter le nombre total de facture pour un fournisseur donné au moment de la validation d'une nouvelle facture dans la table T_Facture ?

J'a voulu faire un trigger AFTER sur la table T_Factures
Extrait du code :

Code :

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

1
2
Select count(*) into nCount from T_Facture where id_fournisseur = :old.id_fournisseur;
update T_Fournisseur set nb_facture = nCount where id_fournisseur = :old.id_fournisseur;

Mais j'ai l'erreur oracle de la table en mutation car je veux lire la table T_Factures qui vient d'être mise à jour alors que mon trigger est de type AFTER.

Je ne comprend pas.

Connaissez-vous une meilleur solution ?

Merci

[orafrance]

une recherche sur le forum devrait t'apporter des éléments, un sujet a été l'objet d'un long débat pas plus tard que la semaine dernière

PS : le code du trigger serait nettement plus intéressant avec les balises ad hoc

[olive-andre]

Utilise une procédure ou une fonction (tu pourras utiliser cette derniere dans une requète) qui se chargera
1) de l'insertion dans la table T_FACTURE
2) de la mise à jour du compteur dans la table T_FOURNISSEUR.

Sinon, tu peux aussi utiliser une vue sur la table T_FOURNISSEUR de ce type

Code :

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

1
2
3
4
5
6
7
create or replace view V_FOURNISSEUR as 
select 
    id_fournisseur as id_fournisseur,
    (select count(*) 
        from T_FACTURE FA 
        where FA.id_fournisseur = FO.id_fournisseur) as nb_facture
from T_FOURNISSEUR FO ;

Elle marche

[RSI06]

Merci de ta réponse olive-andré mais l'insert se fait via le navigateur WEB.

La vue ne m'apportera rien car la colonne nb_facture est sujet à des calculs à d'autres moments.

Merci

L'idée de la procédure est intéressante
Je vais y travailler

[McM]

Il suffit de gérer tous les cas

Code :

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

1
2
3
4
5
6
7
8
9
10
11
IF INSERTING
THEN
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) + 1 WHERE id_fournisseur = :NEW.id_fournisseur;
ELSIF DELETING
THEN
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) - 1 WHERE id_fournisseur = :OLD.id_fournisseur;
ELSIF UPDATING AND :OLD.id_fournisseur <> :NEW.id_fournisseur
THEN
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) - 1 WHERE id_fournisseur = :OLD.id_fournisseur;
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) + 1 WHERE id_fournisseur = :NEW.id_fournisseur;
END IF;

Edit c'est un trigger BEFORE INSERT OR UPDATE OR DELETE FOR EACH ROW

[RSI06]

Il suffit de gérer tous les cas

Code :

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

1
2
3
4
5
6
7
8
9
10
11
IF INSERTING
THEN
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) + 1 WHERE id_fournisseur = :NEW.id_fournisseur;
ELSIF DELETING
THEN
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) - 1 WHERE id_fournisseur = :OLD.id_fournisseur;
ELSIF UPDATING AND :OLD.id_fournisseur <> :NEW.id_fournisseur
THEN
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) - 1 WHERE id_fournisseur = :OLD.id_fournisseur;
	UPDATE T_Fournisseur SET nb_facture = NVL(nb_facture,0) + 1 WHERE id_fournisseur = :NEW.id_fournisseur;
END IF;

Edit c'est un trigger BEFORE INSERT OR UPDATE OR DELETE FOR EACH ROW

Je vais tenter cette solution.
Je vous tiens au courant

[mnitu]

Je ne sais plus pourquoi mais il me semble qu’il faut ajouter aussi une contrainte sur la table facture. Mais j’ai du mal à me rappeler pourquoi

[olive-andre]

je ne sais pas ce que tu entends par

La vue ne m'apportera rien car la colonne nb_facture est sujet à des calculs à d'autres moments.

mais, pourquoi dans le code de ton trigger tu ne calcules juste pas

Code :

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

UPDATE T_Fournisseur SET nb_facture = nCount WHERE id_fournisseur = :old.id_fournisseur;

penses aussi à soustraire les factures que tu détruits, etc....

[olive-andre]

Désolé, MCM a tiré plus vite que moi ....

[McM]