Problème avec valeur de retour d'une fonction

Version imprimable

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:

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.

17/01/2009, 22h27
Antoun

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.

Merci de la réponse :)

Alors voilà maintenant ce que j'ai:

Code:

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.

18/01/2009, 08h20
Antoun

et si tu rajoutais var1=0 dans ton ELSE ?

Rha, toujours pas :(

Voici le code de ma fonction maintenant:

Code:

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

18/01/2009, 14h12
Antoun

Comment tu appelles ta fonction ?
18/01/2009, 14h16
ouranos21

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
19/01/2009, 01h38
Antoun

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

c'est quoi, ton client avec une drôle de tronche ?
19/01/2009, 17h19
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.
19/01/2009, 17h47
Antoun

Citation:

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.

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:

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

20/01/2009, 08h52
ouranos21

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