Problème avec valeur de retour d'une fonction

**ouranos21** · 17/01/2009, 18h16

Bonjour,

J'ai un petit souci avec une valeur de retour d'une fonction. Évidemment ça doit être une erreur toute bête, mais le nez dedans, l'erreur, je la vois pas.

L'erreur mysql que je reçois lorsque je veux créer cette fonction c'est l'errorcode 1320 qui correspond à "pas de valeur de retour pour la fonction.

Voici la fonction:

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
 
CREATE FUNCTION add_user (p1 varchar(60),p2 varchar(100),p3 varchar(60),p4 varchar(60),p5 varchar(60),p6 varchar(120),p7 varchar(60),p8 date,fk_group int,fk_player int,fk_country int) RETURNS INT
BEGIN
 
DECLARE v_login,v_login_tm,v_email varchar(60) default null;
DECLARE v_fk_group,v_fk_player,v_fk_country INT DEFAULT 0;
DECLARE var1 int default 0;
 
      SELECT login INTO v_login FROM users
      WHERE login=p1;
      SELECT tm_login INTO v_login_tm FROM users
      WHERE tm_login=p3;
      SELECT email INTO v_email FROM users
      WHERE email=p7;
      SELECT id_country INTO v_fk_country FROM country
      WHERE id_country = fk_country;
 
      IF v_login != null THEN
      SET var1=1;
              ELSEIF v_tm_login != null THEN
              SET var1=2;
                      ELSEIF v_email != null THEN
                      SET var1=3;
                              ELSE
                                    insert into users (login,password,login_tm,first_name,last_name,adress,email,last_logon,fk_id_group,fk_id_player,fk_id_country)
                                    values (p1,p2,p3,p4,p5,p6,p7,p8,1,null,v_fk_country);
                      END IF;
              END IF;
      END IF;
 
      return var1;
END

Pourquoi ce code ne passe pas ?
Où est l'erreur ?

D'avance merci.
Franck.

**Antoun** · 17/01/2009, 22h27

IF v_login != NULL ne sera jamais vérifié. Il faut utiliser IS NOT NULL. Tu devrais également mettre un ELSE pour le cas où v_login est NULL.

**ouranos21** · 18/01/2009, 00h33

Merci de la réponse

Alors voilà maintenant ce que j'ai:

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
 
CREATE FUNCTION add_user (p1 varchar(60),p2 varchar(100),p3 varchar(60),p4 varchar(60),p5 varchar(60),p6 varchar(120),p7 varchar(60),p8 date,fk_group int,fk_player int,fk_country int) RETURNS INT
BEGIN
 
DECLARE v_login,v_login_tm,v_email varchar(60) DEFAULT NULL;
DECLARE v_fk_group,v_fk_player,v_fk_country INT DEFAULT 0;
DECLARE var1 int DEFAULT 0;
 
      SELECT login INTO v_login FROM users
      WHERE login=p1;
      SELECT tm_login INTO v_login_tm FROM users
      WHERE tm_login=p3;
      SELECT email INTO v_email FROM users
      WHERE email=p7;
      SELECT id_country INTO v_fk_country FROM country
      WHERE id_country = fk_country;
 
      IF v_login IS NOT NULL THEN
      SET var1=1;
              ELSEIF v_tm_login IS NOT NULL THEN
              SET var1=2;
                      ELSEIF v_email IS NOT NULL THEN
                      SET var1=3;
                              ELSE
                                    insert into users (login,pass,login_tm,first_name,last_name,adress,email,last_logon,fk_id_group,fk_id_player,fk_id_country)
                                    values (p1,p2,p3,p4,p5,p6,p7,p8,1,null,v_fk_country);
                      END IF;
              END IF;
      END IF;
 
      RETURN var1;
END

Et ça ne fonctionne toujours pas :s

D'autres idées ?

D'avance merci.
Franck.

**Antoun** · 18/01/2009, 08h20

et si tu rajoutais var1=0 dans ton ELSE ?

**ouranos21** · 18/01/2009, 13h54

Rha, toujours pas

Voici le code de ma fonction maintenant:

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
 
CREATE FUNCTION `tmallstars`.`add_user2` (p1 varchar(60),p2 varchar(100),p3 varchar(60),p4 varchar(60),p5 varchar(60),p6 varchar(120),p7 varchar(60),p8 date,fk_group int,fk_player int,fk_country int) RETURNS INT
BEGIN
 
DECLARE v_login,v_login_tm,v_email varchar(60) DEFAULT NULL;
DECLARE v_fk_group,v_fk_player,v_fk_country INT DEFAULT 0;
DECLARE var1 int DEFAULT 0;
 
      SELECT login INTO v_login FROM users
      WHERE login=p1;
      SELECT login_tm INTO v_login_tm FROM users
      WHERE login_tm=p3;
      SELECT email INTO v_email FROM users
      WHERE email=p7;
      SELECT id_country INTO v_fk_country FROM country
      WHERE id_country = fk_country;
 
      IF v_login IS NOT NULL THEN
      SET var1=1;
              ELSEIF v_tm_login IS NOT NULL THEN
              SET var1=2;
                      ELSEIF v_email IS NOT NULL THEN
                      SET var1=3;
                              ELSE
                                    INSERT INTO users (login,pass,login_tm,first_name,last_name,adress,email,last_logon,fk_id_group,fk_id_player,fk_id_country)
                                    VALUES (p1,p2,p3,p4,p5,p6,p7,p8,1,NULL,v_fk_country);
                                    SET var1=0;
                      END IF;
              END IF;
      END IF;
 
      RETURN var1;
 
END

En tout cas merci pour l'aide

**Antoun** · 18/01/2009, 14h12

Comment tu appelles ta fonction ?

**ouranos21** · 18/01/2009, 14h16

C'est à la création de la fonction que j'ai l'erreur...

Donc je ne l'appelle pas encore :s

Voici l'erreur que j'ai: http://img223.imageshack.us/img223/1...urmysqlmt2.jpg

**Antoun** · 19/01/2009, 01h38

ah, fallait commencer par là ! à tous les coups, c'est une histoire de ;

c'est quoi, ton client avec une drôle de tronche ?

**ouranos21** · 19/01/2009, 17h19

Outch, désolé, huhu.

J'utilise Mysql Query Browser là (avec MySQL v5.1). Et j'utilise le tool de création de Procédure/Fonction inclus dans le GUI pour créer ma fonction... peut-être devrais-je directement passer par le scripting... je vais essayer.

Encore désolé.
Franck.

**Antoun** · 19/01/2009, 17h47

Envoyé par ouranos21

Outch, désolé, huhu.

J'utilise Mysql Query Browser là (avec MySQL v5.1). Et j'utilise le tool de création de Procédure/Fonction inclus dans le GUI pour créer ma fonction... peut-être devrais-je directement passer par le scripting... je vais essayer.

Encore désolé.
Franck.

Non, si tu utilises bien le machin de création de script, ça devrait passer...

Si tu me donnes les CREATE TABLE nécessaire, je ferai un test ce soir.

**Antoun** · 19/01/2009, 23h17

En fait c'est ta syntaxe qui est mauvaise. Il ne faut pas mettre un END IF pour chaque ELSEIF, mais un seul END IF à la fin de l'ensemble :

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
CREATE FUNCTION add_user (p1 varchar(60),p2 varchar(100),p3 varchar(60),p4 varchar(60),p5 varchar(60),p6 varchar(120),p7 varchar(60),p8 date,fk_group int,fk_player int,fk_country int) RETURNS INT
BEGIN
 
DECLARE v_login,v_login_tm,v_email varchar(60) DEFAULT NULL;
DECLARE v_fk_group,v_fk_player,v_fk_country INT DEFAULT 0;
DECLARE var1 int DEFAULT 0;
 
      SELECT login INTO v_login FROM users
      WHERE login=p1;
      SELECT tm_login INTO v_login_tm FROM users
      WHERE tm_login=p3;
      SELECT email INTO v_email FROM users
      WHERE email=p7;
      SELECT id_country INTO v_fk_country FROM country
      WHERE id_country = fk_country;
 
      IF v_login != NULL THEN
		SET var1=1;
      ELSEIF v_tm_login != NULL THEN
		SET var1=2;
	  ELSEIF v_email != NULL THEN
        SET var1=3;
	  ELSE
		INSERT INTO users (login,password,login_tm,first_name,last_name,adress,email,last_logon,fk_id_group,fk_id_player,fk_id_country)
        VALUES (p1,p2,p3,p4,p5,p6,p7,p8,1,NULL,v_fk_country);
      END IF;
 
      RETURN var1;
END

**ouranos21** · 20/01/2009, 08h52

C'est bon à savoir en tout cas. Merci beaucoup :-)

Problème avec valeur de retour d'une fonction

SQL Procédural MySQL

Discussions similaires

Partager

Partager