Procédures Stockées IF ELSE / BEGIN END [Résolu]

[olibara]

Bonjour

Je tatonne toujours a chercher les bonnes syntaxe en procédure stockée.
En C# j'aurais déja écrit ca de maniere tres simple mais j'aimerais quand meme essayer de maitriser la syntaxe en SP

J'ai écrit la petite procedure suivante ca compile mais je ne sais pas si ca s'execute je pense que c'est vraiment mauvais comme maniere d'ecrire

Plusieurs questions ;
1 Je n'ai pas du faire de bloc BEGIN END pour mon IF et mon ELSE c'est normal ?

2 Le ELSE est un cas particulier du IF si ChauffeurID a été défini, avez vous une suggestion pour mieux integrer cette situation sans reecrire toute la methode comme je l'ai fait ?

Merci de m'aider pour mieux comprendre cette syntaxe (vraiment mal documentée )

Code :

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
USE [Clark]
GO
/****** Object:  StoredProcedure [dbo].[sp_DoTotal]    Script Date: 03/06/2011 00:12:00 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
 
ALTER PROCEDURE [dbo].[sp_DoTotal]
   @ChauffeurID  int,
   @TotalType  int
AS
SET @TotalType = (SELECT 
                     CASE @TotalType
                        WHEN 0 THEN 0
                        WHEN 9 THEN 9
                        ELSE 0
                     END)
IF @ChauffeurID IS NULL
BEGIN
INSERT INTO  
 [Clark].[dbo].TotalChauffeur 
 (
   ChauffeurID,
   ChauffeurName,
   Date,
   Gewicht
  )
SELECT 
      [ChauffeurID]
      ,[ChauffeurName]
      ,MAX(Date) AS Date
      ,SUM([bruto])-sum([tarra]) AS Gewicht
  FROM [Clark].[dbo].[BakMove]
  WHERE ChauffeurID IN (SELECT ChauffeurID FROM [BakMove] WHERE TotalStatus=@TotalType)
  GROUP BY 
  [ChauffeurID]
  ,[ChauffeurName]
  ,CAST(Date AS Date)
  END
ELSE
  BEGIN
INSERT INTO  
 [Clark].[dbo].TotalChauffeur 
 (
   ChauffeurID,
   ChauffeurName,
   Date,
   Gewicht
  )
SELECT 
      [ChauffeurID]
      ,[ChauffeurName]
      ,MAX(Date) AS Date
      ,SUM([bruto])-sum([tarra]) AS Gewicht
  FROM [BakMove]
  WHERE ChauffeurID IN 
  (SELECT 
    ChauffeurID 
    FROM [BakMove] 
    WHERE TotalStatus=@TotalType
    AND ChauffeurID=@ChauffeurID)
  GROUP BY 
  [ChauffeurID]
  ,[ChauffeurName]
  ,CAST(Date AS Date)  
  END
  RETURN

[hmira]

1 - Le BEGIN et le END pour le IF et le ELSE ne sont pas nécessaires, mais ils donnent de la clarté dans le code. Personnellement j'aurais pas pareil. Donc tu peux les garder !
2 - Le seul moyen que je connais, sous SQL Server, et qui permet de répondre à ce genre de situation, (c.à.d, ne pas réécrire plusieurs fois la mêmes requêtes avec des variantes légères), est le sql dynamique (EXECUTE sp_sqlexecute) voir le lien ci-dessous pour plus de détails:
http://msdn.microsoft.com/en-us/library/ms188001.aspx

3 - Je te suggère, même si cela n'est pas obligatoire (a) de rajouter systématiquement un point virgule à la fin de chaque inscruction T-SQL. En effet cela fait partie de la norme ANSI SQL-2
Exemple :

Code :

1
2
3
4
5
6
SET @TotalType = (SELECT 
                     CASE @TotalType
                        WHEN 0 THEN 0
                        WHEN 9 THEN 9
                        ELSE 0
                     END); 

(a) Le point virgule est parfois obligatoire : avant une déclaration de CTE, ou avant certaines instructions du Services Broker, etc.

A+

[olibara]

merci hmira, je vais étudier cela !

[SQLpro]

Je ne voit pas pourquoi vous faîtes une procédure alors qu'une une seule requête tout va :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
INSERT INTO Clark.dbo.TotalChauffeur 
(ChauffeurID,
 ChauffeurName,
 Date,
 Gewicht)
SELECT ChauffeurID
      ,ChauffeurName
      ,MAX(Date)
      ,SUM(bruto)-sum(tarra)
  FROM Clark.dbo.BakMove
  WHERE ChauffeurID IN (SELECT DISTINCT COALESCE(@ChauffeurID, ChauffeurID) 
                        FROM   BakMove 
                        WHERE  TotalStatus=CASE @TotalType
                                              WHEN 0 THEN 0
                                              WHEN 9 THEN 9
                                              ELSE 0
                                           END)
  GROUP BY ChauffeurID
          ,ChauffeurName
          ,Date

A +

[olibara]

Merci SqlPro

Effectivement ta requete est beaucoup plus elegante, c'est exactement ce que j'espérais et je vais l'utiliser dans ma procedure.
(je vais aussi lire la doc pour comprendre l'usage de COALESCE

Code :

SELECT DISTINCT COALESCE(@ChauffeurID, ChauffeurID)

En gros je suppose que c'est un peu la meme chose que

Code :

ISNULL(@ChauffeurID, ChauffeurID)

N.B.:J'utilise une procedure pour decharger l'application de l'appel de la requete
L'application etant sur un device mobile connecté en WIFI, j'essaye de minimiser les dialogues entre le serveur et le device
Cette procedure pourrait alors elle meme etre appellée par un Trigger

[olibara]

Bonjour SqlPro

J'utilise souvent une petite astuce que tu n'avais pas repris dans ton exemple

Dans le GROUP BY je fais

Code :

CAST(Date AS DATE)

Et dans le Select je fais

Code :

MAX(Date)

Cela me permet de grouper toutes les operations d'un meme jour et de conserver la date de la derniere opération

[elsuket]

Bonjour,

Effectivement ISNUL() et COALESCE() remplissent une fonction (sans jeu de mots) similaire, à ceci près que :

- ISNULL() ne prend que deux paramètres
- COALESCE() en prend autant que vous voulez (je ne connais pas la limite )

Les deux fonctions retournent NULL si toutes les valeurs sont NULL.

@++

[SQLpro]

Et COALESCE c'est la norme SQL. Vous le retrouverez donc sur tous les SGBDR alors que ISNULL (en un seul mot) est spécifique à SQL Server.

A +

[olibara]

Merci a tous !