[procedure] une erreur de syntaxe, mais où ?

Version imprimable

Bonjour,

Je démarre dans la création des procédures MySQL. Voici mon code et j'ai une erreur visiblement au niveau de l'affectation de c...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
DELIMITER //
CREATE PROCEDURE Liste_Pays()
BEGIN
    DECLARE i INT DEFAULT 1;
    DECLARE c INT DEFAULT 1; 
    INSERT INTO c VALUES(SELECT COUNT(nom_pays) FROM pays);
    WHILE i<=c DO
        SELECT nom_pays, id_pays FROM pays;
        SET i:=i+1;
    END WHILE;    
END//
DELIMITER ;

Peut-onm'aider ? merci.

30/11/2007, 17h18
jota5450

slt...

set i = ... il y a pas les :

Citation:

INSERT INTO c VALUES(SELECT COUNT(nom_pays) FROM pays);

Ton c est int... du devrais pas inserer dans une table?

et puis la bonne syntaxe est:

Code:

insert into TA_TABLE(TON_CHAMP) select count(nom_pays) from pays

SEt au lieu d'INSERT

Bonjour, merci de ces précisions

Est-ce qu'il ne vaudrait mieux pas quelque chose du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 DELIMITER // CREATE PROCEDURE Liste_Pays() BEGIN DECLARE i INT DEFAULT 1; DECLARE c INT DEFAULT 1; SET c = SELECT COUNT(nom_pays) FROM pays); WHILE i<=c DO SELECT nom_pays, id_pays FROM pays; SET i=i+1; END WHILE; END// DELIMITER ;
L'idée est de donner le résultat de count comme valeur à c...
Je vais tester.

Citation:
Envoyé par elvan49
Bonjour, merci de ces précisions

Est-ce qu'il ne vaudrait mieux pas quelque chose du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 DELIMITER // CREATE PROCEDURE Liste_Pays() BEGIN DECLARE i INT DEFAULT 1; DECLARE c INT DEFAULT 1; SET c = SELECT COUNT(nom_pays) FROM pays); WHILE i<=c DO SELECT nom_pays, id_pays FROM pays; SET i=i+1; END WHILE; END// DELIMITER ;
L'idée est de donner le résultat de count comme valeur à c...
Je vais tester.
Je crois que la syntaxe que tu recherches est
Code:

1 2 3 DECLARE c INT; SELECT COUNT(nom_pays) INTO c FROM pays;
(http://dev.mysql.com/doc/refman/5.0/...statement.html)

02/12/2007, 19h26
elvan49

Merci beaucoup

Merci, c'est ça !
Le lien vers mysqlAB good bien vu. J'y suis souvent mais quand on ne sait pas quoi chercher, c'est pas simple... Là direct !:king:
03/12/2007, 09h07
elvan49

J'ai dit "c'est ça" mais...

Bonjour,

Je me suis un peu trop emporté.
La procédure a bien une syntaxe correcte mais son appel ne renvoie pas le résultat escompté, au contraire j'ai :

Code:

#1312 - PROCEDURE base_type.Liste_Pays can't return a result set in the given context

Je voulais la liste des pays... :bug:

je progresse vers la vérité...

C'est déjà mieux et ça me donne une indication sur ce qui me manque.
Visiblement, il ne me reste plus qu'à trouver comment lui dire de stocker les résultats nonpas dans une variable mais dans une colonne...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 DELIMITER // CREATE PROCEDURE Liste_Pays() BEGIN DECLARE i INT DEFAULT 1; DECLARE liste VARCHAR(50); DECLARE c INT DEFAULT 1; SELECT COUNT(nom_pays) INTO c FROM pays; WHILE i<=c DO SELECT nom_pays INTO liste FROM pays; SET i:=i+1; END WHILE; END// DELIMITER ;
Je précise que ce code donne cette erreur :

Code:

#1172 - Result consisted of more than one row

à suivre...

03/12/2007, 09h47
pop_up

le probleme vient du fait que ton

Code:

SELECT nom_pays INTO liste FROM pays;

te renvoi plusieurs resultats.

tu peux utiliser un curseur.

Dernière étape (...?...)

Voici le code de procédure valide (en tout cas sans faute de syntaxe)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
DELIMITER //
CREATE PROCEDURE Liste_Pays()
BEGIN
    DECLARE i INT DEFAULT 1;
    DECLARE liste VARCHAR(50);
    DECLARE c INT DEFAULT 1;
    DECLARE curseur1 CURSOR FOR SELECT nom_pays FROM pays;
    DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
    OPEN curseur1;
    SELECT COUNT(nom_pays) INTO c FROM pays;
    WHILE i<=c DO
        FETCH curseur1 INTO liste;
        SET i:=i+1;
    END WHILE;
    CLOSE curseur1;
END//
DELIMITER ;

L'appel ne renvoie plus d'erreur, mais il ne m'affiche rien

essaye ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
DELIMITER //
CREATE PROCEDURE Liste_Pays()
BEGIN
    DECLARE i INT DEFAULT 1;
    DECLARE liste VARCHAR(50);
    DECLARE c INT DEFAULT 1;
    DECLARE curseur1 CURSOR FOR SELECT nom_pays FROM pays;
    DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
    OPEN curseur1;
    FETCH curseur1 INTO liste;
    SELECT COUNT(nom_pays) INTO c FROM pays;
    WHILE NOT done DO
        SET i=i+1;
        FETCH curseur1 INTO liste; 
    END WHILE;
    CLOSE curseur1;
END//
DELIMITER ;

J'ai modifié la condition while et les fetch.

EDIT: il faut que tu declares ta variable "done" en INT aussi

03/12/2007, 10h25
elvan49

Toujours rien d'affiché

Merci du tuyaux,

Le résultat semble être le même qu'avec la procédure précédente. Requête éxécutée mais aucun résultat affiché...
Ne faudrait-il pas un SELECT pour qu'elle affiche quelque chose ?
Et un SELECT sur quoi ???

que veut tu que ça t'affiche ?

si tu veux afficher i, essaye ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
DELIMITER //
CREATE PROCEDURE Liste_Pays() RETURNS INT
BEGIN
    DECLARE i INT DEFAULT 1;
    DECLARE liste VARCHAR(50);
    DECLARE c INT DEFAULT 1;
    DECLARE curseur1 CURSOR FOR SELECT nom_pays FROM pays;
    DECLARE CONTINUE HANDLER FOR SQLSTATE '02000' SET done = 1;
    OPEN curseur1;
    FETCH curseur1 INTO liste;
    SELECT COUNT(nom_pays) INTO c FROM pays;
    WHILE NOT done DO
        SET i=i+1;
        FETCH curseur1 INTO liste; 
    END WHILE;
    CLOSE curseur1;
    RETURN i;
END//
DELIMITER ;

03/12/2007, 11h11
elvan49

RETURN uniquement pour FUNCTION

Désolé, mais RETURN n'est valide que pour les fonctions. Pas pour les Procédures.
Je voudrai afficher la liste des pays (leur nom) l'équivalent de :

Code:

SELECT nom_pays FROM pays ORDER BY nom_pays ASC;

La requête de base est simple et je peux la traiter en PHP, mais vu qu'elle ne nécessite aucun paramètre (type POST ou GET) je voudrai l'enlever de mes scripts et donc la stocker (avec d'autres du même genre) directement dans la base (je crée ainsi un niveau d'abstratction supllémentaire).

J'éspère avoir été plus clair sur mes intentions. Merci entout cas pour ton aide.
03/12/2007, 11h25
pop_up

j'ai trouvé ce lien

j'espere que ça t'aidera