Trigger Php Admin

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:

---

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:

---

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 =)

Il manque FOR EACH ROW avant le BEGIN je pense.

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

requête SQL:

Code:

---

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

---

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

salut,

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

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

Code:

---

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? :aie:

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:

---

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

---

je vais chercher comment parcourir un curseur oO

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

Code:

---

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:

---

#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

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.

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 "-- "

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 ^^

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:

---

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

---

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:

---

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:

---

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:

---

1 2 3 4

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

---

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

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...

:calim2:

au pire tu utilises la version sans signal... :zoubi: