Discussion :

[Romulus1985]

Bonjour,

Je souhaite appeler une procédure dans une procédure.

Supposons 3 procédures P1, P2 et P3 qui nécessitent respectivement une variable V1 , V2 et V3 pour qu'elles s'exécutent correctement.

Comment puis-je faire pour qu'une procédure regroupe ces trois procédures (P1, P2 et P3) et que l'utilisateur puisse choisir d'éxecuter l'une d'entre elles ?

Merci à vous!

Romain

[Waldar]

C'est un peu léger en terme d'informations communiquées, mais avec des branchements IF sur vos paramètres vous devriez vous en sortir.

[SQLpro]

Dans le langage SQL :

• une fonction ne doit en aucun cas modifier l'état de la base de données, que ce soit au niveau du DML (INSERT, UPDATE, DELETE...) ni au niveau du DDL (CREATE, ALTER, DROP...) ni même au niveau du DCL (GRANT, REVOKE...).
• c'est le rôle d'une procédure de modifier l'état de la base de données d'une quelconque manière que ce soit...

On ne peut donc pas appeler une procédure dans une fonction, ne serait-ce que pour des problème de sécurité... Hélas, certains SGBD mal conçus permettent cela (PostGreSQL en particulier) ce qui a conduit à la fille de sécurité https://wiki.postgresql.org/wiki/A_G...ur_Search_Path dont la plupart des serveurs actuels peuvent être victimes !

A +

[Romulus1985]

Bonjour,

Comme c'est ma première procédure je galère un peu désolé pour la demande un peu vague au début.

Je dispose de trois procédures :
- CABYMONTH --> l'utilisateur doit préciser l'année en variable --> choix 1
- CABYquarter --> l'utilisateur doit préciser l'année en variable --> choix 2
- CA_BY_YEAR --> aucune variable --> choix 3

Je souhaite que l'utilisateur choisisse une de ces options (1,2 ou3) en appelant une procédure qui appellera ensuite la procédure stockée.
Autant j'ai compris comment créer une procédure mais pour qu'une procédure en appelle une autre je suis bloqué.
Je tiens à préciser qu'il n'y a aucune modif de table dans mes procédures (code ci-dessous).

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
CREATE PROCEDURE CABYMONTH
 
	@YEAR INT
 
AS 
 
select 
CustomerName
, month(Invoicedate) as Monthdate
 ,sum(SIL.QUANTITY * SIL.UnitPrice) as TotalInvoices
 
from sales.Orders as SO
LEFT JOIN Sales.OrderLines AS SOL on SO.OrderID=SOL.OrderID
LEFT JOIN Sales.Invoices AS SI on SO.OrderID=SI.OrderID
LEFT JOIN Sales.InvoiceLines as SIL  on SIL.InvoiceID=SI.InvoiceID 
Left join sales.customers as SC on SO.CUSTOMERID=SC.CustomerID
 
Where SI.InvoiceID is not null and SOL.StockItemID =SIL.StockItemID 
and year(Invoicedate) =@YEAR
 
 
Group by SC.CustomerName, month(Invoicedate)
 
Order by SC.CustomerName, month(Invoicedate)
 
GO

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
 
CREATE PROCEDURE CABYquarter
 
	@YEAR INT
 
AS 
 
select 
CustomerName
, ((month(Invoicedate)+2)/3) as Monthdate
 ,sum(SIL.QUANTITY * SIL.UnitPrice) as TotalInvoices
 
from sales.Orders as SO
LEFT JOIN Sales.OrderLines AS SOL on SO.OrderID=SOL.OrderID
LEFT JOIN Sales.Invoices AS SI on SO.OrderID=SI.OrderID
LEFT JOIN Sales.InvoiceLines as SIL  on SIL.InvoiceID=SI.InvoiceID 
Left join sales.customers as SC on SO.CUSTOMERID=SC.CustomerID
 
Where SI.InvoiceID is not null and SOL.StockItemID =SIL.StockItemID 
and year(Invoicedate) =@YEAR
 
 
Group by SC.CustomerName, ((month(Invoicedate)+2)/3)
 
Order by SC.CustomerName, ((month(Invoicedate)+2)/3)
GO

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
CREATE PROCEDURE CA_BY_YEAR
 
AS 
 
select 
CustomerName
, YEAR(Invoicedate) as YEARdate
 ,sum(SIL.QUANTITY * SIL.UnitPrice) as TotalInvoices
 
from sales.Orders as SO
LEFT JOIN Sales.OrderLines AS SOL on SO.OrderID=SOL.OrderID
LEFT JOIN Sales.Invoices AS SI on SO.OrderID=SI.OrderID
LEFT JOIN Sales.InvoiceLines as SIL  on SIL.InvoiceID=SI.InvoiceID 
Left join sales.customers as SC on SO.CUSTOMERID=SC.CustomerID
 
Where SI.InvoiceID is not null and SOL.StockItemID =SIL.StockItemID 
 
Group by SC.CustomerName, YEAR(Invoicedate)
 
Order by SC.CustomerName, YEAR(Invoicedate)
 
GO

[SQLpro]

Donc, aucun intérêt de faire une procédure. Une simple et unique fonction table suffira et sera plus optimisée, surtout si elles doit être jointe avec d'autres tables, vues...

D'autre part vous avez mélangé les jointures JOIN ON et le WHERE...
Vous annulez les jointures externes avec votre clause WHERE !

Donc en une seule fonction :

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
CREATE FUNCTION dbo.F_CA_BY (@DATEPART VARCHAR(16), @YEAR INT)
RETURNS TABLE
AS 
SELECT CustomerName,
       @YEAR AS YEAR_DATE,
       CASE @DATEPART 
          WHEN 'year'    THEN YEAR(Invoicedate) 
          WHEN 'quarter' THEN DATEPART(quarter, Invoicedate) 
          WHEN 'month'   THEN MONTH(Invoicedate) 
       END as YEAR_PART,
       sum(SIL.QUANTITY * SIL.UnitPrice) as TotalInvoices
FROM   sales.Orders as SO
       JOIN Sales.OrderLines AS SOL 
          ON SO.OrderID=SOL.OrderID
       JOIN Sales.Invoices AS SI 
          ON SO.OrderID=SI.OrderID
       JOIN Sales.InvoiceLines AS SIL  
          ON SIL.InvoiceID=SI.InvoiceID 
             AND SOL.StockItemID =SIL.StockItemID --> ce que vous aviez mis dans le WHERE est qui fait partit de la jointure !
       JOIN sales.customers AS SC 
         ON SO.CUSTOMERID=SC.CustomerID
WHERE  SI.InvoiceID IS NOT NULL
       AND YEAR(Invoicedate) =@YEAR
GROUP  BY SC.CustomerName, 
       CASE @DATEPART 
          WHEN 'year'    THEN YEAR(Invoicedate) 
          WHEN 'quarter' THEN DATEPART(quarter, Invoicedate) 
          WHEN 'month'   THEN MONTH(Invoicedate) 
      END;
GO

Que vous pouvez utiliser comme ceci :

Code :

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

1
2
3
SELECT *
FROM    dbo.F_CA_BY('quarter', 2020)
ORDER  BY CustomerName, YEAR_PART;

A +

[Romulus1985]

J'ai ce problème la quand je tente de créer la fonction. J'ai essayé de corriger mais j'avoue que je suis perdu. Pourriez vous m'aider svp ?

"Vous annulez les jointures externes avec votre clause WHERE !" --> Merci beaucoup je n'y avais pas pensé!

[SQLpro]

Il manque juste le mot clé RETURN entre le AS et le SELECT.

A +

[Romulus1985]

Merci beaucoup!

J'ai beaucoup appris avec ton code et je te remercie vraiment d'avoir pris le temps de le faire!

Bonne journée