Discussion :

[Yanmeunier]

Bonjour,
J'ai fait un trigger qui controle la mise à jour de la colonne d'une ligne. Si cette mise à jour est effectué alors j'effectue une mise à jour sur d'autres lignes. Seulement quand je change la valeur sur lequel est attribué le trigger alors SQL me dit que un curseur est déjà ouvert.
Voila mon trigger:

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
 
CREATE TRIGGER PassageBF ON [dbo].[F_DOCLIGNE] 
FOR  UPDATE
AS
DECLARE @PasseBF NUMERIC(13), @Qte NUMERIC(13),@Type SMALLINT,@Piece VARCHAR(9),@DL_No INTEGER,@QteAvant NUMERIC(13),@Test INTEGER
 
SELECT @Type=DO_Type,@PasseBF=PassageBF,@Qte=DL_Qte,@Piece=DO_Piece,@DL_No=DL_No FROM Inserted
SELECT @QteAvant=DL_Qte FROM Deleted
Set @Test=0
IF @Type=25
BEGIN
	WHILE @Test=0
	BEGIN
		DECLARE curseurOF CURSOR 
		FOR SELECT DL_Qte FROM F_DOCLIGNE WHERE DL_No<>@DL_No AND DO_Piece=@Piece
 
		Open curseurOF
 
		FETCH NEXT FROM curseurOF
  				WHILE @@FETCH_STATUS = 0
  				BEGIN
					Set @Test=1
					UPDATE F_DOCLIGNE SET DL_Qte=(DL_Qte*@Qte)/@QteAvant FROM F_DOCLIGNE WHERE DL_No<>@DL_No AND DO_Piece = @Piece
  					COMMIT
					FETCH NEXT FROM CurseurOF;
  				END
		CLOSE curseurOF 
		DEALLOCATE curseurOF
	END
 
 
END

Merci

[Yanmeunier]

En faite je me suis rendu compte que ceci venait de mon UPDATE mais j'ai toujours la même erreur.

[Fabien Celaia]

Pourquoi un curseur dans le trigger ?
Pourquio ne traitez-vous pas votre set en ensembliste ?

[Yanmeunier]

Explique plus

[SQLpro]

1) vous pouvez utiliser IF UPDATE(DO_Type) pour limiter le trigger si aucune modif n'est intervenue sur cette colonne

2) un UPDATE dans un curseur portant sur les données de la table sous jacente au curseur doit se faire en utilisant un filtre WHERE CURRENT OF
<nom_curseur>. Sans cela vous desynchroniserez le jeu de données du curseur.

3) utiliser un curseur est toujours peu perfromant. Utiliser un trigger est très couteux. L'association trigger + curseur, est exécrable en terme de performances. Ajoutons à cela une vrai fausse boucle WHILE et on obtient des temps de réponse catastrophique. Évidemment le tout ne sera visible que lors de la montée en charge lorsque l'appli sera livrée et en exploit !...

Utilisez une logique ensembliste du genre (c'est à dire une requête) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
CREATE TRIGGER PassageBF ON [dbo].[F_DOCLIGNE]
FOR  UPDATE
AS
 
UPDATE F_DOCLIGNE 
SET    DL_Qte = DL_Qte * DL_Qte ) / i.DL_Qtet 
FROM   F_DOCLIGNE L
       INNER JOIN inserted i ON ??? --(clef de L = clef de i)
       INNER JOIN deleted  d ON ??? --(clef de L = clef de d)
WHERE  L.DO_Type = 25
AND    L.DL_No <> i.DL_No

Si vous voulez une formation SQL Server, je fais du cours standard comme à la carte !

A +