Discussion :

[Lo²]

Bonjour à tous,

Je rencontre un petit problème sur la déclaration d'un curseur dans une procédure stockée.
Voici le code (allégé) de la procédure stockée :

Code :

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

1
2
3
4
5
6
7
8
9
CREATE PROCEDURE P0600 
AS
BEGIN
DECLARE  @I1 VARCHAR(5000)
DECLARE C1 CURSOR FOR @I1
SET @I1 = 'SELECT * FROM F0600'
OPEN C1
FETCH NEXT FROM C1
END

J'ai une erreur de syntaxe sur le curseur, au niveau de @I1.

Pour être plus clair, la requête du curseur sera créée dynamiquement selon les paramètres reçues par la procédure.

Je suppose que l'erreur provient du fait que je déclaire le curseur sur une variable, mais je n'arrive pas à trouver la bonne syntaxe, ou méthode d'utilisation.

Merci pour votre aide.

[mikedavem]

Bonjour,

Voici une syntaxe 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
CREATE PROCEDURE P0600 
AS
BEGIN
 DECLARE C1 CURSOR FOR 
 SELECT <nomcolonne> FROM F0600
 
 OPEN C1
 FETCH NEXT FROM C1 INTO <unevariable>
 WHILE @@FETCH_STATUS = 0
 BEGIN
   ...
 
   FETCH NEXT FROM C1 INTO <unevariable>
 END
 CLOSE C1
 DEALLOCATE C1
END

Cependant dans la plupart des cas il est tout à fait possible de remplacer votre curseur par un code ensembliste. Avec un curseur les performances de votre procédure risquent de ne pas être au rendez-vous (Voir ce billet)

++

[Lo²]

Bonjour et merci pour votre réponse rapide.

J'avais lu cet article et un autre, concernant cette "mauvaise" habitude sur l'utilisation du curseur.

Les procédures que je créée me permettent de construire dynamiquement une requête, le "SELECT * FROM F0600" étant la base de départ de la requête.
Suivant les paramètres envoyés à la procédure, j'ajoute via des CASE... IF... des jointures, des restrictions.
Finalement, je ne vois pas comment faire autrement qu'utiliser un curseur pour me retourner le résultat, je me trompe ?

Pour information, l'utilisation des procédures stockées est une contrainte sur le développement en cours.

[mikedavem]

L'utilisation des procédures stockée est une bonne chose.

Pour répondre à votre question concernant les curseurs, il faudrait avoir plus de précisions concernant votre problème. Que voulez vous faire exactement ?

++

[Lo²]

Je vois que je ne suis pas assez claire :
Voici une partie du code moins allégé de ma procédure :

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
CREATE PROCEDURE P0600 
	@V0 SMALLINT
	, @V1 CHAR(5)
	, @V2 CHAR(5)
	, @V3 CHAR(8)
	, @V4 CHAR(8)
	, @V5 INTEGER
	, @V6 VARCHAR(30)
	, @V7 CHAR(2)
	, @V8 CHAR(2) 
AS
BEGIN
 DECLARE  @I1 VARCHAR(5000)
 DECLARE C1 CURSOR @I1
SET @I1 = 'SELECT * FROM F0600'
IF (@V0 = 1)
 BEGIN
 SET @I1 = @I1 + ' INNER JOIN <table> ON ... '
 END
ELSE IF @V0 = 2
 BEGIN
 SET @I1 = @I1 + ' WHERE ...'
 IF @V2 <> '00000' 
  BEGIN
  SET @I1 = @I1 + ' ORDER BY...'
  END
 END
OPEN C1
FETCH NEXT FROM C1
END

Au final, je peux me retrouver avec l'exécution de "SELECT * FROM F0600" ou de "SELECT * FROM F0600 INNER JOIN <table> ON...".

La requête à exécuter par le curseur peut être différente d'un paramètre à un autre, et donc voilà pourquoi je voudrais déclarer un curseur sur une variable de type chaine.

[SQLpro]

Dans ce cas vous devez créer un curseur dynamique. Jettez un coup d'oil à sp_describe_cursor

A +

[Lo²]

Merci, je vais y jeter un oeil.
Autrement, j'ai trouvé cela :
http://www.developpez.net/forums/d23...rsor-executer/

[Lo²]

Voici une solution au problème pour ceux que ça intéresserait :

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
CREATE PROCEDURE P0600 
	@V0 SMALLINT
	, @V1 CHAR(5)
	, @V2 CHAR(5)
	, @V3 CHAR(8)
	, @V4 CHAR(8)
	, @V5 INTEGER
	, @V6 VARCHAR(30)
	, @V7 CHAR(2)
	, @V8 CHAR(2) 
AS
BEGIN
 DECLARE  @I1 NVARCHAR(4000)
 DECLARE @cur_C1 CURSOR
SET @I1 = N'SET @cur_C1 = CURSOR STATIC FOR 
		SELECT * FROM F0600; 
		OPEN @cur_C1';
EXEC sp_executesql @I1
	, N'@cur_C1 cursor OUTPUT', 
	@cur_C1 OUTPUT
FETCH NEXT FROM @cur_C1
END

Attention, il faut penser à déclarer la variable contenant la requête à exécuter en NVARCHAR.