Discussion :

[azebou]

voila depuis un heure je bataille avec cette fonction mais rien marche est ce que la syntaxe est jute svp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
CREATE FUNCTION VERIFIERMOTIF (
    dv date,
    di date
    )
    RETURNS boolean deterministic
    BEGIN
        set @ok = false;
 
        if (@dv < @dinf )  then set    @ok = true;
        end if;
 
        return (@ok );
    END;
//

[CinePhil]

Tu n'as pas indiqué le DELIMITER et il y a le caractère HTML   qui fait bizarre dans les paramềtres passés à la fonction.

[azebou]

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
24
25
26
27
28
29
CREATE FUNCTION VERIFIERMOTIF (
    im int(11),
    div date,
    dif date
    )
    RETURNS boolean deterministic
    BEGIN
 
        select 
            m.id_motif 
        from
            pvi p,
            motif m
        where 
            m.id_motif = p.id_motif and 
            m.id_motif= im and
            m.date_validite_amende = div and
            p.date_infraction = dif;
 
        set @ok = false;
 
        if (@div < @dif) then set    @ok = true;
        end if;
 
        return (@ok );
    END;
//
 
delimiter ;

les trucs html à mon avis c'est quand j'ai copier la fonction
ca fonctionne tooujours

[MightyDucks]

Bonjour,

'div' est un mot clef réservé donc il faut utiliser un autre nom.

Ensuite il manque le début de votre procédure à savoir le premier délimiter.

quelque chose de ce type devrait être plus correct :

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
24
25
26
 
DELIMITER $$
CREATE FUNCTION VERIFIERMOTIF (im int(11), div1 date, dif date )
    RETURNS BOOLEAN deterministic
    BEGIN
        SELECT 
            m.id_motif 
        FROM
            pvi p,
            motif m
        WHERE 
            m.id_motif = p.id_motif AND 
            m.id_motif= im AND
            m.date_validite_amende = div1 AND 
            p.date_infraction = dif$$
 
        SET @ok = false$$
 
        IF (@div1 < @dif) then 
            SET @ok = true$$
        end IF$$
 
        RETURN (@ok )$$
    END$$
 
delimiter ;

Ensuite vous effectuez un select sans récupérer la ou les valeurs retournées or vous avez spécifié un type de retour BOOLEAN. La requête va être exécutée et retournée un résultat erroné de part la signature de votre fonction. Après sans connaître le retour attendu de votre fonction difficile d'en dire plus

[ericd69]

salut,

juste une petite explication sur les variables dans les procédure stockées ou fonctions:

tu as les variables locales à la procédure soit passé dans l'entête genre:
pour une procédure:

Code sql :

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

in bidule int(4)

pour une fonction:

Code sql :

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

bidule int(4)

ou déclarée dans le corps de celle-ci:

Code sql :

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

DECLARE truc int(4);

et les variables globales commençant par un @ qui ne sont pas typée et qui peut être définie avant l'appel à la procédure et dont la valeur persiste après sa fin jusqu'à la fin de la session en cours pour l'utilisateur...

donc truc et @truc c'est PAS DU TOUT la même variable...

pour mightyduck, tu as rien compris à l'utilisation de DELIMITER... l'idée est que l'interpréteur sql ne s'arrête pas sur les ';' des instructions dans le corps de la procédure ou fonction pour exécuter correctement le CREATE PROCEDURE ou FUNCTION...

pour tous les 2, on évite d'utiliser des globales quand c'est pas nécessaire et on leur préfère une variable locale... votre select ne sert à rien tel quel vu qu'il ne stocke rien... comme ça il ne peut que générer une erreur...

le IF fait que quelque soit le résultat du sélect seul l'ordre des date compte... gros problème de modélisation de ce qui est voulu....

je suppose que tu voulais chercher si un motif est apparu entre 2 dates...

une version un peut mieux pourrais être:

Code sql :

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
DELIMITER $$
CREATE FUNCTION VERIFIERMOTIF (im int(11), div1 date, dif date )
    RETURNS BOOLEAN deterministic
    BEGIN
        DECLARE ok bool;
        SELECT 
            count(m.id_motif)>0
        INTO ok
        FROM
            pvi p,
            motif m
        WHERE 
            m.id_motif = p.id_motif AND 
            m.id_motif= im AND
            m.date_validite_amende = div1 AND 
            p.date_infraction = dif;
 
        RETURN ok;
    END$$
 
delimiter ;

l'écriture de la requête ne correspond pas aux standards actuel de l'écriture des jointures...

je vous conseille à tous les 2 de réviser les JOINTURES en sql...

sqlpro a pondu un cours pas trop mal sur le sujet je vous recommande sa lecture...

sans les tables et l'explication de ce qui est réellement cherché... je crois que tu as les pistes pour déjà avancer dans ton problème

[MightyDucks]

Oula effectivement j'ai bouletisé le délimiteur est très mal placé !

Je devais être à moitié réveillé ne tenez pas compte de mon post.