Error 1064 pour fonction.

Version imprimable

Bonjour,

Je suis en train de créer un script en MYSQL (version 5.6.17).
Je viens d'avoir une erreur au début du script :

Citation:

Error 1064 (42000) at line 5 : Erreur de syntaxe près de 'BEGIN
DECLARE StartDate DATETIME;
DECLARE EndDate DATETIME;
DECLARE PreceedingOldValue INT DEFAULT 0;
DECLARE Preceedin'

Et voici mon code, qui commence comme ça:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 DROP FUNCTION IF EXISTS getHours; DELIMITER // CREATE FUNCTION getHours() BEGIN DECLARE StartDate DATETIME; DECLARE EndDate DATETIME; DECLARE PreceedingOldValue INT DEFAULT 0; DECLARE PreceedingNewValue INT DEFAULT 0; DECLARE CurrentTicket INT DEFAULT 0; DECLARE NbWorkingDays INT DEFAULT 0; DECLARE items_id, id, old_value, new_value, id_users INT(10); DECLARE date_mod, creation_date DATETIME; DECLARE name VARCHAR(50); DECLARE NbHours INTEGER DEFAULT 0; (...) END; // DELIMITER;
J'ai suivi la documentation MYSQL mais je ne vois pas où ça coince, quelqu'un une idée ?

Merci.

26/06/2015, 16h01
Antoun

Si c'est une procédure stockée, il faut utiliser CREATE PROCEDURE.

Si c'est une fonction, il faut préciser le type de résultat avec une clause RETURNS avant le BEGIN.
26/06/2015, 16h32
catadanna

... merci donc :

Bonjour,

Merci pour votre réponse.
Il s'agit bien d'une fonction, pas d'une procédure stockée. Il n'y a pas de valeur retournée, donc je n'ai pas mis de clause "RETURNS <type>".
26/06/2015, 17h30
Antoun

RETURNS est obligatoire pour les fonctions...

OK, donc j'ai mis à la fin un

Code:

return 1

Ma fonction est :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 DROP FUNCTION IF EXISTS getHours; DELIMITER // CREATE FUNCTION getHours() RETURNS BIT; BEGIN DECLARE StartDate DATETIME; DECLARE EndDate DATETIME; DECLARE PreceedingOldValue INT DEFAULT 0; DECLARE PreceedingNewValue INT DEFAULT 0; (...) RETURN 1; END; //
J'ai la même erreur.
Je ne vois toujours pas d'où ça peut venir :
Est-ce qu'il faut supprimer des lignes vides? Est-ce qu'il y a des conditions particulières pour les noms/longueur des variables?

Merci.

27/06/2015, 16h22
Antoun
Citation:

Envoyé par catadanna

J'ai la même erreur.
Je ne vois toujours pas d'où ça peut venir :
Est-ce qu'il faut supprimer des lignes vides? Est-ce qu'il y a des conditions particulières pour les noms/longueur des variables?

Merci.

Il ne faut pas mettre de point-virgule entre RETURNS et BEGIN :
Code:

1 2 3 4 5 6 7 8 DELIMITER // CREATE FUNCTION getHours() RETURNS BIT BEGIN DECLARE StartDate DATETIME; RETURN 1; END; //
Citation:

Query OK, 0 rows affected (0.00 sec)
29/06/2015, 08h10
catadanna

Merci!

En effet, ça marche comme ça! Merci à vous !
30/06/2015, 14h50
Antoun

Ceci dit, si tu n'utilises pas le résultat de la fonction, ça devrait logiquement être une procédure stockée... en fait ça change surtout la façon dont elle est appelée.
07/07/2015, 11h11
catadanna

Merci!

En fait, le résultat est stocké dans une table temporaire. Je ne crée pas de procédure stockée, mais une fonction, car j'ai des restrictions concernant l'insertion dans la base.

Amicalement.
07/07/2015, 11h16
catadanna

Autre chose :

Bonjour,

Puisque j'y suis : je n'arrive pas à lancer la fonction. Logiquement, à la fin du fichier, après la déclaration, je lance la fonction :

Code:

getHours();

J'ai le même message d'erreur.

Quelqu'un a une idée?

Merci.
07/07/2015, 12h42
Antoun

On en arrive au fait que tu ne veux pas faire une fonction mais une proc' stock'.

Si tu voulais vraiment lancer ta fonction telle quelle, il faudrait faire un truc comme ça :

Code:

SELECT getHours();

Ce qui t'afficherait 1 et en outre devrait faire ce qu'il y a dans ta fonction (sauf limitation).

Normalement, tu devrais la transformer en procédure, ce qui te permettrait de lever certaines limitations et de l'appeler de manière moins contournée :

Code:

CALL getHours();

Merci!

Comme je l'ai dit, je ne peux pas faire de procédure, pour des raisons de réglementation (indépendants des questions techniques - c'est comme ça).

Cette histoire de SELECT ça marche (merci).

Par contre, si j'ai dans mon fichier plusieurs fonctions, ça marche plus. J'ai essayé plusieurs cas de figure, pas de résultat. Faut-il mettre un seul délimiteur pour tout ?
Je ne peux pas mettre tout dans une seule fonction, car func3 est récursive ;-).
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 35 36 37 38 39 40 41 42 43 DELIMITER // DROP FUNCTION IF EXISTS func1; DELIMITER // function func1() RETURNS INT BEGIN DECLARE myvar INT; (...) SET myvar=func2() (...) RETURN 1; END; // DELIMITER ; DROP FUNCTION IF EXISTS func2; DELIMITER // function func2() RETURNS INT BEGIN DECLARE myvar INT; (...) SET myvar = func3(); (...) RETURN myvar; END; // DELIMITER ; DROP FUNCTION IF EXISTS func3; DELIMITER // function func3() RETURNS INT BEGIN DECLARE myvar INT; (...) RETURN myvar; END; // DELIMITER ;

07/07/2015, 14h33
Antoun
Je pense que les deux premières lignes sont contradictoires :

Citation:

DELIMITER //
DROP FUNCTION IF EXISTS func1;

Le DROP doit pouvoir s'exécuter indépendamment du CREATE (et de préférence avant ;)). Donc tu peux faire ça :
Code:

1 2 3 4 DELIMITER ; DROP FUNCTION IF EXISTS func1; DELIMITER // CREATE FUNCTION...
ou encore ça :
Code:

1 2 3 DELIMITER // DROP FUNCTION IF EXISTS func1 // CREATE FUNCTION...

Bonjour,

Il y a une une faute de frappe, mon code est comme ça en effet :

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
35
36
37
38
39
40
41
42
43
44
DROP FUNCTION IF EXISTS func1;
 
DELIMITER //
 
function func1()
RETURNS INT
BEGIN
DECLARE myvar INT;
(...)
SET myvar=func2()
(...)
RETURN 1;
END; //
DELIMITER ;
 
DROP FUNCTION IF EXISTS func2;
 
DELIMITER //
 
function func2()
RETURNS INT
BEGIN
DECLARE myvar INT;
(...)
SET myvar = func3();
(...)
RETURN myvar;
END; //
DELIMITER ;
 
DROP FUNCTION IF EXISTS func3;
 
DELIMITER //
 
function func3()
RETURNS INT
BEGIN
DECLARE myvar INT;
(...)
RETURN myvar;
END; //
DELIMITER ;
 
SELECT func1();

08/07/2015, 18h34
Antoun

Citation:

Envoyé par catadanna

Par contre, si j'ai dans mon fichier plusieurs fonctions, ça marche plus. J'ai essayé plusieurs cas de figure, pas de résultat. Faut-il mettre un seul délimiteur pour tout ?
Je ne peux pas mettre tout dans une seule fonction, car func3 est récursive ;-).

Quand tu dis "ça ne marche plus", ça veut dire quoi ? Ton message laisse penser que les fonctions ne se créent pas, c'est bien ça ?
08/07/2015, 18h38
Antoun

Il faut que tu ajoutes CREATE avant chaque FUNCTION.

Il manque un point-virgule après SET myvar=func2().

... et je ne parle que des lignes que tu as gardées ;)
09/07/2015, 11h47
catadanna

Merci!

Ça marche en effet, merci! J'étais intéressée surtout s'il faut mettre des délimiteurs séparés pour chaque fonction.
Il y a CREATE avant chaque fonction, et point virgule à la fin de "SET ...".
Oui les fonctions sont créées et stockées.

Merci pour le coup de main! On ne trouve pas toutes ces choses sur le site officiel de MYSQL.

Encore une chose :

Je voudrais savoir simplement si je peux récupérer la valeur de retour d'une fonction dans une variable.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 DELIMITER // CREATE function func1(unedate DATETIME) RETURNS DATETIME DECLARE madate DATETIME; BEGIN (...) SET madate = func1(madate); (...) RETURN madate; END; // DELIMITER ; SELECT func1('2015-03-17 12:23:36');
J'ai essayé ceci dans mon code mais j'ai l'impression qu'on n'entre pas dans le rappel de fonction. Je n'ai rien trouvé sur le net à ce sujet.

Merci!

09/07/2015, 14h31
Antoun

T'es pas en train de faire une récursion infinie ?
09/07/2015, 15h14
catadanna

NON! La fonction s'arrête (réponse : pas de résultat). Note : je l'ai testée en phpmyadmin, j'espère qu'il n'y a pas un paramétrage qui pose problème.
Il y a une condition d'arrêt, on arrive là en tout cas.
09/07/2015, 15h40
Antoun

Quand je tente de créer une fonction récursive, j'obtiens ça à l'exécution :

Citation:

mysql> select testrecu(3) ;
ERROR 1424 (HY000): Recursive stored functions and triggers are not allowed.
28/07/2015, 11h00
catadanna

OK! La discussion peut être fermée.
Concernant la récursion, j'ai trouvé, il faut ajouter le nombre de niveaux avant la déclaration de la procédure :
Code:

1 2 3 4 5 DELIMITER // SET @@SESSION.max_sp_recursion_depth=25; // CREATE PROCEDURE ...
Merci pour tout, mon code marche maintenant !
Comment peut-on faire pour fermer la discussion?

Cdt.
28/07/2015, 17h46
SQLpro

Citation:

Envoyé par catadanna

Comme je l'ai dit, je ne peux pas faire de procédure, pour des raisons de réglementation

Je ne savais pas qu'il y avait une réglementation sur les procédures !!!!!

C'est ta boite qui l'interdit ?

A +
29/07/2015, 16h13
catadanna

Réponse réglementation procédures :

Oui, c'est ma boîte qui interdit, en principe (réglementation de la boîte), ce n'est pas une contrainte technique.
Cdt.
30/07/2015, 17h40
aieeeuuuuu

8O

êtes vous sûr que ce n'est pas l'inverse : "interdiction de créer des fonctions qui contiennent des INSERT, UPDATE ou DELETE " ?

ça paraitrait plus sensé.
30/07/2015, 18h45
Artemus24

Salut à tous.

En tout cas, cela ne me surprend pas du tout.
Il y a encore quelques années, chez certains clients (gros système IBM), ils nous étaient interdits de faire des procédures stockées, des jointures car trop coûteux en termes de performances. Cela remonte à plus de vingt-ans.
Alors on faisait comment ? Et bien on déchargeaient les tables sous forme de fichiers séquentiels et l'on faisait le merge en cobol.
Je sais, c'était lourd à faire, mais on n'avait pas le choix !

Maintenant, les systèmes ont évolué et il est de plus en plus rare d'avoir se genre de comportement.
Il se peut aussi que le DBA n'a pas toute la compétence requise pour la maîtrise de ce qui se passe dans sa boîte. Et par peur, il interdit certains outils.

@+