Aide pour réécrire une requête

**facteur** · 18/06/2018, 15h26

Bonjour,
J'ai créé une requête SQL qui ne fonctionne malheureusement pas.
je penses que je l'ai mal écrite..
Y aurait il une âme charitable qui pourrait m'aider à reconstruire ma requête sql

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
45
46
47
 
 
DECLARE @EVENT_TYPE_SORTING   Int = 2  -- Tri
DECLARE @EVENT_TYPE_REJECT     Int = 34 -- Rejet
 
DECLARE @StartDateTime DateTime = '20180613 04:30:00'
DECLARE @EndDateTime DateTime =   '20180614 13:30:00'
 
 
SELECT 
Parcel.[Id] as id,
Tache.Num as TacheTri,
Chute.Num as num,
dsc.SortingType as type,
IDT13.[Barcode] as Cab_Suivi,
IDT24.[Barcode] as Cab_Pch,
IDT28.[Barcode] as Cab_Geolabel,
Parcel.[PostalCodeAcqMode] as Acquisition_PostalCode,
FROM [Chute] 
 
Left outer join  
 
(Vacation
           JOIN Parcel
                ON (Vacation.Debut BETWEEN @StartDateTime AND @EndDateTime OR 
                    Vacation.Fin BETWEEN @StartDateTime AND @EndDateTime) AND
                      Vacation.Id=Parcel.VacationId
           JOIN ParcelEvent
           ON Parcel.Id=ParcelEvent.ParcelId AND 
              ParcelEvent.EventTypeNum IN (@EVENT_TYPE_SORTING,@EVENT_TYPE_REJECT) AND
                      ParcelEvent.[DateTime] BETWEEN @StartDateTime AND @EndDateTime
 
Left outer join [Barcode] as IDT13 with(nolock) 
on IDT13.[ParcelId] = Parcel.[Id] and len(IDT13.[Barcode])='13' 
Left outer join [Barcode] as IDT24 with(nolock) 
on IDT24.[ParcelId] = Parcel.[Id] and len(IDT24.[Barcode])='24'
Left outer join [Barcode] as IDT28 with(nolock) 
on IDT28.[ParcelId] = Parcel.[Id] and len(IDT28.[Barcode])='28')
 
 
ON Chute.Num =Parcel.PhysicalChuteNum 
 
left join dsc
on dsc.chute=chute.num and dsc.Idtache=Vacation.idtache
 
join tache
on tache.id= dsc.IdTache

**escartefigue** · 18/06/2018, 15h43

Bonjour,

Elle ne fonctionne pas, c'est à dire ?
Un message d'erreur ? lequel ?
Un résultat inattendu ? quel résultat ? lequel est souhaité ?

**aieeeuuuuu** · 18/06/2018, 15h44

bonjour,

essayez de mettre un alias pour la sous requête en table dérivée.

sinon... avec le message d'erreur, ce serait plus pratique....

**III Jonathan III** · 18/06/2018, 15h55

Amha il manque le select et le from dans la sous requête

**escartefigue** · 18/06/2018, 15h58

Envoyé par III Jonathan III

Amha il manque le select et le from dans la sous requête

Exact bien vu

, et aussi, le prédicat de jointure avec la table dérivée, porte non pas sur l'alias (manquant) de la table dérivée, mais sur l'une des tables concourant à la construction de cette table dérivée (en l'occurrence "parcelle")

**aieeeuuuuu** · 18/06/2018, 16h43

en relisant, ce n'est en fait pas une sous requête, mais des jointures imbriquées.

Il paraitrait logique de mettre une jointure externe pour la table tache, puisqu'elle est jointe sur une table elle même en jointure externe.

quel est le message d'erreur ?

**facteur** · 19/06/2018, 11h23

Bonjour tout le monde,
ce n'est pas une sous requête mais bien des jointures imbriquées
je n'aie pas de message d'erreur ; la requête mouline sans jamais me donner de résultat.
A l'origine ma requete n'avait pas le bout de code suivant et elle fonctionnait trés bien

Code :

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

1
2
3
4
5
6
Left outer join [Barcode] as IDT13 with(nolock) 
on IDT13.[ParcelId] = Parcel.[Id] and len(IDT13.[Barcode])='13' 
Left outer join [Barcode] as IDT24 with(nolock) 
on IDT24.[ParcelId] = Parcel.[Id] and len(IDT24.[Barcode])='24'
Left outer join [Barcode] as IDT28 with(nolock) 
on IDT28.[ParcelId] = Parcel.[Id] and len(IDT28.[Barcode])='28')

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
45
46
DECLARE @EVENT_TYPE_SORTING   Int = 2  -- Tri
DECLARE @EVENT_TYPE_REJECT     Int = 34 -- Rejet
 
DECLARE @StartDateTime DateTime = '20180613 04:30:00'
DECLARE @EndDateTime DateTime =   '20180614 13:30:00'
 
 
SELECT 
Parcel.[Id] as id,
Tache.Num as TacheTri,
Chute.Num as num,
dsc.SortingType as type,
--IDT13.[Barcode] as Cab_Suivi,
--IDT24.[Barcode] as Cab_Pch,
--IDT28.[Barcode] as Cab_Geolabel,
Parcel.[PostalCodeAcqMode] as Acquisition_PostalCode
FROM [Chute] 
 
Left outer join  
 
(Vacation
           JOIN Parcel
                ON (Vacation.Debut BETWEEN @StartDateTime AND @EndDateTime OR 
                    Vacation.Fin BETWEEN @StartDateTime AND @EndDateTime) AND
                      Vacation.Id=Parcel.VacationId
           JOIN ParcelEvent
           ON Parcel.Id=ParcelEvent.ParcelId AND 
              ParcelEvent.EventTypeNum IN (@EVENT_TYPE_SORTING,@EVENT_TYPE_REJECT) AND
                      ParcelEvent.[DateTime] BETWEEN @StartDateTime AND @EndDateTime)
 
/*Left outer join [Barcode] as IDT13 with(nolock) 
on IDT13.[ParcelId] = Parcel.[Id] and len(IDT13.[Barcode])='13' 
Left outer join [Barcode] as IDT24 with(nolock) 
on IDT24.[ParcelId] = Parcel.[Id] and len(IDT24.[Barcode])='24'
Left outer join [Barcode] as IDT28 with(nolock) 
on IDT28.[ParcelId] = Parcel.[Id] and len(IDT28.[Barcode])='28')*/
 
 
 
ON Chute.Num =Parcel.PhysicalChuteNum 
 
left join dsc
on dsc.chute=chute.num and dsc.Idtache=Vacation.idtache
 
join tache
on tache.id= dsc.IdTache

Lorsque j'ai rajouter ces lignes, elle mouline sans jamais me donner de résultat

**SQLpro** · 19/06/2018, 11h42

Pour information votre requête est mathématiquement fausse d'un point de vu de gestion des intervalles de temps.

En effet quand l'on veut "matcher" des intervalles avec des valeurs il faut choisir de gérer ces intervalles soit :
OUVERT à gauche, FERMÉ à droite [ ... [
FERMÉ à gauche, OUVERT à droite [ ... [
Or l'opérateur BETWEEN travaille avec un intervalle OUVERT à droite et OUVERT à gauche ] ... [ (les bornes étant toutes les deux incluses). Vous aurez donc des intersections temporelles et non des continuités temporelles...

A +

**facteur** · 19/06/2018, 11h48

Bonjour sql pro
J'ai du mal à comprendre ..
L'opérateur between fonctionnait bien avant que je rajoute les ligne de codes avec les tables TD132,ID24 et ID28

**aieeeuuuuu** · 19/06/2018, 12h04

En supposant que chacune des jointures avec BarCode ne retourne qu'au plus une seule ligne, alors vous pouvez ecrire comme ceci :

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
 
DECLARE @EVENT_TYPE_SORTING INT= 2;  -- Tri
DECLARE @EVENT_TYPE_REJECT INT= 34; -- Rejet
 
DECLARE @StartDateTime DATETIME= '20180613 04:30:00';
DECLARE @EndDateTime DATETIME= '20180614 13:30:00';
 
 
SELECT Parcel.[Id] AS id,
       Tache.Num AS TacheTri,
       Chute.Num AS num,
       dsc.SortingType AS type,
	   Cab_Suivi,
	   Cab_Pch,
	   Cab_Geolabel,
       Parcel.[PostalCodeAcqMode] AS Acquisition_PostalCode
FROM [Chute]
     LEFT OUTER JOIN Vacation
                     JOIN Parcel ON(Vacation.Debut BETWEEN @StartDateTime AND @EndDateTime
                                    OR Vacation.Fin BETWEEN @StartDateTime AND @EndDateTime)
                                   AND Vacation.Id = Parcel.VacationId
                     JOIN ParcelEvent ON Parcel.Id = ParcelEvent.ParcelId
                                         AND ParcelEvent.EventTypeNum IN(@EVENT_TYPE_SORTING, @EVENT_TYPE_REJECT)
                                         AND ParcelEvent.[DateTime] BETWEEN @StartDateTime AND @EndDateTime
    LEFT JOIN (
		  SELECT 
				BarCode.ParcelId
			 ,	MAX(CASE WHEN LEN(BarCode.Barcode) = 13 THEN BarCode END ) AS Cab_Suivi
			 ,	MAX(CASE WHEN LEN(BarCode.Barcode) = 24 THEN BarCode END ) AS Cab_Pch
			 ,	MAX(CASE WHEN LEN(BarCode.Barcode) = 28 THEN BarCode END ) AS Cab_Geolabel
		  FROM 	BarCode 
		  WHERE LEN(BarCode.Barcode) IN (13, 24, 28)
		  GROUP BY BarCode.ParcelId
    ) AS BC
		 ON BC.ParcelId = Parcel.[Id]
    ON Chute.Num = Parcel.PhysicalChuteNum
LEFT JOIN dsc ON dsc.chute = chute.num
                      AND dsc.Idtache = Vacation.idtache
JOIN tache ON tache.id = dsc.IdTache;

**facteur** · 19/06/2018, 13h10

Bonjour Sql pro

dans ma requete

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
DECLARE @EVENT_TYPE_SORTING   Int = 2  -- Tri
DECLARE @EVENT_TYPE_REJECT     Int = 34 -- Rejet
 
DECLARE @StartDateTime DateTime = '20180613 04:30:00'
DECLARE @EndDateTime DateTime =   '20180614 13:30:00'
 
 
SELECT 
Parcel.[Id] as id,
Tache.Num as TacheTri,
Chute.Num as num,
dsc.SortingType as type,
IDT13.[Barcode] as Cab_Suivi,
IDT24.[Barcode] as Cab_Pch,
IDT28.[Barcode] as Cab_Geolabel,
Parcel.[PostalCodeAcqMode] as Acquisition_PostalCode,
FROM [Chute] 
 
Left outer join  
 
(Vacation
           JOIN Parcel
                ON (Vacation.Debut BETWEEN @StartDateTime AND @EndDateTime OR 
                    Vacation.Fin BETWEEN @StartDateTime AND @EndDateTime) AND
                      Vacation.Id=Parcel.VacationId
           JOIN ParcelEvent
           ON Parcel.Id=ParcelEvent.ParcelId AND 
              ParcelEvent.EventTypeNum IN (@EVENT_TYPE_SORTING,@EVENT_TYPE_REJECT) AND
                      ParcelEvent.[DateTime] BETWEEN @StartDateTime AND @EndDateTime
.......

j'ai réecrie la ligne de code 29

Code :

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

ParcelEvent.[DateTime] BETWEEN @StartDateTime AND @EndDateTime

en remplaçant les variables scalaires @StartDateTime et @EndDateTime par les dates en dure

Code :

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

ParcelEvent.[DateTime] BETWEEN '20180613 04:30:00'  AND '20180614 13:30:00'

Et ça marche..je ne comprend pas trés bien pourquoi ??

aieeeuuuuu, ta proposition fonctionne

**SQLpro** · 19/06/2018, 14h11

Quel est le type de données de vos colonnes Debut et Fin ?

Ensuite comme dit BETWEEN n'est pas adapté. Vous devez faire

Code :

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

Debut <= valeur1 AND Fin > valeur2

ou

Code :

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

Debut < valeur1 AND Fin >=  valeur2

et non ce que fait le BETWEEN à savoir :

Code :

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

Debut <= valeur1 AND Fin >= valeur2

A +

**facteur** · 19/06/2018, 15h41

Sql pro
A la question Quel est le type de données de vos colonnes Debut et Fin ? c'est un format dateTime
je comprend la subtilité entre le between et les > et >.
Mais cela ne répond pas la question pour lorsque j'utilise la fonction between avec des dates en dure cela fonctionne ?

**aieeeuuuuu** · 19/06/2018, 15h55

Lorsque vous mettez les valeurs en dur, le moteur utilise ces valeurs pour générer le plan d’exécution (plan optimisé pour ces valeurs précises de dates)

Lorsque vous utilisez des variables, le moteur cherche à générer un plan qui sera optimal quelles ques soient les valeurs des variables au moment de l’exécution. En effet, la même requête pourrait être réutilisée avec d'autres valeurs pour ces variables et le même plan d'exécution sera utilisé.

Vous pouvez ajouter des indicateurs en fin de requête pour forcer le moteur à optimiser son plan pour des valeurs particulières pour les variables :

Code :

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

1
2
3
4
5
6
 
OPTION (OPTIMIZE FOR (
	   @StartDateTime  = '20180613 04:30:00'
    ,   @EndDateTime  =   '20180614 13:30:00'
    )
)

**facteur** · 19/06/2018, 16h25

aieuuuuuuuuuuuuuuuu
formidale cette ligne de code ci dessous. je ne la connaissais pas. elle m'affiche presque instantanément la requête

Code :

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

1
2
3
4
5
OPTION (OPTIMIZE FOR (
	   @StartDateTime  = '20180312 04:30:00'
    ,   @EndDateTime  =   '20180402 04:30:00'
    )
)

Si j'ai bien compris, il faudrait qu j'utilise cette option lorsque mes requêtes demande du temps pour s'afficher ?

**aieeeuuuuu** · 19/06/2018, 16h36

C'est un peu plus compliqué que ça. en premier lieu, lorsqu'une requête parait trop longue, il faut trouver la raison.

Il faut être prudent avec les indicateurs et les utiliser uniquement si nécessaire...

**facteur** · 20/06/2018, 19h57

Bonjour Aieuuu

Je suis d'accord avec toi lorsque tu dis que lorsqu'une requête met du temps à s’exécuter , il faut trouver la raison.
J'ai trois questions à te poser lorsque tu écris

Il faut être prudent avec les indicateurs et les utiliser uniquement si nécessaire...

1) les indicateurs est ce ta ligne de code ?

Code :

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

1
2
3
4
5
OPTION (OPTIMIZE FOR (
	   @StartDateTime  = '20180312 04:30:00'
    ,   @EndDateTime  =   '20180402 04:30:00'
    )
)

2) A quel moment dois je me dire que les indicateurs sont nécessaires dans une requête SQL ?

3) Quel est le risque de mettre des indicateurs lorsque cela n 'est pas nécessaire dans une requête SQL ?

**aieeeuuuuu** · 21/06/2018, 10h11

Envoyé par facteur

1) les indicateurs est ce ta ligne de code ?

Code :

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

1
2
3
4
5
OPTION (OPTIMIZE FOR (
	   @StartDateTime  = '20180312 04:30:00'
    ,   @EndDateTime  =   '20180402 04:30:00'
    )
)

Oui, mais il en existe bien d'autres

Envoyé par facteur

2) A quel moment dois je me dire que les indicateurs sont nécessaires dans une requête SQL ?

Je ne sais pas s'il existe vraiment une bonne réponse à cette question...
si vous avez une requête anormalement longue, que vous identifiez clairement que le plan généré est mauvais et en cause dans la lenteur, et que vous ne ne parvenez pas à obtenir de bon résultats en réécrivant la requête, en indexant, etc,
bref, quand toutes les techniques d'optimisation n'ont pas abouti, alors les indicateurs peuvent être envisagés. ça reste un palliatif...

Envoyé par facteur

3) Quel est le risque de mettre des indicateurs lorsque cela n 'est pas nécessaire dans une requête SQL ?

Le principal risque est que le remède soit pire que le mal.
Une base de données évolue, parfois rapidement, et il se peut qu'un indicateur soit bénéfique un jour, et néfaste le lendemain... (suite à la modification des cardinalités, création d'indexes,...)
Il faut donc, en plus d'une utilisation parcimonieuse, réévaluer régulièrement la pertinence des indicateurs utilisés.
Afin de faciliter la maintenance, il peut être préférable d'utiliser des repères de plan plutôt que de mettre les indicateurs directement dans les requêtes.

**facteur** · 21/06/2018, 23h13

Merci pour ces indications aieuuuu
très très intéressant..
Je clôture cette discussion

**facteur** · 27/06/2018, 13h28

Bonjour aieuuuuuuuuuuuuu
je réouvre cette discussion car je souhaiterais modifier momentanément un morceau de ma requête( du moins la tienne de reqête lol).
Ci dessous, le bout de ta requête que je souhaiterais changer

Code :

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

1
2
3
4
5
6
7
8
SELECT 
				BarCode.ParcelId
			 ,	MAX(CASE WHEN LEN(BarCode.Barcode) = 13 THEN BarCode END ) AS Cab_Suivi
			 ,	MAX(CASE WHEN LEN(BarCode.Barcode) = 24 THEN BarCode END ) AS Cab_Pch
			 ,	MAX(CASE WHEN LEN(BarCode.Barcode) = 28 THEN BarCode END ) AS Cab_Geolabel
		  FROM 	BarCode 
		  WHERE LEN(BarCode.Barcode) IN (13, 24, 28)
		  GROUP BY BarCode.ParcelId

je voudrais obtenir, les champs pour lequels:
les lignes (BarCode.Barcode) = 13 soit null
les lignes (BarCode.Barcode) = 24 soit null
les lignes (BarCode.Barcode) = 28 soit null

Merci d'avance aieuuu

Aide pour réécrire une requête [2012]

Développement SQL Server

Discussions similaires

Partager

Partager