Discussion :

[djobert]

Bonjour, je désir créer une procédure stockée, mais quelque chose accroche et ça ne fonctionne pas. Je vous fais la mise en situation, j'ai copié plus bas mon code, et j'ai aussi copié les 2 erreurs que SQL me sort.

Je tente de faire cela avec SQL Management Studio pour SQL Server 2005

J'ai 2 tables (qui sont en fait des vues)


-ADM.VPlans

Champs : NO_PLAN, titre

-ADM.VPlans_Secteurs

Champs : NO_PLAN, Secteur

« NO_PLAN » contient les mêmes numéros de plans que le champ « NO_PLAN » de ADM.VPlans, de sorte que pour chaque ligne de ADM.VPlans, je peux avoir dans ADM.VPlans_Secteurs n lignes, dépendamment si j'ai 0, 1 ou n « Secteur » différents pour un même NO_PLAN.


Mon but est d'avoir une table :

-ADM.VPlans_Complets

Champs : NO_PLAN, titre, Secteurs

« Secteurs » doit contenir la list des Secteurs, séparés par des virgules

Ex : (dsl si ce n'est pas tout à fait clair, mais la page d'édition du message ne prend pas en compte les Tabulations et les suites d'espaces... Vous avez le nom des tables, le nom des champs, et ensuite les données..)

Code :

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

1
2
3
4
5
6
7
8
9
10
V3R.ADM.VPlans--|--V3R.ADM.VPlans_Secteur--|--V3R.ADM.VPlans_Complets 
 
 
 NO_PLAN---titre--|--NO_PLAN---Secteur---|--NO_PLAN---titre---Secteurs 
 
1----------tata---|----1----------TR-----|-----1--------tata----TR,CAP 
2----------titi---|----1----------CAP----|-----2--------titi------CAP 
3----------toto---|----2----------CAP----|-----3--------toto-----TR,TRO 
------------------|----3----------TR 
------------------|----3----------TRO

Voilà procédure stockée :

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
CREATE PROCEDURE dbo.MergeMyTwoTables AS
 
DELETE * FROM V3R.ADM.VPlans_Complets
 
DECLARE @NO_PLAN int 
 
DECLARE @titre varchar(1000) 
 
 
DECLARE @Concatenated varchar(2000) 
 
 
DECLARE cursorSourceTable CURSORFORSelect NO_PLAN, titre FROM V3R.ADM.VPlans ORDERBY NO_PLAN ASC 
 
OPEN cursorSourceTable 
 
 
FETCH NEXT FROM cursorSourceTable INTO @NO_PLAN, @titre 
 
WHILE @@FETCH_STATUS <> - 1 BEGIN 
 
 
      --For this line, get all values from sub-lines and concatenate them 
 
      SELECT @Concatenated  =null 
 
      SELECT @Concatenated =coalesce(@Concatenated +', ','')+cast(Secteur asvarchar(40))
      FROM V3R.ADM.VPlans_Secteurs 
      WHERE NO_PLAN = @NO_PLAN 
 
 
 
      --Insert result in ADM.VPlans_Complets 
 
      INSERT INTO V3R.ADM.VPlans_Complets (NO_PLAN, titre, Secteurs)
      VALUES (@NO_PLAN, @titre, coalesce(@Concatenated,'') 
 
 
 
      FETCH NEXTFROM cursorSourceTable INTO @NO_PLAN, @titre 
 
END 
 
CLOSE cursorSourceTable 
 
DEALLOCATE cursorSourceTable 
 
GO

Et ensuite, je créerais une Tâche planifiée pour répéter la procédure plusieurs fois par jour avec la commande

Requête sql pour runner la stored proc:
Exec MergeMyTwoTables

Voilà les messages d'erreur que SQL me donne lorsque j'exécute ma procédure. Je ne comprend pas.


Msg 102, Niveau 15, État 1, Procédure MergeMyTwoTables, Ligne 3

Syntaxe incorrecte vers '*'.

Msg 156, Niveau 15, État 1, Procédure MergeMyTwoTables, Ligne 23

Syntaxe incorrecte vers le mot clé 'FETCH'.

[mikedavem]

Bonsoir,

Plusieur choses :

- DELETE * FROM ca ne marche pas --> DELETE FROM ma table est plus correct
- Dans la mesure du possible évitez les curseurs lorsque vous pouvez vous en passez.

Voici un code qui devrait marcher chez vous (A adapter avec les types de données de vos tables)

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
 
CREATE PROCEDURE dbo.MergeMyTwoTables
AS
 
DECLARE @titre VARCHAR(50)
DECLARE @secteurs VARCHAR(1000)
DECLARE @no_plan_maj INT
 
-- Suppression enreg 
DELETE FROM dbo.VPlans_Complets
 
-- Insertion des no de plan dans une table temporaire
SELECT no_plan, titre, 0 AS maj INTO #temp
FROM dbo.VPlans
 
-- Traitement du champ secteur + insertion dans VPlans_Complets
WHILE EXISTS(SELECT * FROM #temp WHERE maj = 0)
BEGIN
 SET @secteurs = NULL
 
 SELECT TOP 1 @no_plan_maj = no_plan, @titre = titre
 FROM #temp
 WHERE maj = 0
 ORDER BY no_plan
 
 SELECT @secteurs = COALESCE(@secteurs + ',','') + s.secteur
 FROM dbo.VPlans p
 INNER JOIN dbo.VPlans_Secteurs s
 ON p.no_plan = s.no_plan
 WHERE p.no_plan = @no_plan_maj
 
 INSERT INTO dbo.VPlans_Complets VALUES(@no_plan_maj,@titre,@secteurs)
 
 UPDATE #temp
 SET maj = 1
 WHERE no_plan = @no_plan_maj
 
END
 
-- Suppression table temporaire
DROP TABLE #temp

++

[SQLpro]

DELETE * n'existe pas. SELECT * oui. Mais pour DELETE, c'est DELATE MaTable...

Pour la 2e erreur, votre code étant trèsn mal écrit difficile de dire exactement d'ou elle vien :
1) on ne colle pas les mots clef, ni les parenthèses
2) on termine ses phrases SQL par un point virgule.

Voici votre proc corrigée :

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
CREATE PROCEDURE dbo.MergeMyTwoTables
AS
 
DELETE FROM V3R.ADM.VPlans_Complets;
 
DECLARE @NO_PLAN int, @titre varchar(1000) , @Concatenated varchar(2000) 
 
DECLARE cursorSourceTable CURSOR 
FOR Select NO_PLAN, titre 
    FROM   V3R.ADM.VPlans 
    ORDER  BY NO_PLAN ASC;
 
OPEN cursorSourceTable;
 
FETCH NEXT cursorSourceTable INTO @NO_PLAN, @titre; 
 
WHILE @@FETCH_STATUS <> - 1 
BEGIN 
 
      --For this line, get all values from sub-lines and concatenate them 
 
      SELECT @Concatenated = null;
 
      SELECT @Concatenated = coalesce(@Concatenated +', ','') + cast(Secteur asvarchar(40))
      FROM   V3R.ADM.VPlans_Secteurs 
      WHERE  NO_PLAN = @NO_PLAN;
 
 
      --Insert result in ADM.VPlans_Complets 
 
      INSERT INTO V3R.ADM.VPlans_Complets (NO_PLAN, titre, Secteurs);
      VALUES (@NO_PLAN, @titre,coalesce(@Concatenated,''); 
 
 
      FETCH NEXT cursorSourceTable INTO @NO_PLAN, @titre; 
 
END 
 
CLOSE cursorSourceTable; 
 
DEALLOCATE cursorSourceTable; 
 
GO

Si cela cloche encore, voyez la ligne d'avant.

Enfin pour ce qui est de faire cela sans curseur il suffit de la faire la concaténation dans une CTE et insérer le tout en une seule requête SQL.

Bien évidemment, comme vous n'avez pas respecté la charte de postage difficile de vous en dire plus !

A +