Discussion :

[davelop]

Bonjour,
J'essaye de faire un trigger qui mettrait à jour la quantité d'une table en fonction de l'insert ou du delete d'une autre table
voici les 2 tables :

COMPO
--------
ID_COMPO
ID_PRODUIT
Quantite

PRODUIT
---------
ID_PROD
NOM_PROD
Quantite_Restante

Je voudrais que si Insert dans COMPO cela me retranche la quantite du produit dans PRODUIT
Et que si Delete dans COMPO cela me recrédite la quantite dans PRODUIT

J'ai tenté ce trigger pour l'insert mais cela m'a mis à NULL toutes les quantite_restante de PRODUIT.
Et si je ne met pas le "where" cela me retire la quantité à toutes les lignes de PRODUIT

Code :

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

1
2
3
4
5
CREATE TRIGGER [retire_quantite] ON [COMPO] 
FOR INSERT
AS
update [PRODUIT]
set [quantite_restante]=[quantite_restante]-(select [quantite]  from inserted where ID_PROD=ID_PRODUIT)

Je suis sûr que vous allez voir tout de suite ce qu'il ne va pas
Merci

[aieeeuuuuu]

Bonjour,

Oui, en effet, ce qui ne va pas, c'est cette colonne "quantite_restante" dans la table Produit...
Elle devrait plutot être dans une vue qui fait le calcul...

Enfin pour votre trigger, le comportement est normal :

Code :

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

1
2
 
SELECT [quantite]  FROM inserted WHERE ID_PROD=ID_PRODUIT

Renvoit NULL pour le Produit n'ayant pas fait l'objet de mise à jour (donc pas dans la table inserted)

du coup :

Code :

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

1
2
 
[quantite_restante]-(SELECT [quantite]  FROM inserted WHERE ID_PROD=ID_PRODUIT)

est NULL aussi. Ansi, vous mettez à NULL toutes les quantités pour les produits qui ne sont pas dans inserted.

Pour corriger cela, il faut ne mettre à jour que les produits qui sont dans inserted :

Code :

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

1
2
3
4
5
6
 
UPDATE P
SET [quantite_restante] -= [quantite]
FROM Produit P
INNER JOIN inserted
    ON    ID_PROD=ID_PRODUIT

[davelop]

Super j'ai compris et cela fonctionne très bien.

En ce qui concerne de faire plutôt une vue qui effectuerai le calcul, c'est très certainement plus "propre" mais c'est préférable pour nous de modifier en "dur" dans la table les quantités restantes

Merci !!

[aieeeuuuuu]

En ce qui concerne de faire plutôt une vue qui effectuerai le calcul, c'est très certainement plus "propre"

Ce n'est pas juste plus propre, c'est aussi plus simple, plus sûr et plus performant qu'un trigger...