Discussion :

[PHPkoala]

Bonjour,
je me suis à la recherche d'une solution pour arriver à faire un trigger.

j'ai trois table,

table1 (ID,ID_devise,ID_Siren)
table2 (ID_devise,Devise)
table3 (ID_Siren,Siren)

j'ai réaliser un constraint sur ID_siren et ID_Devise.

lorsque je réalise une requete de select ca fonctionne

Code :

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

1
2
3
4
5
6
7
 
select * from table1
inner join table2 
	on dbo.table2.ID_DEVISE = dbo.table1.ID_devis
inner join  dbo.SIREN
	on dbo.table3.ID_SIREN = dbo.table1.ID_SIRENT
where DEvis = 'EUR' and SIRENT = '999999999'

mais lorsque je tente de faire un trigger de ce style là

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
CREATE TRIGGER teste_tab ON table1 after INSERT as
 
BEGIN
 
IF (DEvis = 'EUR' and SIRENT = '999999999' from dbo.table1
inner join dbo.DEVIS 
	on dbo.table2.ID_DEVIS = dbo.table1.ID_devis
inner join  dbo.SIREN
	on dbo.table1.ID_SIREN = dbo.table3.ID_SIRENT)
 
set typeDEVISE = '1';
 
 
elseif (dbo.DEVIS.DEvis = 'EUR' and dbo.SIREN.SIRENT <> '999999999' from dbo.table1
inner join dbo.DEVIS 
	on dbo.table2.ID_DEVIS = dbo.table1.ID_devis
inner join  dbo.SIREN
	on dbo.table1.ID_SIREN = dbo.table3.ID_SIRENT)
 
set typeDEVISE = '2';
 
else set typeDEVISE = '3';
 
END if;
END

en faite il aime pas du tout le from dans if mais j'ai vraiment besoin de déclarer les jointure pour pouvoir récupérer les valeurs de devis et siren.
Auriez vous une idéee?

Merci de votre aide,
Cordialement.

[elsuket]

Bonjour,

Effectivement pour tout FROM il faut un SELECT.
Ensuite en SQL Server les triggers sont ensemblistes.

Vous auriez donc du écrire :

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
31
32
33
34
35
36
37
38
39
40
41
CREATE TRIGGER teste_tab
	ON dbo.table1
	AFTER INSERT
AS 
BEGIN
	UPDATE dbo.Table1
	SET typeDEVISE = '1'
	FROM dbo.Table1 AS T
	JOIN INSERTED AS I
		ON T.PK_ID = I.PK_ID
	JOIN dbo.DEVIS AS D
		ON D.ID_DEVIS = T.ID_DEVIS
	JOIN dbo.SIREN AS S
		ON T.ID_SIREN = S.ID_SIRENT
	WHERE D.DEvis = 'EUR'
	AND S.SIRENT = '999999999'
 
	UPDATE dbo.Table1
	SET typeDEVISE = '2'
	FROM dbo.Table1 AS T
	JOIN INSERTED AS I
		ON T.PK_ID = I.PK_ID
	JOIN dbo.DEVIS AS D
		ON D.ID_DEVIS = T.ID_DEVIS
	JOIN dbo.SIREN AS S
		ON T.ID_SIREN = S.ID_SIRENT
	WHERE D.DEvis = 'EUR'
	AND S.SIRENT <> '999999999'
 
	UPDATE dbo.Table1
	SET typeDEVISE = '3'
	FROM dbo.Table1 AS T
	JOIN INSERTED AS I
		ON T.PK_ID = I.PK_ID
	JOIN dbo.DEVIS AS D
		ON D.ID_DEVIS = T.ID_DEVIS
	JOIN dbo.SIREN AS S
		ON T.ID_SIREN = S.ID_SIRENT
	WHERE D.DEvis <> 'EUR'
	AND S.SIRENT <> '999999999'
END

Il aurait peut-être mieux valu écrire un trigger INSTEAD OF INSERT ...

@++

[PHPkoala]

elsuket je te remerci infiniment .

Par contre pourquoi un insted of insert ?

[elsuket]

Parce qu'avec le trigger que je viens de vous donner, vous mettez les données à jour après les avoir insérées dans la table, alors que vous pourriez filtrer le données directement avant de les insérer

@++

[SQLpro]

Didons elsuket, tu t'es pas foulé sur le 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
CREATE TRIGGER teste_tab
	ON dbo.table1
	AFTER INSERT
AS 
BEGIN
	UPDATE dbo.Table1
	SET typeDEVISE = CASE 
	                    WHEN D.DEvis = 'EUR'
	                         AND S.SIRENT = '999999999' 
	                       THEN '1'
	                    WHEN D.DEvis = 'EUR'
	                         AND S.SIRENT <> '999999999'
	                       THEN '2' 
	                    WHEN D.DEvis <> 'EUR'
	                         AND S.SIRENT <> '999999999'
	                       THEN '3' 
	FROM dbo.Table1 AS T
	JOIN INSERTED AS I
		ON T.PK_ID = I.PK_ID
	JOIN dbo.DEVIS AS D
		ON D.ID_DEVIS = T.ID_DEVIS
	JOIN dbo.SIREN AS S
		ON T.ID_SIREN = S.ID_SIRENT
 
END

A +

[elsuket]

Effectivement je ne me félicite pas

@++

[Sergejack]

Honte à toi elsuket, tu viens de suivre une consigne d'optimasation donnée par SQLPro (dans son article sur les optimisation (CASE)).
Heureusement que.. SQLPro (?!) est là pour remettre... les... pen... ... mouais.