Discussion :

[Duck_ArMyx]

Bonjour,

j'essaie de faire un trigger qui quand j'ajoute une vue dans ma table, celui-ci change la valeur d'un champs selon le champs d'une autres table qui est en relation plusieurs a plusieurs je vous mets mon code pour le moment avec l'erreur

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
 
CREATE  TRIGGER ajout_mention     
   AFTER INSERT 
   ON produit     
   FOR EACH ROW  
   Begin
     	If  risque.LibelleRisque = 	'H200'OR 'H201'OR 'H202'OR 'H203'OR 'H205'OR 'H224'OR 'H225'OR 'H220'OR 'H222'OR 'H270'OR 'H271'OR 'H240'OR 'H241'OR 'H250'OR 'H251'OR 'H300'OR 
                                                'H301'OR 'H310'OR 'H330'OR 'H311'OR 'H331'OR 'H370'OR 'H334'OR 'H318'OR 'H304'OR 'H372'OR 'H350'OR 'H340'OR 'H360' 										
        Then
           	update produit SET mention = 2;
    	End if;
	If  risque.LibelleRisque =	'H204'OR 'H226'OR 'H221'OR 'H223'OR 'H252'OR 'H290'OR 'H280'OR 'H281'OR 'H302'OR 'H312'OR 'H332'OR 'H371'OR 'H335'OR 'H336'OR 'H315'OR 'H319'OR 
                                                'H317'OR 'H373'OR 'H351'OR 'H341'OR 'H361'OR 'H400'OR 'H410'OR 'H420'Then									
		update produit SET mention = 1;
	End if;
  End ;

et MySQL me répond : #1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 9

je sais même pas si ce que je fais est possible, si oui pouvez vous me dire ou sont mais problème

merci d'avance pour vos réponse

[al1_24]

L'erreur est ici :

Code :

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

If  risque.LibelleRisque = 	'H200'OR 'H201'OR...

.
Cette syntaxe est totalement erronée et ne peut être comprise par l'interpréteur.
Il faudrait écrire :

Code :

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

If  risque.LibelleRisque = 'H200' OR risque.LibelleRisque = 'H201' OR risque.LibelleRisque = ...

ou

Code :

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

If  risque.LibelleRisque IN ('H200', 'H201', ...)

[Duck_ArMyx]

Merci d'avoir répondu,
J'ai donc essayé avec votre deuxième syntaxe mais l'erreur reste la même, elle intervient au niveau du UPDATE

[isabelle.letrong]

Bonjour Duck_ArMyx

Erreur 1 :
Tu es dans un trigger AFTER : il est impossible de mettre à jour la table cible du trigger dans un trigger AFTER (puisque la mise à jour a déjà été faite ....)
=> il faut utiliser un trigger BEFORE

Erreur 2 :
la table cible du trigger (la table 'produit' référencée après la clause ON) ne se met pas à jour par un UPDATE mais par SET NEW.colonneCible = ...)
Donc ici il faut remplacer 'update produit SET mention = 2;' par 'SET NEW.mention = 2'

Bien cordialement.

[Duck_ArMyx]

Merci mais malheureusement même avec ces changements le trigger ne fonctionne toujours pas donc je pense modifier ma bdd et l'utilisateur devras rentré les mentions manuellement via un formulaire

MySQL a répondu:
#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 13

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
CREATE  TRIGGER ajout_mention     
	BEFORE INSERT 
   ON produit     
   FOR EACH ROW  
   Begin
		If  risque.LibelleRisque IN ('H200', 'H201', 'H202', 'H203', 'H205', 'H224',
 									'H225', 'H220', 'H222', 'H270', 'H271', 'H240', 
 									'H241', 'H250', 'H251', 'H300', 'H301', 'H310', 
									'H330', 'H311', 'H331', 'H370', 'H334', 'H318',
 									'H304', 'H372', 'H350', 'H340', 'H360')
 
 			Then
          SET NEW.mention = 2;
   	End if;
		If  risque.LibelleRisque IN	('H204', 'H226', 'H221', 'H223', 'H252', 'H290',
 									'H280', 'H281', 'H302', 'H312', 'H332', 'H371',
 									'H335', 'H336', 'H315', 'H319', 'H317', 'H373',
 									'H351', 'H341', 'H361', 'H400', 'H410', 'H420')
		Then									
			SET NEW.mention = 1;
		End if;
  End ;

[escartefigue]

bonjour,

Vu le nombre de valeurs à tester, il serait préférable de créer une table associant les listes de valeurs 'H200', 'H201' etc... au code '1' ou '2' recherché, ce sera plus facile à maintenir

[Duck_ArMyx]

bonjour

donc si je comprend le résonnement il faudrait que je relie ma table risque et mention ?

[skuatamad]

Qu'est-ce que RISQUE.LIBELLERISQUE ?
Ça n'a pas l'air de venir de la table PRODUIT, ça n'est donc pas exploitable directement depuis le trigger, il faut faire une requête pour récupérer les données associées de la table RISQUE.

[Duck_ArMyx]

encore merci de répondre et de m'aider dans ce problème

risque.LibelleRisque est le champ LibelleRisque de la table risque qui est en relation plusieurs à plusieurs avec la table produits via la table avertir.

Du coup je dois faire une jonction inner join ? si oui ou dois-je la faire avant ou dans le trigger ?

[al1_24]

Dans le trigger... Sinon, comment serait-elle liée au travers de la requête INSERT ?

[isabelle.letrong]

Effectivement :

Code :

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

1
2
3
4
5
 
If  risque.LibelleRisque IN	('H204', 'H226', 'H221', 'H223', 'H252', 'H290',
 									'H280', 'H281', 'H302', 'H312', 'H332', 'H371',
 									'H335', 'H336', 'H315', 'H319', 'H317', 'H373',
 									'H351', 'H341', 'H361', 'H400', 'H410', 'H420')

n'est pas du SQL !!!!

J'imagine qu'il faut faire une jointure, une description des trois tables nous aiderait

[Duck_ArMyx]

voila les table concerné il n'ont pas d'autre relation a part produit

[Duck_ArMyx]

c'est bon j'ai réussi donc j'ai juste pris 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
Begin
		If  risque.LibelleRisque IN ('H200', 'H201', 'H202', 'H203', 'H205', 'H224',
 									'H225', 'H220', 'H222', 'H270', 'H271', 'H240', 
 									'H241', 'H250', 'H251', 'H300', 'H301', 'H310', 
									'H330', 'H311', 'H331', 'H370', 'H334', 'H318',
 									'H304', 'H372', 'H350', 'H340', 'H360')
 
 			Then
          SET NEW.mention = 2;
   	End if;
		If  risque.LibelleRisque IN	('H204', 'H226', 'H221', 'H223', 'H252', 'H290',
 									'H280', 'H281', 'H302', 'H312', 'H332', 'H371',
 									'H335', 'H336', 'H315', 'H319', 'H317', 'H373',
 									'H351', 'H341', 'H361', 'H400', 'H410', 'H420')
		Then									
			SET NEW.mention = 1;
		End if;
  End ;

et je l'ai créée manuellement via le déclencheur mysql encore merci de m'avoir aider et comme je dis je vous aime tous car tout le monde est unique donc on est fort dans notre domaine

[escartefigue]

Du coup puisque votre problème est résolu, j'en profite pour aborder un autre point : le choix des types de données.

Du varchar pour des quantités et montants c'est complètement inadapté (poids, qte_totale, qte_restante, masse_molaire, densité) ! à remplacer par du decimal(n,p)
Du varchar pour des colonnes de petite taille, c'est moins grave, mais peu performant

Et aussi, pourquoi la colonne de jointure porte -t- elle un nom différent entre les 2 tables de gauche ?

[Duck_ArMyx]

ce n'est pas moi qui ai fait la bdd et le site je doit faire des modification et des ajouts du coup je ne peut pas tout modifier je m'adapte au code déjà existant desole

[escartefigue]

Signalez ces anomalies aux personnes en charge, avec un peu de chance ils prendront en compte pour une maintenance future

Un mouvais typage d'attributs est lourd de conséquence, et rattraper ces erreurs sur des tables chargées est une opération souvent pénible

[Artemus24]

Salut à tous.

Si vous désirez mettre la liste de vos codes libellés dans le trigger, voici comment vous devez procéder :

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
--------------
SET AUTOCOMMIT = 0
--------------
 
--------------
START TRANSACTION
--------------
 
--------------
DROP DATABASE IF EXISTS `base`
--------------
 
--------------
CREATE DATABASE `base`
        DEFAULT CHARACTER SET `latin1`
        DEFAULT COLLATE       `latin1_general_ci`
--------------
 
--------------
DROP TABLE IF EXISTS `risque`
--------------
 
--------------
CREATE TABLE `risque`
( `id`           integer  unsigned  not null auto_increment primary key,
  `libelle`      char(04)           not null
) ENGINE=InnoDB
  DEFAULT CHARSET=`latin1` COLLATE=`latin1_general_ci`
  ROW_FORMAT=COMPRESSED
--------------
 
--------------
insert into `risque` (`libelle`) values
  ('H200'),('H221'),('H225'),('H302'),('H332')
--------------
 
--------------
select * from `risque`
--------------
 
+----+---------+
| id | libelle |
+----+---------+
|  1 | H200    |
|  2 | H221    |
|  3 | H225    |
|  4 | H302    |
|  5 | H332    |
+----+---------+
--------------
DROP TABLE IF EXISTS `produit`
--------------
 
--------------
CREATE TABLE `produit`
( `id`           integer  unsigned  not null auto_increment primary key,
  `mention`      smallint unsigned  not null,
  `clef`         integer  unsigned  not null,
  CONSTRAINT `FK_risque` FOREIGN KEY (`clef`) REFERENCES `risque` (`id`) ON DELETE CASCADE ON UPDATE CASCADE
) ENGINE=InnoDB
  DEFAULT CHARSET=`latin1` COLLATE=`latin1_general_ci`
  ROW_FORMAT=COMPRESSED
--------------
 
--------------
DROP TRIGGER IF EXISTS `ajout`
--------------
 
--------------
CREATE TRIGGER `ajout`
BEFORE INSERT ON `produit`
FOR EACH ROW
BEGIN
  DECLARE _code char(04)  default '';
  DECLARE _val  smallint  default 0;
 
  SET _code = (select libelle from `risque` where id = new.clef);
 
  set _val  = case _code when 'H200' then 1  when 'H201' then 1  when 'H202' then 1  when 'H203' then 1  when 'H205' then 1  when 'H224' then 1
                         when 'H225' then 1  when 'H220' then 1  when 'H222' then 1  when 'H270' then 1  when 'H271' then 1  when 'H240' then 1
                         when 'H241' then 1  when 'H250' then 1  when 'H251' then 1  when 'H300' then 1  when 'H301' then 1  when 'H310' then 1
                         when 'H330' then 1  when 'H311' then 2  when 'H331' then 1  when 'H370' then 1  when 'H334' then 1  when 'H318' then 1
                         when 'H304' then 1  when 'H372' then 1  when 'H350' then 1  when 'H340' then 1  when 'H360' then 1
                         when 'H204' then 2  when 'H226' then 2  when 'H221' then 2  when 'H223' then 2  when 'H252' then 2  when 'H290' then 2
                         when 'H280' then 2  when 'H281' then 2  when 'H302' then 2  when 'H312' then 2  when 'H332' then 2  when 'H371' then 2
                         when 'H335' then 2  when 'H336' then 2  when 'H315' then 2  when 'H319' then 2  when 'H317' then 2  when 'H373' then 2
                         when 'H351' then 2  when 'H341' then 2  when 'H361' then 2  when 'H400' then 2  when 'H410' then 2  when 'H420' then 2
                         else 0 end;
 
  set new.mention = _val;
END
--------------
 
--------------
insert into `produit` (`clef`) values
  (4),(3),(2),(1),(5)
--------------
 
--------------
select * from `produit`
--------------
 
+----+---------+------+
| id | mention | clef |
+----+---------+------+
|  1 |       2 |    4 |
|  2 |       1 |    3 |
|  3 |       2 |    2 |
|  4 |       1 |    1 |
|  5 |       2 |    5 |
+----+---------+------+
--------------
COMMIT
--------------
 
--------------
SET AUTOCOMMIT = 1
--------------
 
Appuyez sur une touche pour continuer...

Maintenant, si la liste de vos codes libellés est longue, le mieux est de créer une table qui à partir de ce code libellé va vous donner la mention que vous rechercher.

@+