Discussion :

[steve15]

Bonjour,

Est-ce qu'il y a moyen de faire la requête ci-dessous plus simplifiée et plus propre ?

Merci.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT        TOP (100) PERCENT dbo.E_ANLA.ANLN1, dbo.E_ANLA.TXT50, dbo.E_ANLA.GDLGRP, dbo.E_ANLA.ORD43, dbo.E_ANLA.ANLN2, dbo.E_ANLZ.STORT
FROM            dbo.E_ANLA 
 
				INNER JOIN dbo.E_ANLZ ON dbo.E_ANLA.ANLN1 = dbo.E_ANLZ.ANLN1
 
				INNER JOIN (SELECT        ANLN1, MAX(ANLN2) AS Max2
                               FROM            dbo.E_ANLA
                               GROUP BY ANLN1) ReqMax ON dbo.E_ANLA.ANLN1=ReqMax.ANLN1 and dbo.E_ANLA.ANLN2=ReqMax.Max2
 
WHERE        (dbo.E_ANLZ.BDATU = '99991231') AND (dbo.E_ANLZ.GSBER = '0001')
ORDER BY dbo.E_ANLA.ANLN1

Comme vous pouvez le voire, le but est de prendre le MAX du champ ANLN2 de la table E_ANLA, faire une jointure avec ANLN1 qui se trouvent dans les 2 table spour récupérer le champ STORT de E_ANLZ, et filtrer avec le WHERE à la fin sur les 2 champs BDATU et GSBER de la table E_ANLZ

[al1_24]

Plus propre, certainement : en utilisant des alias sur les noms de tables, en déplaçant la restriction sur la valeur maximum de ANL2 dans la clause WHERE, en supprimant le TOP (100) PERCENT sans intérêt...

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
SELECT  NLA.ANLN1
    ,   NLA.TXT50
    ,   NLA.GDLGRP
    ,   NLA.ORD43
    ,   NLA.ANLN2
    ,   NLZ.STORT
FROM    dbo.E_ANLA  NLA
    INNER JOIN
        dbo.E_ANLZ  NLZ
        ON  NLA.ANLN1 = NLZ.ANLN1
WHERE   EXISTS
        (   SELECT  NULL
            FROM    dbo.E_ANLA Req
        	WHERE	NLA.ANLN1 = Req.ANLN1
            HAVING  NLA.ANLN2 = MAX(Req.ANLN2)
        )
    AND NLZ.BDATU = '99991231'
    AND NLZ.GSBER = '0001'
ORDER BY NLA.ANLN1
;

[SQLpro]

Le top ne sert à rien si ce n'est que faire perdre de la performances.
Récrivez comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT A.ANLN1, A.TXT50, A.GDLGRP, A.ORD43, A.ANLN2, Z.STORT
FROM   dbo.E_ANLA AS A
       INNER JOIN dbo.E_ANLZ AS Z ON A.ANLN1 = Z.ANLN1
  		 INNER JOIN (SELECT MAX(ANLN2) AS Max2
                   FROM   dbo.E_ANLA
                   GROUP  BY ANLN1) ReqMax 
             ON A.ANLN1 = ReqMax.ANLN1 and A.ANLN2=ReqMax.Max2
WHERE  Z.BDATU = '99991231' AND Z.GSBER = '0001'
ORDER  BY ANLN1

Vous pouvez tester aussi :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
WITH T AS 
(
SELECT A.ANLN1, A.TXT50, A.GDLGRP, A.ORD43, A.ANLN2, Z.STORT, 
       MAX(ANLN2) OVER(PARTITION BY ANLN1 ) AS MAX2
FROM   dbo.E_ANLA AS A
       INNER JOIN dbo.E_ANLZ AS Z ON A.ANLN1 = Z.ANLN1
WHERE  Z.BDATU = '99991231' AND Z.GSBER = '0001'
)
SELECT ANLN1, TXT50, GDLGRP, ORD43, ANLN2, STORT
FROM   T
WHERE  ANLN2 = MAX2
ORDER  BY ANLN1;

Qui ira sans doute plus vite !

Comme vous n'avez pas respecter la charte de postage il est impossible de vérifier ces requêtes !
La charte : http://www.developpez.net/forums/a69...gage-sql-lire/

A +

[steve15]

Super, merci ,tout fonctionne.

Cependant, si je souhaite rajouter un MAX également sur le champ BDATU de la table ANLZ, de quelle façon faut-il procéder ?

Merci

[SQLpro]

De la même faço, mais tout dépend du groupage que vous voulez :
Pour un max absolu :

Code :

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

MAX(BDATU) OVER() AS MAX3

Pour un max relatif :

Code :

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

MAX(BDATU) OVER(PARTITION BY ANLN1 ) AS MAX3

A +