Procédure stockée résultat différent en fonction du paramètre

Bonne rencontre,

Je suis confronté à une procédure stockée qui m’avait l’air simple mais dont je ne trouve pas une solution élégante.

Voici la situation en simplifier.

Table banner (idbanner, titre, image)
Table siteweb (idsite, titre, url)
Table banner_siteweb (idbanner,idsite,nbrview)

Je souhaiterai passer dans ma PS réalise deux choses.

• Donner la liste de toutes les bannières quand on ne spécifie pas un idsite
• Donner la liste des bannières d’une seul site quand on spécifie un idsite

Ma procédure a donc comme argument @idsite int = null

Je ne désire pas faire un :

Code:

---

1 2 3 4 5 6 7 8 9

IF @idsite IS NULL BEGIN SELECT … END ELSE BEGIN SELECT … END

---

Ni avoir une query dynamique.

J’aurais aimé n’avoir qu’un seul SELECT mais je tourne en rond… quelque chose comme ceci :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

SELECT [dbo].[BannersPub].[idbanner], [dbo].[BannersPub].[Title], [dbo].[BannersPub].[image], FROM [Banner] LEFT JOIN [dbo].[banner_siteweb] ON [dbo].[banner_siteweb].[idbanner] = [dbo].[BannersPub].[idbanner] LEFT JOIN [dbo].[siteweb] ON [dbo].[siteweb].[idsite] = [dbo].[banner_siteweb].[idsite] WHERE (ISNULL (@idsite, [dbo].[siteweb].[idsite]) = [dbo].[siteweb].[idsite]) OR [dbo].[siteweb].[idsite] IS NULL

---

Bon le WHERE est bidon comme je n’arrive pas au résultat souhaité mais c’est pour vous monter vers quoi je me dirige.

Avez-vous une idée ? Un conseil ?

Merci pour votre aide.
Raphaël.

Un truc comme ça :

Code:

---

1 2	where idsite = @idsite OR CASE WHEN @idsite IS NULL THEN 1 ELSE 0 END = 1

---

Mais franchement, la version séparée des requêtes me parait plus claire.

Merci pour ton retour rapide. Comme je m'adresse à un dba, j'en profite. :)

On est en train de faire notre premier web service et essaye d'être carré. On se prend un peu la tête ici, on ne sait pas ce qu'il est préférable de faire.

Avoir plusieurs petites procédures stockées et découpées. Exemple : getAllLivre, getOneLivre, GetLivreByAuteur

Avoir une procédure stockée avec des IF et ELSE. Exemple : IF @IDlivre SELECT One livre ELSE SELECT All Livre et un autre IF qui test l'auteur

Commencer à travailler avec des vues et avoir notre PS qui attaque la vue.

Avant je travaillais avec

Code:

---

1 2 3 4 5 6 7 8

IF @idsite IS NULL BEGIN SELECT … END ELSE BEGIN SELECT … END

---

mais on trouve cela difficile à maintenir surtout quand la PS commence à faire x centaines de lignes.

Amicalement,
Raphaël.

Si idiste est NOT NULL :

Code:

---

where idsite = COALESCE(@idsite, idsite, )

---

Si idiste est NULL :

Code:

---

where COALESCE(idsite, '') = COALESCE(@idsite, idsite, '')

---

A +

Sur le coup, je dois avouer ne pas avoir compris. :?

Aurais-tu l'amabilité de m'expliquer ?

D'avance, merci.

Raphaël.

http://sqlpro.developpez.com/cours/sqlaz/select/#L5.2

A +

Bonjour,

Je trouve assez étonnant qu'on conseille de spécifier des filtres non-sargables, alors qu'on sait que c'est contre-performant.
Dans le cas du CASE, le plan est optimisé avec la valeur de la variable lors de la compilation, si tant est qu'elle en ait une (i.e. NULL).
Cela peut être néfaste aux exécutions suivantes de la requête ...

On peut donc s'en remettre à du SQL dynamique :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

DECLARE @sql nvarchar(max) = ' SELECT BP.idbanner, BP.Title, BP.image, FROM Banner AS B LEFT JOIN dbo.banner_siteweb AS BS ON BS.idbanner = BP.idbanner LEFT JOIN dbo.siteweb AS S ON S.idsite = BS.idsite WHERE 1 = 1' + CASE WHEN @idsite IS NULL THEN '' ELSE ' AND S.idsite = @_idsite' END   EXEC sp_executesql @sql , N'@_idsite int' , @_idsite = @idsite

---

@++ ;)