Discussion :

[warri0r21]

Bonjour à tous,

Voici mon soucis : j'aimerais créer une copie de la table dboutillage.dbo.Table_correspondance_nuance vers une autre base de données dbfilage.dbo.NUANCESPU

Voici 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
Create Trigger dbo.NUANCESPUTrigger ON dbo.Table_correspondance_nuance After Insert, Update
As
Begin
	DECLARE @ID as int
	DECLARE @NUANCE as NVARCHAR(50)
	DECLARE @CULOT as NVARCHAR(50)
	DECLARE @GRADE as NVARCHAR(50)
 
	SET @ID = (SELECT [ID Correspondance] FROM inserted)
	SET @NUANCE = (SELECT [Nuance SPU] FROM inserted)
	SET @CULOT = (SELECT [Nuance CULOT] FROM inserted)
	SET @GRADE = (SELECT [Engineering grade] FROM inserted)
 
If NOT Exists(select d.ID from dbfilage.dbo.NUANCESPU d where d.ID in (Select @ID from inserted))
    BEGIN
      INSERT INTO dbfilage.dbo.NUANCESPU VALUES (@ID,@NUANCE,@CULOT,@GRADE)
	END
   ELSE
	BEGIN           
		UPDATE  tab 
		SET  tab.NUANCESPU = @NUANCE,
		SET  tab.NUANCECULOT = @CULOT,
		SET  tab.ENGINEERINGGRADE = @GRADE,
		WHERE tab.ID = @ID
   END
 
END

J'ai une erreur de syntaxe dans le UPDATE mais par ailleurs j'aimerais avoir si c'est possible et si de base je pars dans la bonne direction.

EDIT : l'insert marche niquel.

[mikedavem]

Hello,

De base tu ne pars pas bien car les triggers sont ensemblistes par nature. Le code que tu proposes ne l'est pas.

J'aurais également tendance à séparer en 2 triggers distincts pour chaque type d'action INSERT et UPDATE pour plus de lisibilité et de maintenance.

++

[aieeeuuuuu]

Bonjour

Bonjour à tous,

Voici mon soucis : j'aimerais créer une copie de la table dboutillage.dbo.Table_correspondance_nuance vers une autre base de données dbfilage.dbo.NUANCESPU

Dans quel but ? Ne serait-ce pas plus simple de créer une vue dans la base cible ?

Voici mon trigger :
[...]
J'ai une erreur de syntaxe dans le UPDATE

En effet, le mot clef SET ne doit être spécifié qu'une seul fois. séparez ensuite vos colonnes par une virgule.
ça, c'est pour l'erreur de syntaxe sur la commande UPDATE mais je vous renvoi à la remarque de mikedavem concernant le coté non ensembliste de votre trigger : si plusieurs lignes sont mises à jour ou insérées en même temps, votre trigger ne traitera au mieux qu'une ligne, voire pire : il mélangera les données entre les lignes !

Cependant, pour ma part, j'aurai bien fait un seul trigger pour INSERT et UPDATE, mais utilisé une commande MERGE pour répercuter les modifications dans la table cible.

mais par ailleurs j'aimerais avoir si c'est possible et si de base je pars dans la bonne direction.

Pas vraiment, mais je crois que vous l'aviez compris

Détaillez un peu plus le contexte afin que l'on puisse mieux vous aider. (cf mes deux premières questions)

[mikedavem]

Cependant, pour ma part, j'aurai bien fait un seul trigger pour INSERT et UPDATE, mais utilisé une commande MERGE pour répercuter les modifications dans la table cible.

J'avais pensé au MERGE également mais je me suis retrouvé souvent dans des cas où les règles businees évoluaient tellement en cours de projet que la commande MERGE ne suffisait pas à répondre aux besoins .. du coup on finissait par séparer de toute façon le code de l'INSERT et de l'UPDATE pour plus de flexibilité de maintenance.

Mais effectivement l'idée du MERGE est une très bonne idée

++

[warri0r21]

J'ai changé de méthode, je fais tout dans la même base de données. Mais cela ne change pas trop je cherche en cas d'update modifier un paramètre.

j'explique mon problème.

j'ai deux tables : une table Table_liste_nuance_SPU et une table Table_correspondance_nuance

Table_liste_nuance_SPU:
ID (PK int)
Nuance SPU (nvarchar(50))

Table_correspondance_nuance:
ID (PK int)
ID NUANCE SPU (FK relié à Table_liste_nuance_SPU, int)
NUANCE SPU (nvarchar(50))

J'aimerai, en fait, lorsque l'utilisateur modifie Nuance SPU dans la table Table_liste_nuance_SPU cela affecte la table Table_correspondance_nuance car ces colonnes ne sont pas reliés dans SQL (seulement leurs ID).

J'ai tenté de faire un trigger d'update mais le multi row ne marche pas :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
ALTER TRIGGER [dbo].[NUANCESPU]
   ON  [dbo].[Table_liste_nuance_SPU]
   FOR UPDATE
AS 
	UPDATE dboutillage.dbo.Table_correspondance_nuance
	SET dboutillage.dbo.Table_correspondance_nuance.[Nuance SPU] = i.[Nuance SPU] 
	FROM dboutillage.dbo.Table_correspondance_nuance t2
	INNER JOIN inserted i
	ON t2.[ID Nuance SPU] = i.[ID Nuance SPU]
 
GO

Merci à vous

[aieeeuuuuu]

a quoi sert la table Table_correspondance_nuance ?

Pourquoi y remettre la colonne NUANCE SPU (nvarchar(50)) ?

[warri0r21]

La table correspondance possède d'autres paramètres :

Table_correspondance_nuance:
ID (PK int)
ID NUANCE SPU (FK relié à Table_liste_nuance_SPU, int)
NUANCE SPU (nvarchar(50))
IDCULOT (FK int)
NUANCE CULOT(nvarchar(25))
ENGINEERING GRADE (nvarchar(25))

Ce sont des tables de paramètres ou un utilisateur via une appli pourra choisir une nuance (datagridview). Ces tables sont utilisées pour effectuer des calculs.
Un programme calcul va lire ses tables et va récupérer la nuance (par exemple) pour faire ses calculs.

Donc j'ai besoin de réaffecter la nuance dans cette table et non pas uniquement l'ID

[aieeeuuuuu]

Donc j'ai besoin de réaffecter la nuance dans cette table et non pas uniquement l'ID

C'est précisément sur ce point surtout que vous ne partez pas dans la bonne direction : vous avez un problème de modélisation !

Si je suis votre raisonnement, vous allez mettre en place les même déclencheur pour la table Nuance_culot...

En faisant ainsi, non seulement vous vous compliquez la vie (mais ça vous etes en train de vous en rendre compte ) mais en plus vous augmentez artificiellement le volume de votre base, les opération d'io.... Bref, a terme, cette solution sera impossible a maintenir, et les performances vont vite être catastrophiques.

Votre table Table_correspondance_nuance ne devrait contenir que les identifiants des NUANCE SPU et CULOT.

Lorsque votre programme viendra chercher des données dans cette table, il lui suffira de faire une jointure avec les tables référencées pour obtenir les libellés. Vous pouvez aussi créer une vue qui contient ces informations, et votre programme fera une simple requête dans cette vue (qui aurait a peu près la même structure que votre table actuelle).

[warri0r21]

Rebonjour,

Merci pour cette réponse rapide ! Certes je savais que je n'étais pas dans la bonne direction et à vrai dire j'avais pensé a faire des vues pour mieux structurer mes tables.Et pourtant j'en ai fais de la modélisation... surtout avec merise. Bref mon ancienne bdd/appli (ma toute première ) est schématisée comme ça mais je ne referais plus cette "erreur".
Même si ça demande plus de temps en bdd, e suis parti sur le droit chemin.

Merci sujet résolu

[aieeeuuuuu]

Même si ça demande plus de temps en bdd, e suis parti sur le droit chemin.

Qu'est-ce qui demande plus de temps ?

[warri0r21]

J'avais au préalable déjà défini le programme pour cette ancienne modélisation. Etant donné que je change tout, je vais prendre plus de temps coté BDD. Mais mon code et ma bdd seront plus propres