Transact-SQL , les triggers

[lepolo2oo8]

Salut a tous!
G besion de votre aide!
En faite g develope un programme de gestion commercial . Aimerai savoir comment ecrire un trigger qui met a jour des lignes d' une colonne. voisi mes exemples de tables au depart avec trigger:
---------------------
table Produit:
IDProd article
1 article1
2 article2
--------------------
-------------------------------------------------
table Commande:
IDCom - IDProd - QteProdAchete - QteProdRestante
1 1 150 150
2 1 200 200
3 1 100 100
4 2 50 50
5 2 70 70
6 2 80 80
--------------------------------------------------
table ProduitVendu:
IDProdVen -IDProd - QteProdVendu
1 1 300
2 2 100
-----------------------------------------------------
Alors voici ma question: Comment creer un trigger qu' apres insertion sur la table ProduitVendu va update les lignes de colonne QteProdRestante de la table commande
en faisant le calcul : (QteProdRestante - QteProdVendu ) puis mettre le resultat a la derniere ligne de la dite colonne parraport au IDProd.
Donc exple (150+200+100) - 300 = 150 et resultat doit etre :
-------------------------------------------------
Commande:
IDCom - IDProd - QteProdAchete - QteProdRestante
1 1 150 0
2 1 200 0
3 1 100 150
------------------------------------------------------
Merci de votre aide...

[iberserk]

Comment creer un trigger qu' apres insertion sur la table ProduitVendu va update les lignes de colonne QteProdRestante de la table commande

C'est très simple: ne le créez pas.
Vous cherchez à stocker une information qui est le fruit d'un calcul... faites ce calcul en temps réel quand vous en avez besoin...

[elsuket]

Bonjour,

Comme le dit si bien iberserk, vous n'avez pas besoin de créer un trigger pour cela.
Les triggers sont conçus pour implémenter des règles métier complexes.
Il faut également noter que les triggers allongent les transactions, que l'on veut autant que c'est possible faire s'exécuter dans le temps le plus court possible.

Si vous pensez devoir mettre à jour cette quantité à partir de plusieurs autres procédures stockées, crééz donc une procédure stockée qui se charge seulement de la mise à jour de cette colonne.
Vous pourrez ainsi l'appeler dans d'autres procédures stockées.

Si vous n'avez qu'une seule procédure stockée qui ajoute des lignes dans la table Commande, et que vous pensez qu'il y a de faibles chances que vous ayez d'autres procédures stockées qui fassent de même, alors faites tout le calcul dans cette procédure stockée.

Si vous restez sur une seule procédure stockée pour le moment, voici ce que vous pourriez écrire :

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
31
CREATE PROCEDURE usp_ProduitVendu_ajouter
	@_IDProd int
	, @_QteProdVendu int
AS
BEGIN
	SET NOCOUNT ON
 
	INSERT	dbo.ProduitVendu
	(
		IDProd
		, QteProdVendu
	)
	VALUES
	(
		@_IDProd
		, @_QteProdVendu
	)
 
	;WITH
		CTE AS
		(
			SELECT	MAX(IDCom) AS last_IDCom_IDProd
			FROM	dbo.Commande
			WHERE	IDProd = @_IDProd
		)
	UPDATE	dbo.Commande
	SET	QteProdRestante = QteProdRestante - @_QteProdVendu
	FROM	dbo.Commande AS C
	JOIN	CTE AS T ON C.IDCom = T.last_IDCom_IDProd
	WHERE	IDProd = @_IDProd
END

@++

[SQLpro]

Envoyé par iberserk

C'est très simple: ne le créez pas.
Vous cherchez à stocker une information qui est le fruit d'un calcul... faites ce calcul en temps réel quand vous en avez besoin...

Mieux, utilisez des vues !!!!

Normalement on ne devrait jamais développer d'application qu'avec des vues et non pas directement en accédant aux tables ce qui est aberrant !

A +

[lepolo2oo8]

En faite mon probleme est resoulu partiellement, car je veux que les lignes qu' ils ont ete deja traiter prennent la valeur 0; comme ca par exemple:
-------------------------------------------------------
Commande:
IDCom - IDProd - QteProdAchete - QteProdRestante
1 1 150 0
2 1 200 0
3 1 100 150
------------------------------------------------------

[SQLpro]

Encore une fois, faites une vue :

Code :

1
2
3
4
5
6
7
8
9
CREATE VIEW V_COMMANDE_SOLDE
AS
SELECT IDCom, IDProd, QteProdAchete, 
       CASE WHEN QteProdRestante - QteProdVendu < 0 THEN 0
            ELSE QteProdRestante - QteProdVendu
       END AS QteProdRestante
FROM   Commande AS C
       LEFT OUTER JOIN ProduitVendu AS PV
            ON C.IDProd  = PV.IDProd;

Vous pouvez même insérez, modifier ou supprimer les données à travers les vues ...

Exemple :

Code :

1
2
INSERT INTO V_COMMANDE_SOLDE (IDCom, IDProd, QteProdAchete)
VALUES (4, 14, 50);

A +

[lepolo2oo8]

Je crois que je m' explique mal.
IDProd QteProdRestante
1 15
2 12
3 25
alors le resultat que je veus obtenir apres insertion de de QteProdVendu = 21 soit l' update suivant:
IDProd QteProdRestante
1 0
2 0
3 31

ou ceci
IDProd QteProdRestante
1 31
2 0
3 0

Bref que le resultat de la soustraction (15+12+25)-21=31 soit place a un des champs de QteProdRestante et le reste des champs prennent la valeur o. Car meme avec la procedure donc vous m avez donne ,ca update le dernier champ mais les autres champs gardent leur valeurs de depart et avec la vue j' obteins les valeurs 0 sur tous les champs.

Merci .
Mes excuses pour le derangement .

[elsuket]

Dans ce cas il est clair que la solution de SQLPro est à tous points de vue la plus intelligente.

Si néanmoins vous souhaitez utiliser la procédure stockée, il vous faudrait la changer en :

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 PROCEDURE usp_ProduitVendu_ajouter
	@_IDProd int
	, @_QteProdVendu int
AS
BEGIN
	SET NOCOUNT ON
 
	INSERT	dbo.ProduitVendu
	(
		IDProd
		, QteProdVendu
	)
	VALUES
	(
		@_IDProd
		, @_QteProdVendu
	)
 
	DECLARE @last_IDCom_IDProd int
	SELECT	last_IDCom_IDProd = MAX(IDCom) AS last_IDCom_IDProd
	FROM	dbo.Commande
	WHERE	IDProd = @_IDProd
 
	UPDATE	dbo.Commande
	SET	QteProdRestante = CASE IDCom
					WHEN last_IDCom_IDProd THEN QteProdRestante - @_QteProdVendu
					ELSE 0
				END
	WHERE	IDProd = @_IDProd
END

@++

[iberserk]

Bref que le resultat de la soustraction (15+12+25)-21=31 soit place a un des champs de QteProdRestante et le reste des champs prennent la valeur o.

oui mais interêt?

[lepolo2oo8]

Envoyé par iberserk

oui mais interêt?

L interet ce que cette colonne me permet d' affiche le reste total des produit apres chaque vente et aussi du fait qu' apres une nouvelle insertion des commandes que celle-ci ne s' addition plus avec des restes deja vendus.
C' est ca tout l' interet de cette colonne...

[lepolo2oo8]

Il y a une erreur du type :Syntaxe incorrecte vers le mot clé 'AS'. . Comment resoudre ce probleme ?
Merci

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
BEGIN
	SET NOCOUNT ON
 
	INSERT	dbo.ProduitVendu
	(
		IDProd
		, QteProdVendu
	)
	VALUES
	(
		@_IDProd
		, @_QteProdVendu
	)
 
	DECLARE @last_IDCom_IDProd int
	SELECT	last_IDCom_IDProd = MAX(IDCom) AS last_IDCom_IDProd
	FROM	dbo.Commande
	WHERE	IDProd = @_IDProd
 
	UPDATE	dbo.Commande
	SET	QteProdRestante = CASE IDCom
					WHEN last_IDCom_IDProd THEN QteProdRestante - @_QteProdVendu
					ELSE 0
				END
	WHERE	IDProd = @_IDProd
END

[iberserk]

DECLARE @last_IDCom_IDProd int
SELECT last_IDCom_IDProd = MAX(IDCom) AS last_IDCom_IDProd
FROM dbo.Commande
WHERE IDProd = @_IDProd

Simple...le mot clé AS est interdit ici lors d'une affectation de variable

Code :

1
2
3
4
 
SELECT	last_IDCom_IDProd = MAX(IDCom)
FROM	dbo.Commande
WHERE	IDProd = @_IDPro

[lepolo2oo8]

Envoyé par iberserk

Simple...le mot clé AS est interdit ici lors d'une affectation de variable

Code :

mais le truc ce que s' il y a plus AS alors ca me donne : Nom de colonne non valide*: 'last_IDCom_IDProd'.

[iberserk]

mais le truc ce que s' il y a plus AS alors ca me donne : Nom de colonne non valide*: 'last_IDCom_IDProd'.

Car la colonne dans votre table commande ne s'appelle pas mais IDCom mais last_IDCom_IDProd...

D'où:

Code :

1
2
3
4
 
SELECT	last_IDCom_IDProd = MAX(AS last_IDCom_IDProd)
FROM	dbo.Commande
WHERE	IDProd = @_IDProd

Mais vous allez avoir un problème.... en effet je me doute que vous voulez assigner cette valeur à la variable @last_IDCom_IDProd...

Donc:

Code :

1
2
3
4
 
SELECT	@last_IDCom_IDProd = MAX(AS last_IDCom_IDProd)
FROM	dbo.Commande
WHERE	IDProd = @_IDProd

[lepolo2oo8]

Envoyé par iberserk

Code :

Ca pas changer ca donne tjour: Syntaxe incorrecte vers le mot clé 'as'.

[elsuket]

Je suis désolé pour mon erreur, mais on peut pas dire que vous cherchiez beaucoup ... :

Code :

1
2
3
SELECT	@last_IDCom_IDProd = MAX(IDCom)
FROM	dbo.Commande
WHERE	IDProd = @_IDProd

On ne peut pas aliaser une colonne dans une affectation de valeur à une variable.

@++

[iberserk]

L interet ce que cette colonne me permet d' affiche le reste total des produit apres chaque vente et aussi du fait qu' apres une nouvelle insertion des commandes que celle-ci ne s' addition plus avec des restes deja vendus.
C' est ca tout l' interet de cette colonne...

Je maintiens que c'est une valeur issue d'un calcul... faites une vue comme vous le préconise SQL PRO dans lequel vous faites le calcul 'en temps réel'

[elsuket]

Il est clair que c'est LA solution

@++