Trigger MAJ quantité

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:

---

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 :roll:
Merci

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:

---

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:

---

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:

---

1 2 3 4 5 6

UPDATE P SET [quantite_restante] -= [quantite] FROM Produit P INNER JOIN inserted ON ID_PROD=ID_PRODUIT

---

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 !! :ccool:

Citation:

---

Envoyé par davelop

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...