Discussion :

[egautier18]

Bonjour à tous.
J'ai un problème lors de mes udates/deletes sur mes déclencheurs.

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
 
 
IF @EtatTrigger = 2 -- Dans le cas d'un Update. 
BEGIN
 
	SET @MyReqInstruction = 'UPDATE MyTable '
	SET @MyReqValue = 'SET '
 
	If (SELECT CHAMP1 FROM DELETED) <> (SELECT CHAMP1 FROM INSERTED)
	BEGIN
		IF len(@MyReqValue) > 4 SET @MyReqValue = (@MyReqValue + ', ')
		SET @MyReqValue += ('CHAMP1 = ' + QUOTENAME((SELECT CHAMP1 FROM INSERTED),''''))
	END
 
	If (SELECT CHAMP2 FROM DELETED) <> (SELECT CHAMP2 FROM INSERTED)
	BEGIN
		IF len(@MyReqValue) > 4 SET @MyReqValue = (@MyReqValue + ', ')
		SET @MyReqValue += ('CHAMP2 = ' + QUOTENAME((SELECT CHAMP2 FROM INSERTED),''''))
	END
 
	If (SELECT CHAMP3 FROM DELETED) <> (SELECT CHAMP3 FROM INSERTED)
	BEGIN
		IF len(@MyReqValue) > 4 SET @MyReqValue = (@MyReqValue + ', ')
		SET @MyReqValue += ('CHAMP3 = ' + QUOTENAME((SELECT CHAMP3 FROM INSERTED),''''))
	END
 
...
...
...
END

Enfait, lorsque mon déclencheur capte une requête, je veux la réécrire (car impossible à capter directement si elle provient d'une requête paramétré..) et la stocker dans une autre table.

Cependant, dans le cas ou il y'a un Update sur plusieurs rows (une seule instruction donc) cela pose problème.

J'aimerais donc idéalement faire un "for each" visiblement impossible en sql et ne tiens pas à utiliser les curseurs.

Mes recherches se sont révélés infructueuses, si quelqu'un à une idée..
Merci à vous.

A+

[amedee]

Bonjour,

Dans ce cas, dans le trigger, il faut trouver un moyen pour réaliser le traitement de manière ensembliste, si possible en une seule requête et des CASE WHEN.

Cordialement.

[SQLpro]

Que cherchez-vous à faire au juste ???

A +

[egautier18]

Bonjour,
Merci à vous de répondre.

Enfait ce que j'aimerais c'est :

Si j'ai 3 update sur MaTable1
Mon triggerTable1 écris 3 requêtes updates dans MaTableLOG avec d'autre info (Ref Clef primaire, DateRequete, NomTable, TypeInstruction...)

(Et je dois également écrire une requêtes avec uniquement les champs qui sont modifiés)..

Si j'ai une requête qui update 1 row. Tout vas bien..
Si j'ai une requête qui update X rows. Tout vas mal..

[amedee]

Pour compléter ce que j'ai dit :
dans le trigger, il faut faire une seule requête avec des CASE WHEN et des jointures sur INSERTED et DELETED

Voilà un exemple à adapter à votre situation :

Code :

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

1
2
3
4
5
6
7
8
 
UPDATE MaTable 
SET MaTableChampX = CASE WHEN INSERTED.MaTableChampX = 1 THEN MaValeur ELSE 0 END           
FROM MaTable
INNER JOIN INSERTED ON INSERTED.MaTableChamp1 = MaTable.MaTableChamp1
INNER JOIN DELETED ON DELETED.MaTableChamp1 = MaTable.MaTableChamp1
INNER JOIN Table2 ON INSERTED.Table2Champ1 = Table2.Table2Champ1 
WHERE DELETED.Table2Champ1 <> INSERTED.Table2Champ1

[egautier18]

Je ne sais pas si je passe complètement à côté ou si cela n'est pas adaptable mais je ne vois pas du tout comment m'inspirer de la requête que vous m'avez donné afin d'avoir en résultat des requêtes écrites (non exécutée, mais stocké dans un champ)

Et je ne vois pas non plus sur quel critère je peux boucler afin de jouer sur mes X enregistrements..

J'avais pensé à jouer avec ROW_NUMBER() mais il ne me sort pas vraiment ce à quoi je m'attendais..

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
DECLARE @MyRowCount int	= 3
DECLARE @UpUp int = 1
 
WHILE (@UpUp <> @MyRowCount)
BEGIN
	SET @MyRequete = 'DELETE MyTable WHERE CHAMP1 = ' + QUOTENAME((SELECT CHAMP1 FROM DELETED where (SELECT ROW_NUMBER() OVER (ORDER BY CLEF1, CLEF2, CLEF3, CLEF4) AS rownumber FROM DELETED) = @UpUp),'''')
..
.. 
..
 
 
        SET @UpUp +=1
END

Si @UpUp = 1, il me récupère toutes les rows de la table (donc bug de nouveau),
Si il est > 1, il ne trouve aucun résultat..

[EDIT]
Désolé, je vous ai mis mes test dans le cas d'un DELETE, avec une seule table temporaire à gérer, dans un premier temps..