Discussion :

[the dude]

Bonjour à tous,

Je cherche depuis maintenant quelques jours comment optimiser le code d'une requête.

En fait, cette procédure stockée reçoit des paramètres d'entrée et en fonction d'un d'entre eux, une condition dans un where doit être retiré de cette façon :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT distinct v.IdIlot,v.IDMachine    
    INTO #FR_AD_iIlots
    FROM dbo.verres_rebutes_operation v
        INNER JOIN #FR_AD_substrat sub
        ON v.CodeSubstat = sub.substrat
        INNER JOIN #FR_AD_surf surf
        ON v.CodeProcedeVerre = surf.surf
        INNER JOIN #FR_AD_cellule c
        ON v.Id_Cellule = c.cellule
    WHERE v.CrenduMission = '004'
        AND v.FlagProximity = @proximity
        AND v.CodeRebut = @rebutChoisi
        AND v.datefin BETWEEN @dated AND @datef

Cette ligne en rouge, en fonction de la valeur de @proximity,est conditionnel.
Je comptais mettre en If dedans mais ça ne marche pas : je reçois une erreur disant qu'il existe déjà une table #FR_AD_iIlots.

Pour l'instant, la seule solution que j'ai trouvé est de faire un double de toute la requête (ce que je montre n'est qu'un bout de la qry) avec d'autres noms de tables temporaires ...

Est-ce que vous auriez une idée pour améliorer cela ?

Merci d'avance

[Avatar]

Bonjour,

La solution est d'utiliser l'algèbre de Boole.
Tout est très bien expliqué dans ce site par exemple : http://weblogs.sqlteam.com/jeffs/arc...11/14/513.aspx

[Invité]

Bonsoir,

(...) je reçois une erreur disant qu'il existe déjà une table #FR_AD_iIlots.

Comme l'indique précisément le message d'erreur, il est impossible de créer deux tables portant le même nom.
Et ce même pour des tables temporaires.
Il faut exécuter un drop table #FR_AD_iIlots entre chaque test.

[soazig]

Bonjour
Une solution serait de remplacer

Code :

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

AND v.FlagProximity = @proximity

Par

Code :

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

AND (@proximity is null or v.FlagProximity = @proximity)

Il y en a surement d'autres.
Cordialement
Soazig

[the dude]

merci beaucoup pour toutes ces réponses !!
Effectivement en suivant le lien de Avatar, j'ai pu alléger mon code : merci !!