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