Discussion :

[olibara]

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 :

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

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

[aieeeuuuuu]

Bonjour,

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

Il manque aussi le BEGIN ... END :

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

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

[SQLpro]

ça devrait marcher :

Code :

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

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 +

[Invité]

ça devrait marcher :

Code :

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

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.

[olibara]

Merci SqlPRO

2 in 1

1- Coup de projecteur sur la bonne syntaxe
2- Amélioration du code initial

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

[SQLpro]

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 +

[Invité]

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 :



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

[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

[SQLpro]

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



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 +

[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

[SQLpro]

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 +