Discussion :

[Totanne]

Bonjour à tous.
J'ai créé la table suivante et (je m'en doutais un peu) le comportement escompté n'est pas au rendez-vous :

Code :

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

1
2
3
4
CREATE TABLE TEST
          (MaDate       SMALLDATETIME,
           MonNum      SMALLINT             IDENTITY(1,1),
           PRIMARY KEY (MaDate, MonNum))

Je voudrais qu'a chaque nouvelle date inssérée, s'associe une clé auto_incrémentée qui démarre a 1
Cela donnerait quelque chose comme ceci :

01/01/2007 -> 1
01/01/2007 -> 2
01/01/2007 -> 3
12/12/2008 -> 1
12/12/2008 -> 2
01/01/2007 -> 4

Est-ce faisable de manière automatique ?

[YoussefTS]

Bonjour,

Avec la syntaxe que tu as utilisé, le champ MonNum est un compteur pour toute la table et non pas pour chaque année.

Si tu veux mettre un compteur par année, il te faudra passer par un trigger à l'insertion et un autre à la mise à jour su jamais tu changes d'année sur ta date.

Si le principe est claire bonne chance, sinon si tu as besoin d'aide n'hésite pas à demander.

[Totanne]

Salut Youssef,

Je suis un parfait débutant SQL, je te remercie donc de m'avoir dirigé vers les Triggers.

J'ai donc tatonné et essayé plusieurs trucs avant de pondre ceci :

Le code ci-dessous marche mais uniquement lorsque MaDate est déja présente avec en face un MonNum non Null :

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
CREATE TABLE TEST
          (MaDate       INT    NOT NULL,
           MonNum      INT)
 
ALTER TRIGGER TRIGTEST 
ON dbo.TEST 
INSTEAD OF INSERT
AS 
BEGIN
	DECLARE @Num INT
	SET @Num = 1 + (SELECT MAX(TEST.MonNum) 
					FROM TEST 
					INNER JOIN INSERTED
					ON TEST.MaDate = INSERTED.MaDate) 
 
	INSERT INTO TEST(MonNum,MaDate)
	SELECT @Num,INSERTED.MaDate FROM INSERTED
END

Pour que cela fonctionne completement il me faudrait entre le SET et le INSERT un code du style :
SI @MonNum EST NUL ALORS @MonNum = 1
J'ai essayé plusieures syntaxes mais n'y connaissant rien je rame ...
Je ne désespère pas car je me dit qu'a force de ramer, je trouverais peut-être de l'aide de l'autre coté de la Méditéranée
Youssef ... si tu m'entends ...

[Totanne]

J'ai essayé pas mal de trucs ...
If NOT @Num ISNULL : Marche Pas.
If @Num = 0 : Chpas.
If @Num < 1 : Toujours pas
If EXISTS(@Num) : ...Et Merde!...
IF @Num IS NULL : Bingo (Encore un et j'abandonnais ...)

Donc Résolu ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
ALTER TRIGGER TRIGTEST 
ON dbo.TEST 
INSTEAD OF INSERT
AS 
BEGIN
	DECLARE @Num INT
	SET @Num = 1 + (SELECT MAX(TEST.MonNum) 
					FROM TEST 
					INNER JOIN INSERTED
					ON TEST.MaDate = INSERTED.MaDate) 
	IF @Num IS NULL SET @Num = 1
	INSERT INTO TEST(MonNum,MaDate)
	SELECT @Num,INSERTED.MaDate FROM INSERTED
END

[SQLpro]

Le SELECT MAX(...) + 1 est une TRES MAUVAISE IDÉE !!!

En effet cela va marcher au début quand votre table va être vide et votre SGBD ne connaître que peu de clients.

Mais au fur et à mesure de la montée en charge cela va prendre de plus en plus de temps et faire attendre tous les autres utilisateurs. En effet pour calculer un max il faut un verrou sur toutes les lignes de la table. AU début cela prendra peu de temps. Lorsque vous aurez beaucoup de lignes cela mettra beaucoup de temps.
De plus en contexte de forte concurrence il se peut que vous ayez les même n° de ligne => violation de clef. En effet entre votre requête de calcul du max et l'insert, un autre trigger peut se déclencher en parrallèle. Il faudrait au moins gérer la transaction au niveau d'isolation SERIALIZABLE. Autrement dit encore plus de blocage en prévision.

Bref si vous voulez pourrir votre base, c'est le code qu'il faut. Si en revanche vous voulez des performances, il faut utiliser une table de compteurs comme indiqué dans l'article que j'ai écrit.
http://sqlpro.developpez.com/cours/clefs/
Rajoutez simplement dans cette table de comptage la colonne MaDate.

A +