Discussion :

[tonyhihi]

Bonjour a tous, voila je débute en trigger et j'ai toujours des problèmes de syntaxe lors de la création de mon trigger que voici:

Je veux qu'avant l'ajout dans la table inscrit, on parcourt la table inscrit et que si la valeur a insérer se trouve dans le champs mot de passe, alors on quitte le trigger, sinon on effectue l'ajout =)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
CREATE TRIGGER AJOUT  BEFORE INSERT ON inscrit
 
BEGIN
 
DECLARE temp varchar(20);
DECLARE curs CURSOR FOR select motdepasse from inscrit;
FETCH curs INTO temp 
if (temp <> New.motdepasse) THEN
LEAVE ;
END IF ;
END $$

Voila, mon délimiteur est $$

Mon message d'erreur est le suivant :

Code :

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

1
2
 
#1064 - Erreur de syntaxe près de 'BEGIN DECLARE temp varchar(20); DECLARE curs CURSOR FOR select motdepasse fr' à la ligne 3

Si quelqu'un peut m'éclairer la dessu ca serait cool =)

[CinePhil]

Il manque FOR EACH ROW avant le BEGIN je pense.

[tonyhihi]

Merci Cinéphil =) maintenant ca bloque à la ligne 9 oO :

requête SQL:

Code :

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

1
2
 
#1064 - Erreur de syntaxe près de '; end if ; END §§' à la ligne 9

[tonyhihi]

J'ai beau remuer dans tous les sens ce trigger je n'y arrive pas....

[ericd69]

salut,

il manque un ; à la fin de ton fetch...

tu ne fais pas de boucle avec ton curseur... ça revient à faire:

Code sql :

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

SELECT motdepasse into temp FROM inscrit limit 1;

en plus, comment garantir que lis la bonne ligne avec ton select? il n'y a qu'un seul inscrit?

[tonyhihi]

Merci de ton intérêt pour mon post,je pensais que le fetch permettait de parcourir mon curseur... j'ai rajouté un ; aprés le fetch, maintenant ca bloque à la ligne 9

Code :

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

1
2
 
#1064 - Erreur de syntaxe près de '; end if; END' à la ligne 9

je vais chercher comment parcourir un curseur oO

[tonyhihi]

Aprés des petites recherches;, j'arrive à ce code suivant, et encore une fois ca bloque dans le bloc if end if .....

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  BEFORE INSERT ON inscrit
FOR EACH ROW
BEGIN
 
DECLARE temp varchar(20);
DECLARE curs CURSOR FOR select motdepasse from inscrit;
open curs;
loop
FETCH curs INTO temp; 
if (temp = New.motdepasse) THEN // g modifé <> en = 
LEAVE ;
end if;
end loop;
Close curs;
END $$

Message d'érreur .... :

Code :

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

#1064 - Erreur de syntaxe près de '; end if; end loop; Close curs; END' à la ligne 11

Ne faut il pas plutot boucler avec genre un while curseur n'est pas vide?

Désolé c'est mon premier trigger oO Dépucelage :p

[CinePhil]

J'ai l'impression que c'est le mot clé LEAVE qui n'est pas compris par MySQL. Tu es sûr qu'il existe ?

Par ailleurs, j'ai l'impression que tu veux empêcher par ce trigger que le même mot de passe existe deux fois. Si c'est ça, un simple index UNIQUE sur la colonne du mot de passe est suffisant ; pas besoin d'un trigger. Il suffit au programme de récupérer l'erreur en cas de tentative d'insertion d'un doublon et de la traiter.

[ericd69]

si si il existe...

2 problèmes:

• faut faire ton fetch dans ta boucle... sinon tu lis pas ton curseur
• et les commentaire ça s'écrit avec "/* */" ou "-- "

[tonyhihi]

Oui cinéphile je sais que si je le met en identifiant c'est réglé, seulement je dois utiliser un trigger dans un projet donc je m'exerce =)

Les commentaires je les ai juste mi dans le post, pas dans le trigger, je vais tester de mettre le fetch dans la boucle =)

Merci à vous en tout cas ^^

[tonyhihi]

Bon j'en avais marre donc j'ai penché vers une solution simpliste bien que pas trés estéthique oO

En gros si la ligne traité contient le mot de passe saisi par l'utilisateur, on appelle une fonction inexistante arrétant le trigger.
L’exception est géré par un try catch sous php, voila le code du trigger :p
Merci a cinéphil et ericd69 pour leurs aides, si vous trouvez une autre option je suis tout de même preneur =) JE met résolu ou pas? xD

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 TRIGGER AJOUT  BEFORE INSERT ON inscrit
FOR EACH ROW
BEGIN
DECLARE done INT DEFAULT 0;
DECLARE temp varchar(20);
DECLARE curs CURSOR FOR select motdepasse from inscrit;
DECLARE CONTINUE HANDLER FOR NOT FOUND SET done = 1;
OPEN curs;
country_loop: LOOP 
FETCH curs INTO temp ;
if (temp = New.motdepasse) THEN
call error() ;
  IF done THEN
      LEAVE country_loop;
   END IF;
end if;
end loop;
END

[ericd69]

un exemple puisque tu peux toujours pas utiliser signal pour générer une erreur dans un déclencheur... (triggers-that-cause-inserts-to-fail-possible)...

ta démarche est la bonne mais ton code a encore un problème:

Code sql :

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
CREATE TRIGGER AJOUT  BEFORE INSERT ON inscrit
FOR EACH ROW
BEGIN
DECLARE done INT DEFAULT 1;
DECLARE temp varchar(20);
DECLARE curs CURSOR FOR SELECT motdepasse FROM inscrit;
DECLARE CONTINUE HANDLER FOR NOT FOUND SET done = 0;
OPEN curs;
country_loop: LOOP 
FETCH curs INTO temp ;
IF done THEN
  IF (temp = New.motdepasse) THEN
    call "ton message d'erreur"; -- tu plantes et génères une erreur
  END IF;
else
  leave country_loop;
end IF;
end loop;
END

le test sur done doit te permettre de contrôler si tu applique un traitement si et seulement si le fetch a rapatrié quelque chose...

une version plus propre:

Code sql :

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
CREATE TRIGGER AJOUT  BEFORE INSERT ON inscrit
FOR EACH ROW
BEGIN
DECLARE done INT DEFAULT 1;
DECLARE temp varchar(20);
DECLARE curs CURSOR FOR SELECT motdepasse FROM inscrit;
DECLARE CONTINUE HANDLER FOR NOT FOUND SET done = 0;
OPEN curs;
country_loop: LOOP 
FETCH curs INTO temp ;
IF done THEN
  IF (temp = New.motdepasse) THEN
    call error("ton message d'erreur");
  END IF;
else
  leave country_loop;
end IF;
end loop;
END

avec ta procédure error:

Code sql :

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

1
2
3
4
create procedure error(in mes varchar(255))
begin
  signal sqlstate '45000' set message_text=mes;
end

[tonyhihi]

MErci bien eric, j'avais testé le signal mais ma version de mysql doit pas connaître signal )= je vais tester d'en prendre une plus récente

[ericd69]

oui c'est possible car c'est un rajout assez récent pour coller au standard...

à savoir que tu peux tester signal en dehors d'une procédure comme simple instruction mais pas directement dans le déclencheur... me demande pas pourquoi...



au pire tu utilises la version sans signal...