Discussion :

[catadanna]

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 :

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 :

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

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

[catadanna]

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

[Antoun]

RETURNS est obligatoire pour les fonctions...

[catadanna]

OK, donc j'ai mis à la fin un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

return 1

Ma fonction est :

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

[Antoun]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Query OK, 0 rows affected (0.00 sec)

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

@+