Discussion :

[joanita_lo]

Bonjour, j'ai écris une procédure stockée pour remplir une grille en fonction du profil de l'utilisateur connecté. Le problème c'est que ça me renvoie une erreur, et je ne sais quoi faire.Aidez moi s'il vous plait. Voilà ma PS:

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
set ANSI_NULLS ON
set QUOTED_IDENTIFIER ON
go
 
ALTER PROCEDURE [dbo].[PS_REMPLIR_GRILLE_PRODEMANDE]
				 @Profil int
AS
BEGIN
 
	DECLARE @Script varchar(8000)
	SET @Script = 'SELECT DISTINCT DEMANDE_INTERVENTION.IdDemande_PK , DEMANDE_INTERVENTION.Objet , 
						DEMANDE_INTERVENTION.DteDemande , DEMANDE_INTERVENTION.DteDebut  , 
						DEMANDE_INTERVENTION.DteFin, DEMANDE_INTERVENTION.NomDemandeur, DEMANDE_INTERVENTION.PrenomDemandeur, 
						DEMANDE_INTERVENTION.IdClient_FK,DEMANDE_INTERVENTION.IdStatut_FK, DEMANDE_INTERVENTION.Observations  
					FROM  DEMANDE_INTERVENTION inner join STATUT inner join PROFIL_STATUT
					ON STATUT.IdStatut_PK = PROFIL_STATUT.IdStatut_FK
					ON DEMANDE_INTERVENTION.IdStatut_FK = PROFIL_STATUT.IdStatut_FK
					WHERE PROFIL_STATUT.IdProfil_FK = + @Profil'
	SET @Script = @Script
	EXEC(@Script)
END

Et voilà le résultat que j'obtiens :

Msg 137, Niveau 15, État 2, Ligne 8
La variable scalaire "@Profil" doit être déclarée.

(1 ligne(s) affectée(s))

[Rei Ichido]

Il faut ou bien ajouter la chaine directement - et donc ne pas laisser @Profil dans la chaine, mais la valeur utilisée - ce qui n'est pas conseillé si la requête est souvent exécutée :

Code :

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

1
2
3
4
5
6
7
8
9
 
	SET @Script = 'SELECT DISTINCT DEMANDE_INTERVENTION.IdDemande_PK , DEMANDE_INTERVENTION.Objet , 
						DEMANDE_INTERVENTION.DteDemande , DEMANDE_INTERVENTION.DteDebut  , 
						DEMANDE_INTERVENTION.DteFin, DEMANDE_INTERVENTION.NomDemandeur, DEMANDE_INTERVENTION.PrenomDemandeur, 
						DEMANDE_INTERVENTION.IdClient_FK,DEMANDE_INTERVENTION.IdStatut_FK, DEMANDE_INTERVENTION.Observations  
					FROM  DEMANDE_INTERVENTION inner join STATUT inner join PROFIL_STATUT
					ON STATUT.IdStatut_PK = PROFIL_STATUT.IdStatut_FK
					ON DEMANDE_INTERVENTION.IdStatut_FK = PROFIL_STATUT.IdStatut_FK
					WHERE PROFIL_STATUT.IdProfil_FK = '+ CONVERT(VARCHAR,@Profil)

Ou bien déclarer la variable lors de l'exécution.

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
 
 
DECLARE @ParmDefinition nvarchar(500)
DECLARE @Script varchar(8000)
 
SET @ParmDefinition = N'@ProfilUsed int'
 
 
	SET @Script = 'SELECT DISTINCT DEMANDE_INTERVENTION.IdDemande_PK , DEMANDE_INTERVENTION.Objet , 
						DEMANDE_INTERVENTION.DteDemande , DEMANDE_INTERVENTION.DteDebut  , 
						DEMANDE_INTERVENTION.DteFin, DEMANDE_INTERVENTION.NomDemandeur, DEMANDE_INTERVENTION.PrenomDemandeur, 
						DEMANDE_INTERVENTION.IdClient_FK,DEMANDE_INTERVENTION.IdStatut_FK, DEMANDE_INTERVENTION.Observations  
					FROM  DEMANDE_INTERVENTION inner join STATUT inner join PROFIL_STATUT
					ON STATUT.IdStatut_PK = PROFIL_STATUT.IdStatut_FK
					ON DEMANDE_INTERVENTION.IdStatut_FK = PROFIL_STATUT.IdStatut_FK
					WHERE PROFIL_STATUT.IdProfil_FK = @ProfilUsed'
 
EXECUTE sp_executesql @Script , @ParmDefinition,
                      @ProfilUsed = @Profil;

[zinzineti]

Code :

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

1
2
 
WHERE PROFIL_STATUT.IdProfil_FK = '+ CONVERT(VARCHAR,@Profil)

/!\ Attention, lorsque la longueur du VARCHAR n'est pas spécifiée dans le CONVERT (ou CAST) la longueur par défaut est 30. Donc tenir compte de la longueur maximale de @Profile.
Je te propose dans ta procédure de :
0. d'abord tester si le profile en entrée est un entier (integer); si oui alors
1. construire ta requête puis exécuter
Il faut aussi prendre l'habitude d'imposer l'option

Code :

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

SET NOCOUNT ON

au début des procédures stockées.

[Kropernic]

Il faut aussi prendre l'habitude d'imposer l'option

Code :

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

SET NOCOUNT ON

au début des procédures stockées.

Cela apporte... ? (un gain en performance ?)

[zinzineti]

Cela apporte... ? (un gain en performance ?)

L'option

Code :

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

SET NOCOUNT ON

évite le renvoi du message indiquant le nombre de lignes retournées par la requête. Donc diminue le niveau de bavardage côté SGBD. On économise ainsi des ressources et donc gain de perf

[Kropernic]

Sur la théorie, 100% d'accord. Mais en pratique, est-ce vraiment significatif ? Dans mon environnement de travail, je pense que je ne verrai jamais la différence alors je vous demande...