Discussion :

[idream]

bonjour a tous
j ai un petit probleme de syntaxe avec une procedure stockee qui m empoisonne un projet
j aimerai que quelqun vienne a mon aide
merci d avance
Msg d erreur dans sql management studio 2005 :

Msg 102 Level 15 ,state 1 procedure cataloggetproductsonfrontpromo,line 20
incorrect syntax near ')'

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
31
32
33
34
35
36
37
38
39
40
procedure stockee
 
 
 
CREATE PROCEDURE CatalogGetProductsOnDeptPromo
(@DepartmentID INT,
@DescriptionLength INT,
@PageNumber INT,
@ProductsPerPage INT,
@HowManyProducts INT OUTPUT)
AS
-- declare a new TABLE variable
DECLARE @Products TABLE
(RowNumber INT,
ProductID INT,
Name NVARCHAR(50),
Description NVARCHAR(MAX),
Price MONEY,
Thumbnail NVARCHAR(50),
Image NVARCHAR(50),
PromoFront bit,
PromoDept bit)
-- populate the table variable with the complete list of products
INSERT INTO @Products
SELECT ROW NUMBER() OVER (ORDER BY ProductID) AS Row,
ProductID, Name, SUBSTRING(Description, 1, @DescriptionLength)
+ '...' AS Description,
Price, Thumbnail, Image, PromoFront, PromoDept
FROM
(SELECT DISTINCT Product.ProductID, Product.Name,
CASE WHEN LEN(Product.Description) <= @DescriptionLength
THEN Product.Description
ELSE SUBSTRING(Product.Description, 1, @DescriptionLength) + '...' END
AS Description, Price, Thumbnail, Image, PromoFront, PromoDept
FROM Product INNER JOIN ProductCategory
ON Product.ProductID = ProductCategory.ProductID
INNER JOIN Category
ON ProductCategory.CategoryID = Category.CategoryID
WHERE Product.PromoDept = 1
AND Category.DepartmentID = @DepartmentID

[elsuket]

Bonjour,

N'oubliez pas la balise code (bouton #) quand vous en publiez

Votre erreur de syntaxe provient du fait que vous avez oublié l'underscore entre ROW et NUMBER(), la fonction étant donc ROW_NUMBER().

Autres remarques :

- Quel est l'intérêt de placer les données que vous recherchez dans une variable de type table ?
Sachez que sur ce type de table, SQL Server ne maintient pas de statistiques, ce qui conduit à des plans de requête qui ne peuvent pas être optimaux puisque l'optimiseur se base sur ces statistiques pour établir un plan efficace.

- Vous n'avez pas qualifié le nom des tables qui participent à votre requête par le nom du schéma auquel elles appartiennent, qui est par défaut dbo (mais ce n'est pas forcément le cas )
Cela oblige SQL Server à interroger les tables de métadonnées pour connaître le propriétaire de la table et le comparer à celui qui exécute votre procédure stockée pour savoir s'il en a le droit d'exécution, ou autres privilèges.

- Vous n'avez pas placé l'option SET NOCOUNT à ON dans votre procédure stockée, ce qui retourne donc le nombre de lignes affectées par les requêtes de votre procédure stockée à l'appelant.
Cela n'est pas une grosse source de problèmes de performance, mais ça ne coûte rien de le mettre

Voici donc un exemple de ce que pourrait être votre procédure stockée, en tenant compte du fait que j'ai laissé la variable de type table car je ne sais pas ce que vous en faites par la suite ...
Je suis certain que vous pouvez vous en passer

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
CREATE PROCEDURE CatalogGetProductsOnDeptPromo
	@DepartmentID INT,
	@DescriptionLength INT,
	@PageNumber INT,
	@ProductsPerPage INT,
	@HowManyProducts INT OUTPUT
AS
BEGIN
	-- declare a new TABLE variable
	DECLARE @Products TABLE
	(
		RowNumber INT,
		ProductID INT,
		Name NVARCHAR(50),
		Description NVARCHAR(MAX),
		Price MONEY,
		Thumbnail NVARCHAR(50),
		Image NVARCHAR(50),
		PromoFront bit,
		PromoDept bit
	)
 
	-- populate the table variable with the complete list of products
	INSERT INTO @Products 
	SELECT ROW_NUMBER() OVER (ORDER BY ProductID) AS Row
		, ProductID
		, Name
		, SUBSTRING(Description, 1, @DescriptionLength) + '...' AS Description
		, Price
		, Thumbnail
		, Image
		, PromoFront
		, PromoDept
	FROM
	(
		SELECT DISTINCT P.ProductID
				, P.Name,
				CASE
					WHEN LEN(P.Description) <= @DescriptionLength THEN Product.Description
					ELSE SUBSTRING(P.Description, 1, @DescriptionLength) + '...'
				END AS Description
				, Price
				, Thumbnail
				, Image
				, PromoFront
				, PromoDept
		FROM		dbo.Product AS P
		INNER JOIN	dbo.ProductCategory AS PC
					ON P.ProductID = PC.ProductID
		INNER JOIN	dbo.Category AS C
					ON PC.CategoryID = C.CategoryID
		WHERE		P.PromoDept = 1
		AND		C.DepartmentID = @DepartmentID
	) AS TMP
END

@++

[WOLO Laurent]

Merci de ne pas oublier de marquer résolu le poste.