Discussion :

[arthuro45]

Bonjour,

Ce bout de code est issue de ma 1er procédure stockée .
J'ai un problème de syntaxe sur la condition IF, j'ai aussi hésité a la remplacer par CASE.
Merci pour votre aide.

Code :

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

IF @date1 = NULL @date1 = 'ok'  else @date1 'pas ok'

Le message d'erreur :

Code :

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

Syntaxe incorrecte vers '@date1'

[mikedavem]

Bonjour,

Les 2 sont possibles : (je suppose que @date1 n'est pas une date ...)

Code :

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

1
2
3
4
IF @date1 IS NULL 
 SET @date1 = 'ok'  
ELSE 
 SET @date1 = 'pas ok'

ou

Code :

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

1
2
3
SET date1 = CASE WHEN @date1 IS NULL THEN 'ok'
                          ELSE 'pas ok'
                  END

++

[arthuro45]

(je suppose que @date1 n'est pas une date

Si c'est une date, ça pose problème à priori ?!

[mikedavem]

Oui parce que vous réaffectez à la variable @date1 est une valeur qui n'est pas conforme ('ok' ou 'pas ok')

Dans ce cas il faut ajouter une 2ème variable @result par exemple ..

Code :

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

1
2
3
4
IF @date1 IS NULL 
 SET @result = 'ok'  
ELSE 
 SET @result = 'pas ok'

Sinon postez votre code entier de procédure pour voir si on peut procéder autrement

++

[arthuro45]

J'ai modifié la condition, c'est mieux avec une date plutôt qu'un varchar

Code :

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

1
2
3
4
IF @date1 IS NULL 
     SET @date1 = '2010-01-01'  
    ELSE 
     SET @date1 = @date1

Si c'est hors post je comprendrais, sinon pourriez-vous me dire si mon début de procédure vous parait correcte :

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
ALTER PROCEDURE [dbo].[CALCUL_TEST](@matric int)
AS
 
DECLARE @pb int;
DECLARE @pm int;
DECLARE @abs1 int;
DECLARE @abs2 int ;
DECLARE @date1 datetime;
DECLARE @date2 datetime;
 
-- Récupération des valeurs nécéssaires au calcul de la prime individuelle
SELECT
 @date1=d_entree
,@date2=d_sortie
,@abs1=absenteisme1
,@abs2=absenteisme2
,@pm=part_manageriale
FROM 
utilisateur_gldr A
WHERE 
peg = 1
AND A.annee = 2010
AND MATRIC = @matric 
 
    -- On vérifit l'existance de la prime de base (@pb)
    IF NOT EXISTS(SELECT prime_Debase FROM peg_historique_calcul WHERE annee = 2010)
BEGIN
    PRINT 'Il manque la prime de base';
    RETURN;
END
    IF EXISTS (SELECT prime_Debase FROM peg_historique_calcul WHERE annee = 2010)
BEGIN
    PRINT 'Le calcul peut commencer :';
 
    IF @date1 IS NULL 
     SET @date1 = '2010-01-01'  
    ELSE 
     SET @date1 = @date1
 
    PRINT @date1;
    RETURN;
 
END

[mikedavem]

Moi j'enleverais la partie IF .. ELSE et je modifierais le 1er SELECT :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
-- Récupération des valeurs nécéssaires au calcul de la prime individuelle
SELECT
 @date1=ISNULL(d_entree, '2010-01-01')
,@date2=d_sortie
,@abs1=absenteisme1
,@abs2=absenteisme2
,@pm=part_manageriale
FROM 
utilisateur_gldr A
WHERE 
peg = 1
AND A.annee = 2010
AND MATRIC = @matric
.....

++