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

[elvan49]

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.

[jota5450]

slt...


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

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

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

[Sivrît]

Envoyé par elvan49

Bonjour, merci de ces précisions

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

Code :

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)

[elvan49]

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 !

[elvan49]

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

[elvan49]

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

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

[elvan49]

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

[pop_up]

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

[elvan49]

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

[pop_up]

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 ;

[elvan49]

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.

[pop_up]

j'ai trouvé ce lien

j'espere que ça t'aidera