Erreur de syntaxe sur Create Function

Version imprimable

Bonjour

J'ai beau relire les exemples de syntaxe d'un Create Function je ne comprends pas pourquoi l'intruction suivante dont j'ai trouvé l'exemple sur Google me donne une erreur a chaque ligne ?

Ou est l'erreur ?

Merci de votre aide
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 CREATE FUNCTION fct_isoyear @MyDate AS dateTime RETURNS INT AS DECLARE @ISOyear INT = DATEPART(YEAR, @MyDate); -- Special cases: Jan 1-3 may belong to the previous year IF (DATEPART(MONTH, @MyDate) = 1 AND DATEPART(ISO_WEEK, @MyDate) > 50) SET @ISOyear = @ISOyear - 1; -- Special case: Dec 29-31 may belong to the next year IF (DATEPART(MONTH, @MyDate) = 12 AND DATEPART(ISO_WEEK, @MyDate) < 45) SET @ISOyear = @ISOyear + 1 RETURN @ISOYear;
Citation:

Msg 102, Level 15, State 1, Procedure isoyear, Line 1
Incorrect syntax near '@MyDate'.
Msg 137, Level 15, State 2, Procedure isoyear, Line 4
Must declare the scalar variable "@MyDate".
Msg 137, Level 15, State 2, Procedure isoyear, Line 7
Must declare the scalar variable "@MyDate".
Msg 137, Level 15, State 2, Procedure isoyear, Line 8
Must declare the scalar variable "@ISOyear".
Msg 137, Level 15, State 2, Procedure isoyear, Line 11
Must declare the scalar variable "@MyDate".
Msg 137, Level 15, State 2, Procedure isoyear, Line 12
Must declare the scalar variable "@ISOyear".
Msg 137, Level 15, State 2, Procedure isoyear, Line 14
Must declare the scalar variable "@ISOYear".

Bonjour,

il manque les parenthèses autour du paramètre de la fonction.

Il manque aussi le BEGIN ... END :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
ALTER FUNCTION fct_isoyear (@MyDate AS dateTime)
RETURNS INT 
AS
BEGIN
    DECLARE @ISOyear INT = DATEPART(YEAR, @MyDate);
 
    -- Special cases: Jan 1-3 may belong to the previous year
    IF (DATEPART(MONTH, @MyDate) = 1 AND DATEPART(ISO_WEEK, @MyDate) > 50)
        SET @ISOyear = @ISOyear - 1;
 
    -- Special case: Dec 29-31 may belong to the next year
    IF (DATEPART(MONTH, @MyDate) = 12 AND DATEPART(ISO_WEEK, @MyDate) < 45)
        SET @ISOyear = @ISOyear + 1
 
    RETURN @ISOYear;
END

07/01/2015, 14h49
Invité

Tiens c'est marrant j'ai bidouillé sur ce bout de code aussi hier !
Cette fonction ne marche pas bien de mes premiers tests donc je l'ai mise à la poubelle de suite.
En plus, quand il faut commencer par corriger la syntaxe, ça ne me donne pas beaucoup de confiance dans cette fonction. :mrgreen:

ça devrait marcher :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
CREATE FUNCTION dbo.F_ISOYEAR (@date AS dateTime)
RETURNS INT 
AS
BEGIN
 
    RETURN CASE 
              WHEN DATEPART(ISO_WEEK, @date) BETWEEN 2 AND 51
                 THEN YEAR(@date)
              WHEN DATEPART(ISO_WEEK, @date) = 1
                 THEN YEAR(DATEADD(day, 7, @date))
              ELSE YEAR(DATEADD(day, -7, @date))
END;

A +

Citation:

Envoyé par SQLpro

ça devrait marcher :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
CREATE FUNCTION dbo.F_ISOYEAR (@date AS dateTime)
RETURNS INT 
AS
BEGIN
 
    RETURN CASE 
              WHEN DATEPART(ISO_WEEK, @date) BETWEEN 2 AND 51
                 THEN YEAR(@date)
              WHEN DATEPART(ISO_WEEK, @date) = 1
                 THEN YEAR(DATEADD(day, 7, @date))
              ELSE YEAR(DATEADD(day, -7, @date))
END;

A +

Merci, j'ai refait mes tests d'hier et c'est tout beau.

07/01/2015, 17h54
olibara

Merci SqlPRO

2 in 1 :plusser:

1- Coup de projecteur sur la bonne syntaxe
2- Amélioration du code initial
:D
07/01/2015, 19h24
Invité

Bon je me rends compte qu'en ce qui me concerne, il reste toujours un petit problème car ISO_Week prend comme premier jour de la semaine le lundi...
Et mon client considère que c'est le dimanche...
Enfin, c'est pas bien grave, j'ai d'autres chats à fouetter aujourd'hui.
09/01/2015, 18h11
SQLpro

Citation:

Envoyé par 7gyY9w1ZY6ySRgPeaefZ

Bon je me rends compte qu'en ce qui me concerne, il reste toujours un petit problème car ISO_Week prend comme premier jour de la semaine le lundi...
Et mon client considère que c'est le dimanche...
Enfin, c'est pas bien grave, j'ai d'autres chats à fouetter aujourd'hui.

Dans ce cas il faut prendre non pas ISO_week, mais week.

A +
09/01/2015, 18h34
Invité

1 pièce(s) jointe(s)

Citation:

Envoyé par SQLpro

Dans ce cas il faut prendre non pas ISO_week, mais week.

Maheureusement, on se retrouve alors avec des semaines coupées en deux au changement d'année :

Pièce jointe 165811

Je me pencherais sur le sujet plus sérieusement dans les prochaines semaines.
09/01/2015, 20h37
olibara

Il n'y a pas de miracle

Une semaine c'est toujours 7 jours que ce soit Week ou Iso_Week

Et si tu divise 365 par 7 tu peux constater que ca ne te donne pas un chiffre entier ;)

Donc dans tous les cas tes semaines seront coupées entre deux années

A moins de decreter une année de 364 jours qui nous mettra Noel en été dans 180 ans
11/01/2015, 18h00
SQLpro
Citation:

Envoyé par 7gyY9w1ZY6ySRgPeaefZ

Maheureusement, on se retrouve alors avec des semaines coupées en deux au changement d'année :

Pièce jointe 165811

Je me pencherais sur le sujet plus sérieusement dans les prochaines semaines.

Dans tous les cas vous en aurez toujours.

La numérotation des semaines obéit à deux algorithmes normalisés :
- norme ASCII pour les US avec le dimanche comme premier jour de semaine
- norme ISO pour le reste du monde avec le lundi comme premier jour de semaine
Ces deux algorithme, tranche justement le fait du numéro de d'une semaine à cheval sur des années....

A +
11/01/2015, 19h29
olibara

Heeuuu !

Je crois que tu te trompe SQLPro

Pour la semaine ISO, le premier jour est le Lundi
Pour les US, le premier jour est le dimanche
12/01/2015, 09h23
SQLpro

Citation:

Envoyé par olibara

Heeuuu !

Je crois que tu te trompe SQLPro

Pour la semaine ISO, le premier jour est le Lundi
Pour les US, le premier jour est le dimanche

Effectivement.. J'ai rectificationné !!!!

A +