trigger clé primaire

Version imprimable

Bonjours,

Je suis en face d'un grand mystère!8O

J'ai 2 tables :
- employe(nemp,salemb...)
- histo_salaire(champs: nemp, datsal,salemb)

Je dois faire un trigger qui:
Lorsque je change le salaire d'un employe, j'historise ce salaire dans la table histo_salaire, il doit y avoir au max 10 enregistrement dans table histo sal qui concerne cet employe.
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 create trigger histo_10_salaire_v2 on employe for update as declare @noemp int declare @date datetime declare @sal float if update(salemb) begin select @noemp=nemp from Deleted select @sal=salemb from Deleted while (select count(*)from histo_salaire where nemp=@noemp)>10 --si appliquée à une BD vieille begin select @date=min(datsal) from histo_salaire where nemp=@noemp delete from histo_salaire where datsal=@date and nemp=@noemp end insert into histo_salaire values(@noemp,getdate(),@sal) end
Jusque là tout a l'air de marcher!!
mais quand j'execute,

Code:

update employe set salemb=40000 where nemp=1

si cela marche (une fois sur 10), il m'insert 2 fois mon employe dans la table histo_salaire, sinon me marque cette erreur:

(1 ligne(s) affectée(s))

Serveur : Msg 2627, Niveau 14, État 1, Procédure histo_10_salaire_v2, Ligne 22
Violation de la contrainte PRIMARY KEY 'PK__histo_salaire__023D5A04'. Impossible d'insérer une clé en double dans l'objet 'histo_salaire'.
L'instruction a été arrêtée.

je ne comprends pas pourquoi il me parle de clé primaire alors que ma table histo_salaire a comme clé nemp et datsal (datetime)!!

merci d'avance

claire

07/11/2008, 23h05
claire13

Après de nombreux test, il me semble que c'est un pb de date.
En effet ma clé 1aire est nemp et datsal.
Or l'insertion est refusée casi à chaque fois, dès fois oui je sais pas pourquoi!!!

pourtant ma datal est de type datetime, donc à chaque fois que je veux faire un insert, l'heure n'est pas la même!!

si quelqu'un a une idée:roll:

merci encore d'avance

claire

Bonjour,

Votre trigger n'est pas ensembliste : que se passera-t-il le jour ou vous mettrez à jour plusieurs salaires ? Hé bien seul le dernier employé affecté par votre mise à jour bénéficiera du traitement de votre trigger, et pas les autres.

Vous avez écrit un traitement itératif dans votre trigger, qui est lent, alors que vous pouvez écrire votre trigger de façon ensembliste :

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
32
33
34
35
36
37
38
39
40
41
CREATE TRIGGER histo_10_salaire_v2
	ON employe
FOR UPDATE
AS
BEGIN
	IF UPDATE(salEmb)
	BEGIN
		-- Expression de table commune qui récupère la liset des employés que l'on vient
		-- de mettre à jour et qui ont plus de 10 lignes dans dbo.histo_salaire
		WITH
			CTE_Emp_10_Histo AS
			(
				SELECT nEmp,
					MIN(datSal) minDatSal,
					COUNT(*) nb
				FROM dbo.histo_salaire H
				JOIN DELETED D ON D.nemp = H.nemp
				GROUP BY nemp
				HAVING COUNT(*) > 10
			)
		-- Suppression des employés trouvés
		-- dans l'expression de table commune
		DELETE FROM dbo.Histo_Salaire
		FROM dbo.Histo_Salaire H
		JOIN CTE_Emp_10_Histo CTE
			ON H.nEmp = CTE.nEmp
			AND H.datSal = CTE.minDatSal
 
		-- Ajout de l'historique pour tous les employés
		-- dont le salaire vient d'être mis à jour
		INSERT INTO dbo.Histo_Salaire
		(
			nEmp,
			datSal,
			salEmb
		)
		SELECT D.nEmp,
			GETDATE(),
			D.sal
	END
END

@++ ;)