Discussion :

[olibara]

Bonjour

J'ai écrit cette requête en passant par des variable pour éviter de faire des CAST(datetime as date) et des calcul sur GetDate() dans l’exécution

Et j'aurais voulu en faire une vue
Mais je constate hélas qu'il n'est pas possible de déclarer des Variables dans une vue

Existe-t-il une autre manière ?

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
15
16
17
18
19
DECLARE @Today AS Datetime
DECLARE @MONBEG AS Datetime
DECLARE @YEARBEG AS Datetime
SET @Today=CAST(GETDATE() AS DATE)
SET @MONBEG=DATEADD(month, DATEDIFF(month, 0, @Today), 0)
SET @YEARBEG=DATEADD(yy, DATEDIFF(yy, 0, @Today), 0)
 
SELECT
 
dbo.TradeInBaskets.AffiliateId,
SUM(CASE WHEN TradeInBaskets.DateTime BETWEEN @YearBeg AND @MonBeg THEN 1 else 0 END) AS YTMBeg ,
SUM(CASE WHEN TradeInBaskets.DateTime BETWEEN @MONBEG AND @Today THEN 1 else 0 END) AS MTYest ,
SUM(CASE WHEN TradeInBaskets.DateTime >@Today THEN 1 else 0 END) AS Today ,
SUM(CASE WHEN TradeInBaskets.IsCancelled=1 THEN 1 ELSE 0 END ) AS NbCancel,
SUM(CASE WHEN TradeInBaskets.IsCancelled=0 THEN TradeInBaskets.TradeInPrice ELSE 0 END) AS Value
 
from TradeInBaskets
WHERE DateTime>@YEARBEG
GROUP by Affiliateid

[al1_24]

Et qu'est-ce qui t’empêche de mettre tes expressions de date dans la requête ?
C'est peut-être un peu lourd à la relecture mais elles ne seront calculées qu'une fois lors de l'exécution.

[aieeeuuuuu]

Bonjour,

En effet, une vue ne doit contenir qu'une requête et rien d'autre.
Vous pourriez passer par une fonction table multi-instruction, mais vous risquez de perdre beaucoup en performances.

Concernant le fait de mettre vos calculs directement dans la requête, au niveau de performances, les calculs ne seront pas faits pour chaque ligne, mais bien une seule fois pour la requete.
Si c'est pour un problème de lisibilité/maintenance, vous pouvez passer par par une CTE (ou sous requete) pour factoriser le code.

Quelque chose comme ceci par exemple (pas testé)

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
 
WITH CTE(Today, monbeg, yearbeg) AS ( 
	SELECT 
			CAST(GETDATE() AS DATE)
		,	DATEADD(month, DATEDIFF(month, 0, @Today), 0)
		,	DATEADD(yy, DATEDIFF(yy, 0, @Today), 0)
)
SELECT  
	dbo.TradeInBaskets.AffiliateId,
	SUM(CASE WHEN TradeInBaskets.DateTime BETWEEN YearBeg AND MonBeg THEN 1 else 0 END) AS YTMBeg ,
	SUM(CASE WHEN TradeInBaskets.DateTime BETWEEN MONBEG AND Today THEN 1 else 0 END) AS MTYest ,
	SUM(CASE WHEN TradeInBaskets.DateTime >Today THEN 1 else 0 END) AS Today ,
	SUM(CASE WHEN TradeInBaskets.IsCancelled=1 THEN 1 ELSE 0 END ) AS NbCancel,
	SUM(CASE WHEN TradeInBaskets.IsCancelled=0 THEN TradeInBaskets.TradeInPrice ELSE 0 END) AS Value
 
from TradeInBaskets
CROSS JOIN CTE 
WHERE DateTime>YEARBEG
GROUP by Affiliateid

[olibara]

excellent !

Merci Aieeeuuuuu

J'avais oublié que c'était possible !

[SQLpro]

Bonjour,

En effet, une vue ne doit contenir qu'une requête et rien d'autre.
Vous pourriez passer par une fonction table multi-instruction, mais vous risquez de perdre beaucoup en performances.

Il peut dans ce cas faire une fonction table en ligne et conserver toutes les performances... Mais je n'en voit pas l'intérêt...

A +

[olibara]

Pour ma part la Solution proposé par Aieeeuuu permet

- une bien meilleure lisibilité (ce qui aide beaucoup la compréhension et la maintenance
- Cela évite les erreurs liée a une syntaxe trop encombrée
- Et A mon avis cela n'affecte pas la performance

[olibara]

Bonjour

J'ai aménagé la solution d'Aieeeuuuu qui ne fonctionne pas tel que (il avait averti ne pas avoir testé)

Il y avait deux soucis :
- Le CTE n'autorise pas la reference directe a @Today calculé dans l'expression
- Il n'est pas possible d'utiliser les élément de CTE sans faire référence a celle ci (j'ai résolu en faisant un JOIN virtuel)

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
WITH CTE(Today, monbeg, yearbeg) AS ( 
	SELECT TOP 1
			CAST(GETDATE() AS DATE) AS Today
		,	DATEADD(month, DATEDIFF(month, 0, CAST(GETDATE() AS DATE)), 0) AS MonBeg
		,	DATEADD(yy, DATEDIFF(yy, 0, CAST(GETDATE() AS DATE)), 0) AS YearBeg
)
SELECT
dbo.TradeInBaskets.AffiliateId,
SUM(CASE WHEN TradeInBaskets.DateTime BETWEEN CTE.YearBeg AND CTE.MonBeg THEN 1 else 0 END) AS YTMBeg ,
SUM(CASE WHEN TradeInBaskets.DateTime BETWEEN CTE.MONBEG AND Today THEN 1 else 0 END) AS MTYest ,
SUM(CASE WHEN TradeInBaskets.DateTime >CTE.Today THEN 1 else 0 END) AS Today ,
SUM(CASE WHEN TradeInBaskets.IsCancelled=1 THEN 1 ELSE 0 END ) AS NbCancel,
SUM(CASE WHEN TradeInBaskets.IsCancelled=0 THEN TradeInBaskets.TradeInPrice ELSE 0 END) AS Value
 
from TradeInBaskets
INNER JOIN CTE ON CTE.monbeg=CTE.monbeg
WHERE DateTime>YEARBEG
GROUP by Affiliateid

[StringBuilder]

Bonjour

Quelle est la raison du "TOP 1" qui se trouve dans la CTE ?

En effet, la CTE ne peut pas avoir plus d'une ligne, et mise à part si avoir explicitement un TOP 1 déclenche quelque changement dans le plan d'exécution (ce dont je toute un peu), il ne sert à rien.

Quand à votre "join virtuel", votre syntaxe est une hérésie, illisible, et potentiellement non pérenne.

"INNER JOIN" permet de lier une table à l'ensemble des tables déjà présentes dans la requête. Avec certains SGBD, le INNER JOIN serait rejeté, puisqu'il ne joint CTE à aucune autre table, et qui plus est, il compare deux champs d'une même table, ce qui est un filtre (certains SGBD tels qu'Access ne le supportent pas dans un JOIN). Donc sauf filtre sémantiquement important (du genre "type_personne = 'client'), un tel test ne doit se trouver que dans la clause WHERE.

Comme dans l'exemple d'aieeeuuuuu, utilisez donc CROSS JOIN pour ajouter une table à votre requête sans jointure. Il se produit alors un produit cartésien, ce que votre INNER JOIN foireux fini par faire faute de faire une jointure.

[olibara]

Merci Stringbuilder

Tu a raison

1- Le TOP c'est' juste une habitude d'écriture quand je sais que je ne m'attends qu'a une ligne
2- De fait je n'avais pas vu le CROSS JOIN de Aieeeuuuu et comme effectivement sans JOIN ca ne peut pas marcher, j'ai inventé ce INNER JOIN Virtuel (Mais je prends note de la remarque)

[SQLpro]

1- Le TOP c'est' juste une habitude d'écriture quand je sais que je ne m'attends qu'a une ligne

hou quelle vilaine habitude !!!

A +