Discussion :

[Raphael1980]

Bonjour,

Dans une procédure stockée, je reçois une variable varchar qui contient une suite d’ID sous cette forme 3,4,5,6

Je voudrais trouver toutes les enregistrements avec ces ID.

J’ai fait comme ça : (il y a pas une méthode plus correcte ?)

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
ALTER PROCEDURE [dbo].[GestionRetourMaterielEtat]
	-- Add the parameters for the stored procedure here
	@PIDDossierTab varchar(300)
 
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;
 
    -- Insert statements for procedure here
    DECLARE @sql AS VARCHAR(255)
 
    SET @sql = 'SELECT  
    Saturna.dbo.dossier.NumDossier
    FROM Saturna.dbo.dossier 
    LEFT JOIN Saturna.dbo.Client_Dossier ON Saturna.dbo.Client_Dossier.IDdossier  = Saturna.dbo.dossier.IDdossier 
    LEFT JOIN Saturna.dbo.Adresse_Dossier ON Saturna.dbo.Adresse_Dossier.IDdossier = Saturna.dbo.dossier.IDdossier 
    WHERE Saturna.dbo.dossier.IDdossier IN (' + @PIDDossierTab + ')'
 
	EXECUTE (@sql);

Quand j’exécute, j’obtiens une erreur :

Code :

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

1
2
3
4
Msg 4145, Level 15, State 1, Line 5
An expression of non-boolean type specified in a context where a condition is expected, near 'Adres'.
 
(1 row(s) affected)

Je me suis basé sur ce post pour le requête : http://forums.whirlpool.net.au/archive/928435



Maintenant, si je retire un des deux LEFT JOIN, j’obtiens bien mes résultats.
Une idée ?

D’avance, merci.

[Raphael1980]

Bon, je retourne dormir...

DECLARE @sql AS VARCHAR(255)

C'était simplement trop petit pour la taille de la query...

N'empêche, il n'y a pas une meilleur méthode ?

[Waldar]

Ce n'est pas faux mais elle a de gros défauts, dont le SQL Injection et la multiplication des plans d'exécutions.

Si vous lisez l'anglais, j'ai trouvé ce lien qui recense de nombreuses méthodes (lien que j'ai simplement survolé) :
http://www.sommarskog.se/arrays-in-sql-2005.html

[Lyche]

Tu peux faire un split des valeurs sur ton séparateur ","

ensuite insérer dans une table temporaire ou dans une table fixe puis faire un select exists des éléments de ta table.

[elsuket]

Bonjour,

Pour le dépouillement de la chaîne, vous pouvez voir le billet que j'ai publié à ce sujet.

Privilégiez l'utilisation de sp_executesql, qui permet de paramétrer les requêtes et donc de réutiliser les plans de requêtes, donc :

- de diminuer le temps de requête en évitant la compilation
- de ne pas pourrir le cache de plans avec des plans qui ont une forte probabilité de ne jamais être réutilisés ...

J'ai donné des exemples d'utilisation de sp_executesql ici

@++

[Raphael1980]

Merci pour votre retour.

Je vais travailler avec sp_executesql.

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
ALTER PROCEDURE [dbo].[GestionRetourMaterielEtat]
	-- Add the parameters for the stored procedure here
	@PIDDossierTab varchar(300)
 
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;
 
	DECLARE @SQLString NVARCHAR(500)
	DECLARE @ParmDefinition NVARCHAR(500)
 
	DECLARE @IntVariable INT
	DECLARE @Lastlname varchar(50)
 
	SET @SQLString = N'SELECT @LastlnameOUT = saturna.dbo.dossier.Responsable
                   FROM saturna.dbo.dossier
                   WHERE saturna.dbo.dossier.IDdossier IN ('+@PIDDossierTab+')'
 
	SET @ParmDefinition = N'@LastlnameOUT varchar(50) OUTPUT'
 
	EXECUTE sp_executesql
		@SQLString,
		@ParmDefinition,
		@LastlnameOUT=@Lastlname OUTPUT
 
	SELECT @Lastlname
 
END

Par contre, je vais devoir stocker les résultats dans une table temporaire avant de faire un SELECT sur cette table.

Bon, je sais que ce n'est pas top...

http://blog.developpez.com/elsuket/p...e-performance/

Mais je ne vois pas comment faire autrement surtout avec mes connaissances.

J'ai lu le billet sur le dépouillement mais ça ne fait pas tilt.