Discussion :

[FreshVic]

Bonjour je connais pas grand chose en procedure stocké et rien du tout en SQL server .
Mais malgré tout je cherche a faire une procadure stocké qui doit remplir une table en deux requete la premiere fournit un premier jeu de données et la deuxieme est parcouru avec un CURSOR pour replacé les info sur les lignes deja existante enfin bon voila le code :

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
 
CREATE PROCEDURE .initExcel AS BEGIN
DECLARE 
	  @cur CURSOR   SELECT * FROM LiteralStrings WHERE Language=2;
	 @lty INTEGER ;
	@lc INTEGER;
	@la INTEGER;
	@literal  VARCHAR;
	INSERT INTO Temporaire4Excel (LiteralType,LiteralCode,Language,Literal) SELECT LiteralType,LiteralCode,Language,Literal From LiteralStrings WHERE Language=1;
	OPEN cur;
                FETCH NEXT FROM cur
                WHILE @@FETCH_STATUS = 0
                BEGIN
                   FETCH  cur INTO lty,lc,la,literal;
                   UPDATE temporaire4Excel set LiteralFR=@literal  WHERE  LiteralType=@lty AND LiteralCode=@lc;
 
               END
 
CLOSE cur;
DEALLOCATE cur;
 
END;
 
GO

Faut etre indulgeant je debute comme je l'ai deja dit et voila l'erreur que j'ai :
Syntaxe incorrect vers @lty; ligne4 (meme chose ligne 13)
et @literal doit etre declaré

d'avance merci

[FreshVic]

Bon j'ai resolu les probleme de syntaxe mais maintenant c'est a l'executiuon que j'ai des soucis .
Apparemment je sais pas bien initialiser et pârcourir un CURSOR vous pouviez m'aider avec un exemple simple de parcours de CURSOR en TransacSQL !!

[FreshVic]

TOut les exemples que j'ai trouvé sont du style :

Code :

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

1
2
 
DECLARE  @cur CURSOR FOR  SELECT * FROM LiteralStrings WHERE Language=2;

Mais dans mon cas (SQLServer 2000) cette syntaxe est incorrect !!

Alors que sans le For la syntaxe est bonne mais a l'executiuon il me dit :

La variable '@cur' n'a présentement pas de curseur qui lui soit alloué.

je comprend plus rien !!

[WOLO Laurent]

Je l'ai amandé un peu pour avoir ceci:

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
CREATE PROCEDURE initExcel 
AS BEGIN 
DECLARE 
   @cur CURSOR   SELECT * FROM LiteralStrings WHERE Language=2, 
   @lty INT, 
   @lc INT, 
   @la INT, 
   @literal  VARCHAR(8000)
 
INSERT INTO Temporaire4Excel(LiteralType,LiteralCode,Language,Literal) SELECT LiteralType,LiteralCode,Language,Literal 
From LiteralStrings 
WHERE Language=1
 
OPEN cur 
FETCH  cur INTO @lty,@lc,@la,@literal
WHILE @@FETCH_STATUS = 0 
BEGIN  
     UPDATE temporaire4Excel 
     SET LiteralFR=@literal  
     WHERE  LiteralType=@lty 
     AND LiteralCode=@lc
 
     FETCH  cur INTO @lty,@lc,@la,@literal
END 
 
CLOSE cur
DEALLOCATE cur 
END
GO

Veille à cette présentation, la lisibilité en découle et plus on te répondra à temps.

[WOLO Laurent]

Bon j'ai resolu les probleme de syntaxe mais maintenant c'est a l'executiuon que j'ai des soucis .
Apparemment je sais pas bien initialiser et pârcourir un CURSOR vous pouviez m'aider avec un exemple simple de parcours de CURSOR en TransacSQL !!

Bon fondamentalement nous avons ce bout de code :

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
-- déclaration des variables de colonnes pour le curseur 
DECLARE @Col1 Type1, @Col2 Type2, @Col3, Type3... 
-- declaration du curseur 
DECLARE MyCursor CURSOR  
FOR  
   SELECT COL1, COL2, COL3 … 
   FROM MyTable 
 
-- ouverture du curseur 
OPEN MyCursor 
 
-- lecture du premier enregistrement 
FETCH MyCursor INTO @Col1, @Col2, @Col3... 
 
-- boucle de traitement 
WHILE @@fetch_Status = 0 
BEGIN 
   traitement 
-- lecture de l'enregistrement suivant 
   FETCH MyCursor INTO @Col1, @Col2, @Col3... 
END 
 
-- fermeture du curseur 
CLOSE myCursor 
 
-- libération de la mémoire 
DEALLOCATE myCursor

Que vous trouverez sur cette page
Bon bonne chance

[MAMMAR]

Bonjour,

je te propose une méthode plus simple: au lieu du curseur:

Code :

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

1
2
3
4
5
6
 
UPDATE temp
 set temp.LiteralFR=lit."champs de LiteralStrings  correpondant à @literal"
from temporaire4Excel temp INNER JOIN LiteralStrings lit
ON temp.LiteralType=lit."champs de LiteralStrings  correpondant à @lty" AND temp.LiteralCode=lit."champs de LiteralStrings  correpondant à @lc " 
WHERE  lit.Language=2

A+

[FreshVic]

Voila ce que j'ai maintenant :

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
 
CREATE PROCEDURE initExcel 
AS BEGIN 
   DECLARE  @cur CURSOR   SELECT * FROM LiteralStrings WHERE Language=2;
   DECLARE  @lty INTEGER; 
   DECLARE   @lc INTEGER;
   DECLARE   @la INTEGER;
   DECLARE   @literal  VARCHAR(255) ;
 
   DELETE FROM Temporaire4Excel;
 
INSERT INTO Temporaire4Excel(LiteralType,LiteralCode,Language,Literal) SELECT LiteralType,LiteralCode,Language,Literal 
From LiteralStrings 
WHERE Language=1 
 
   OPEN @cur 
   FETCH  @cur INTO @lty,@lc,@la,@literal 
  WHILE @@FETCH_STATUS = 0 
  BEGIN  
     UPDATE temporaire4Excel 
     SET LiteralFR=@literal  
     WHERE  LiteralType=@lty 
     AND LiteralCode=@lc 
 
     FETCH  @cur INTO @lty,@lc,@la,@literal 
  END 
 
  CLOSE @cur 
  DEALLOCATE @cur 
END
GO

Mais mon probleme est toujours le meme :

La variable '@cur' n'a présentement pas de curseur qui lui soit alloué.

je comprend pas d'autant plus que si je declare mon CURSOR comme dans ton exemple:

DECLARE MyCursor CURSOR
FOR
SELECT COL1, COL2, COL3 …
FROM MyTable

j'ai une erreur de syntaxe par contre sans le 'FOR ' cela passe tres bien !!!

[WOLO Laurent]

Très très bien MAMMAR.
En fait ta méthode non pas plus simple mais plus performante (Rapidité, on évite les boucles ).
Je n'ai pas proposé à titre pédagogique.
En effet, il veut comprendre les procedures stockées et les curseurs !!!
Et il a aussi dit qu'il vient de débuter.

[MAMMAR]

Bonjour,

En effet, il veut comprendre les procedures stockées et les curseurs !!!
Et il a aussi dit qu'il vient de débuter.

J'ai compris Laurent ton but.

pour le nom du curseur, je pense qu'il se declare sans le @:
cur au lieu de @cur

A+

[WOLO Laurent]

Voila ce que j'ai maintenant :

[code]
CREATE PROCEDURE initExcel
AS BEGIN
DECLARE @cur CURSOR SELECT * FROM LiteralStrings WHERE Language=2;

j'ai une erreur de syntaxe par contre sans le 'FOR ' cela passe tres bien !!!

Enlève @ devant le Cur car un curseur est une variable particulière

[code]
CREATE PROCEDURE initExcel
AS BEGIN
DECLARE cur CURSOR FOR SELECT * FROM LiteralStrings WHERE Language=2;
[/quote]

[WOLO Laurent]

Voila ce que j'ai maintenant :

Code :

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

1
2
3
4
 
CREATE PROCEDURE initExcel 
AS BEGIN 
   DECLARE  @cur CURSOR   SELECT * FROM LiteralStrings WHERE Language=2;

j'ai une erreur de syntaxe par contre sans le 'FOR ' cela passe tres bien !!!

Enlève @ devant le Cur car un curseur est une variable particulière

Code :

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

1
2
3
4
 
CREATE PROCEDURE initExcel 
AS BEGIN 
   DECLARE  cur CURSOR FOR SELECT * FROM LiteralStrings WHERE Language=2;

[FreshVic]

Merci !
C'est un Merci tardif je vous l'accorde mais MERCI qd meme !!

[WOLO Laurent]

Pas de problème mon cher ami.