Discussion :

[sniperricko]

J'ai écris un trigger dans ma base SQL Server 2000 dont voici la syntaxe:

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
43
44
45
 
CREATE TRIGGER insert_stock
ON fMOUVEMENTS INSTEAD OF INSERT
AS
 
declare @id_article int
declare @id_vehicule int
declare @qte int
declare @stock_init int
declare @stock_final int
declare @sens varchar(6)
declare @date datetime
declare @obs varchar(255)
 
	BEGIN
		SELECT @id_article = ID_ARTICLE,  @sens = TYPE_MOUVEMENT, @qte = QTE_MOUVEMENT, @date = DATE_MOUVEMENT,  @id_vehicule = ID_VEHICULE, @obs = OBS_MOUVEMENT FROM inserted
 
		SET @stock_init = (SELECT QTE_STOCK From fARTICLES WHERE ID_ARTICLE = @id_article)
 
 
		IF @sens = 'Sortie'
			BEGIN
				IF  @stock_init < @qte 
					BEGIN
						RAISERROR ('La quantité que vous souhaitez déstockée est indisponible',0,1)
						ROLLBACK TRANSACTION
					END
				ELSE
					BEGIN
						SET @stock_final = @stock_init - @qte
						INSERT INTO fMOUVEMENTS (ID_ARTICLE, DATE_MOUVEMENT, QTE_MOUVEMENT, TYPE_MOUVEMENT, ID_VEHICULE, OBS_MOUVEMENT, STOCK_INITIAL, STOCK_FINAL)
						VALUES (@id_article, @date, @qte, @sens, @id_vehicule, @obs, @stock_init, @stock_final)
						COMMIT TRANSACTION
					END
 
			END
		ELSE
			BEGIN
				SET @stock_final = @stock_init + @qte
				INSERT INTO fMOUVEMENTS (ID_ARTICLE, DATE_MOUVEMENT, QTE_MOUVEMENT, TYPE_MOUVEMENT, OBS_MOUVEMENT, STOCK_INITIAL, STOCK_FINAL)
				VALUES (@id_article, @date, @qte, @sens, @obs, @stock_init, @stock_final)
				COMMIT TRANSACTION
			END
 
	END

J'ai dans ma base 2 tables (fARTICLES et fMOUVEMENTS) qui ont une dépendance. Visiblement la syntaxe du déclencheur est correcte mais à l'exécution j'ai le message de confirmation de mon insertion dans la table fMOUVEMENTS mais quand j'affiche les données de ma table, aucune insertion n'est faite.
Que puis-je faire pour retrouver la cause?

[tornade69]

Bonjour,

dans quel cas cela ne marche pas ?? Sortie ou Autre ??

A mon avis enlève les commit

Tornade

[SQLpro]

Vous utilisez un trigger INSTEAD OF ce qui signifie qu'il n'effectue pas l'insertion. Pour autant il liasse sciement croire qu'elle a eu lieu.

Si vous voulez que l'insertion se fasse malgré tout, deux solutions :
1) terminer par l'insertion
2) prendre un trigger AFTER et non INSTEAD OF

A +

[elsuket]

Bonjour,

En plus, votre trigger n'est pas ensembliste : pour plusieurs lignes insérées, que se passe-t-il à votre avis ?
Hé bien seule la dernière est traitée par votre trigger, pas les autres !

Autre chose dommage : si 1 mouvement sur les 25 lignes que vous venez d'insérer par exemple, est supérieur à la quantité en stock, alors les 24 qui sont correctes ne sont pas traitées. C'est un peu dommage, non ?

Voici ce que vous pouvez faire, par exemple :

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
CREATE TRIGGER insert_stock
	ON fMOUVEMENTS
AFTER INSERT
AS
BEGIN
	-- Ajoute les articles pour lesquels le mouvement est <= à la quantité en stock
	INSERT INTO dbo.fMOUVEMENTS
	(
		ID_ARTICLE,
		DATE_MOUVEMENT,
		QTE_MOUVEMENT,
		TYPE_MOUVEMENT,
		ID_VEHICULE,
		OBS_MOUVEMENT,
		STOCK_INITIAL,
		STOCK_FINAL
	)
	SELECT ID_ARTICLE,
		DATE_MOUVEMENT,
		QTE_MOUVEMENT,
		TYPE_MOUVEMENT,
		ID_VEHICULE,
		OBS_MOUVEMENT,
		STOCK_INITIAL,
		STOCK_FINAL
	FROM INSERTED
	WHERE ART.QTE_STOCK >= I.QTE_MOUVEMENT
	AND I.TYPE_MOUVEMENT = 'Sortie'
 
	-- Relève les articles pour lesquels la quantité en mouvement
	-- est supérieure à la quantité en stock
	DECLARE @szErrMsg VARCHAR(512)
	SELECT @szErrMsg = ISNULL(@szErrMsg, '') + CAST(ID_ARTICLE AS VARCHAR)+ ', '
	FROM dbo.fARTICLES ART
	JOIN INSERTED I ON I.ID_ARTICLE = ART.ID_ARTICLE
	WHERE ART.QTE_STOCK < I.QTE_MOUVEMENT
	AND I.TYPE_MOUVEMENT = 'Sortie'	
 
	RAISERROR('la quantité que vous souhaitez déstocker pour les articles d''ID %s n''est pas disponible', 0, 1, @szErrMsg)
END

Vous pouvez éventuellement faire une jointure sur la table des articles pour avoir leur nom plutôt que leur identifiant dans le libellé du message d'erreur.

@++