Discussion :

[Banzai31]

Bonjour,

Je souhaite faire un trigger, qui puisse empêcher la création de ligne en double dans une table.

Je vous met le code du trigger car je n'arrive pas a voir d'ou vient le problème (le trigger renvoi tout le temps l'erreur de doublon) si quelqu'un peut m'aider svp

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
 
CREATE TRIGGER Manage_Duplicate ON TABLE
AFTER INSERT
AS
 
DECLARE @IsDuplicate char(50)
DECLARE @Cpte char(3)
 
BEGIN
SET @IsDuplicate = (SELECT NOM_CHAMP FROM INSERTED)
 
SELECT TOP 1 @Cpte= NOM_CHAMP FROM NOM_TABLE WHERE NOM_CHAMP=@IsDuplicate 
 
IF @Cpte is not NULL
BEGIN
RAISERROR ('This row already exists in the table', 16, 1, @Cpte)
ROLLBACK TRANSACTION
END
 
END

[aieeeuuuuu]

Bonjour,

1/ la pseudo table INSERTED peut contenir plusieurs ligne : votre trigger ne le prend pas en compte.
2/ vous avez fait un trigger AFTER insert : les données que vous êtes en train d'insérer sont déjà dans la table, c'est pourquoi vous avez toujours votre message d'erreur.
3/ enfin et surtout : pourquoi faire un trigger là où une simple contrainte d'unicité fait l'affaire ?

[Invité]

3/ enfin et surtout : pourquoi faire un trigger là où une simple contrainte d'unicité fait l'affaire ?

Pas mieux !

[Banzai31]

Bonjour,

Merci pour vos suggestions.
J'ai testé la contrainte d'unicité, mais je n'arrive pas à gérer les NULL, qui sont détectés en tant que doublons, d'ou ma demande sur le trigger.

Je n'ai pas trouvé mais y a t'il un moyen d'éviter le test d'unicité sur une valeur NULL ?

Code :

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

1
2
3
4
5
6
 
ALTER TABLE [dbo].[TableTest1] ADD CONSTRAINT [UQ_FIELD] UNIQUE  NONCLUSTERED 
	(
		[FIELD]
	)  ON [PRIMARY] 
GO

[aieeeuuuuu]

C'est en effet un comportement étrange de SQL server que de considérer les null comme des doublons.

Pour contourner, il faut faire un index unique filtré :

Code :

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

1
2
3
4
 
CREATE UNIQUE INDEX UQ_FIELD
    ON [dbo].[TableTest1] ([FIELD])
    WHERE [FIELD] IS NOT NULL

[Banzai31]

Effectivement c'est une bonne idée.
Malheureusement, j'ai l'impression que le WHERE ne passe pas quand on fait un CREATE UNIQUE INDEX.
Il me renvoi "Incorrect syntax near the keyword 'WHERE'"

[aieeeuuuuu]

En effet, je n'avais pas vu que vous étiez en version 2000. Les index filtrés n'y sont pas implémentés.

Vous pouvez modifier votre modélisation, et mettre cette colonne dans une nouvelle table référençant votre table actuelle. Ainsi, plus de NULL, donc plus de problème.

[SQLpro]

Pour répondre à votre problème il existe plusieurs solutions en version 2000.
Le trigger en est une, une colonne calculée persistante en est une autre.

Lisez l'article que j'ai écrit à ce sujet :

http://blog.developpez.com/sqlpro/p5..._norme_sql_iso

A +