Discussion :

[KNITTEL]

Bonjour,

Je recherche une façon de paramétrer les curseurs au sein d'une procédure stockée.

Je souhaiterais créer des cursor dont l'instruction Select fait référence à
une table passée en paramètre de ma procédute stockée du style :

Create PROC MAPROC
(
@maTable as nvarchar(30)
)
as
DECLARE MONCURSEUR CURSOR FOR
SELECT CHAMP1, CHAMP2 FROM @maTable
...

Comment faire ? Bien-entendu ce que j'ai écris ci dessus ne fonctionne pas
et de surcroit l'utilisation de la commande Exec est à proscrire dans ce cas.

Merci d'avance pour vos réponses.

[Oluha]

je suis sûre de rien mais l'autre fois des collègues disaient que c'était pas possible de passer le nom d'une table en paramètre d'une procédure stockée

[SQLpro]

DECLARE @CURSEUR VARCHAR(8000)
SET @CURSEUR = 'CURSOR FOR SELECT TABLE_NAME FROM INORMATION_SCHEMA.TABLES'

OPEN @CURSEUR

FETCH @CURSEUR INTO ...

Bien enetendu la chaine de car. peut être construite dynamiquement.

Tout cela est bien entendu contre performant :
1) l'utilisation des curseurs est déconseillé si l'on peut s'en passer (et dans 95% des cas on peut s'en passer !) parce que contre performant
2) l'utilisation du SQL dynamique est d&éconseillé parce que contreperformant

Conclusion : l'utilisation d'un curseur construit avec du SQL dynamique est TRES contre performante...

Mais la question est : quel est ton but derrière ta demande ?

A +

[KNITTEL]

Bonjour,

Votre solution n'a pas l'air de fonctionner, il refuse obstinément de m'enregistrer ma procédure stockée en me donnant la réponse suivante :
La variable '@SQLSTATEMENT' n'est pas une variable curseur, mais elle se trouve à un endroit où un curseur est attendu. L'erreur se produit sur la commande OPEN.

ou @SQLSTATEMENT correspond à votre variable @CURSEUR.

Ci dessous votre exemple :

Code :

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

1
2
3
4
5
6
DECLARE @CURSEUR VARCHAR(8000) 
SET @CURSEUR = 'CURSOR FOR SELECT TABLE_NAME FROM INORMATION_SCHEMA.TABLES' 
 
OPEN @CURSEUR 
 
FETCH @CURSEUR INTO ...

Ci dessous le mien :

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
ALTER  PROCEDURE REGROUP
(
@TABLE_NAME AS VARCHAR(30)
)
as
DECLARE @MEMO_GSM AS NVARCHAR(15)
DECLARE @GSM AS NVARCHAR(15)
DECLARE @SQLSTATEMENT AS NVARCHAR(150)
declare @SQL_DELETE AS NVARCHAR(150)
 
SET @SQLSTATEMENT = 'CURSOR FOR SELECT GSM FROM ' + @TABLE_NAME + ' ORDER BY GSM, DATENAIS'
 
-- DECLARE STCURSOR CURSOR
-- FOR exec sp_executesql @SQLSTATEMENT
 
OPEN @SQLSTATEMENT
 
FETCH NEXT @SQLSTATEMENT
INTO @GSM
 
WHILE @@FETCH_STATUS = 0
BEGIN
....

En ce qui concerne l'aspect performance, il est vrai que l'utilisation des curseurs n'est pas recommandée, c'est quelque chose dont je suis en parfaite connaissance.
En revanche il n'y a pas lieu de s'inquiéter de la performance de ce traitement (je l'estime en tout cas) et je n'ai réellement (pour l'instant) pas trouver de solution de remplacement par l'utilisation de requêtes, il n'est pas impossible que je sois dans mon cas dans les 5% restants.

Il est à savoir que ce traitement porte sur les enregistrements d'une seule table il n'y a aucune jointure dans mes requêtes et il s'agit dans mon traitement de conserver ma TABLE tout en y supprimant des enregistrements dont je n'ai pas besoin.

En réalité j'importe un fichier texte dans une table du même nom (située dans une basee réservée à ce genre de traitement) et je dois traiter les données de ce fichier en vue de resortir une version plus condensée.
Je réalisait le traitement jusqu'ici dans une base Access à partir de code Vb et je me suis dit qu'il serait plus sympa d'utiliser la puissance de SQL Server pour réaliser ce traitement et donc le confier à une procédure stockée plutôt que de le laisser dans le code ou on multiplie les accès à la base pour rien.

Merci déjà de votre réponse même si elle n'a pas encore apporté ses fruits.

Cordialement,

Alexandre[/b]

[Fabien Celaia]

A essayer (mais je ne suis plus sur que le sql dynamique ne soit pas géré dans une sous-session)

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 REGROUP
(
@TABLE_NAME AS VARCHAR(30)
)
as
DECLARE @MEMO_GSM AS NVARCHAR(15)
DECLARE @GSM AS NVARCHAR(15)
DECLARE @SQLSTATEMENT AS NVARCHAR(150)
declare @SQL_DELETE AS NVARCHAR(150)
 
SET @SQLSTATEMENT = 'DECLARE CURSOR MONCUR FOR SELECT GSM FROM ' + @TABLE_NAME + ' ORDER BY GSM, DATENAIS'
 
exec sp_executesql @SQLSTATEMENT
 
OPEN MONCUR
 
FETCH NEXT MONCUR INTO @GSM
 
WHILE @@FETCH_STATUS = 0
BEGIN
....

[KNITTEL]

Bravo fadace

Ta solution fonctionne et répond donc à mon problème.

Je t'en remercie.