Discussion :

[Herveg]

Bonjour,

Pouvez vous m'aider à optimiser cette 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
select datecr,j.libelle,decode(substr(compte,1,2),'91','LV',
                                              '90','LS',
                                              '92','Ra',
                                              '93','CT') lib,sum(nvl(credit,0))
from  Xecriture e,Xjour j
where j.journal=e.journal
and   e.societe=j.societe and e.societe=1
and   j.journal in('11','12')
and   datecr between '01/03/09' and '31/03/09'
group by decode(substr(compte,1,2),'91','LV',
                                   '90','LS',
                                   '92','Ra',
                                   '93','CT') ,datecr,j.libelle
union 
select datecr,j.libelle, decode(substr(compte,1,1),'7','PA' ) lib,sum(nvl(credit,0))
from Xecriture e,Xjour j
where j.journal=e.journal
and   e.societe=j.societe and e.societe=1
and   j.journal in('11','12')
and   datecr between '01/03/09' and '31/03/09'
group by datecr,j.libelle, decode(substr(compte,1,1),'7','PA' )
union
 select datecr,j.libelle, decode(substr(compte,1,5),'44221','Tb')lib,  sum(nvl(credit,0))                                                
from Xecriture e,Xjour j
where j.journal=e.journal
and   e.societe=j.societe and e.societe=1
and   j.journal in('11','12')
and   datecr between '01/03/09' and '31/03/09'
group by datecr,j.libelle,                                         
decode(substr(compte,1,5),'44221','Tb')

Merci de votre aide

[fatsora]

Bonjour, tu ferais mieux de transerer ca a SQL et non ici

Sinon peux tu préciser la version oracle ?

Et peux tu poster le plan d'execution stp ?

[Waldar]

Le fait de faire trois unions dans votre requête mutiliplie votre credit par trois, est-ce volontaire (surtout avec le libellé qui est nul) ?

Regardez du côté de la fonction CASE, qui est un decode beaucoup plus moderne.
Profitez-en pour moderniser votre jointure également avec un INNER JOIN.
Comparez votre champ "datecr" avec des dates et non pas avec des littéraux.

[McM]

Les UNION sont coûteux car ils font un DISTINCT (donc un tri).
Attention à la date, mieux vaut toujours mettre une vraie date

Avec des decode en une seule requête ça marche.

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
SELECT datecr, j.libelle, 
	DECODE( SUBSTR(compte,1,5), 
		'44221','Tb', 
      		DECODE(SUBSTR(compte,1,2),
      			'91','LV', '90','LS', '92','Ra', '93','CT',
            		DECODE( SUBSTR(compte,1,1),'7','PA')
            )
      ) lib, SUM(NVL(credit,0))
FROM  Xecriture e, Xjour j
WHERE j.journal = e.journal
AND   e.societe = j.societe 
AND e.societe=1
AND   j.journal IN ('11','12')
AND   datecr BETWEEN TO_DATE('01/03/09','DD/MM/RR') AND TO_DATE('31/03/09', 'DD/MM/RR')
GROUP BY datecr, j.libelle, DECODE( SUBSTR(compte,1,5), 
			'44221','Tb', 
      	DECODE(SUBSTR(compte,1,2),
      			'91','LV', '90','LS', '92','Ra', '93','CT',
            	DECODE( SUBSTR(compte,1,1),'7','PA')
            	)
      )