Discussion :

[Gregney]

Bonjour

En stage, on m'a demander de modifier certains PS qui pouvait être plus rapide, dans la liste y en a une qui me pose probleme

Cette Ps est actuellement ecrite comme tel

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
 
DECLARE @TableTemp as Table (CodeDocument int ) 
DECLARE @TableTempo as Table (Code int )         
 
         INSERT INTO @TableTemp
         SELECT Document
         FROM dbo.TABLE1  
         JOIN dbo.TABLE2      ON Numero= Nums
         WHERE Action = @Num
 
         INSERT INTO @TableTempo
         SELECT distinct Code
         FROM Table3   
         WHERE  CodeEnsemble = @Code
 
 -- Déclaration d'un curseur 
      DECLARE c_curseur CURSOR 
      SELECT CodeDocument FROM @TableTemp
      OPEN c_curseur
      FETCH NEXT FROM c_curseur INTO @TCodeDoc
 
      WHILE @@FETCH_STATUS = 0 BEGIN            
            INSERT INTO MaTableInsert (Colonne1, Colonne2, Colonne3, Colonne4) 
 
            SELECT Code, @CodeDoc, GETDATE(), @Pers from @TableTempo
            WHERE CodeEncadre not in (select Code FROM dbo.Table3 
                                                      JOIN dbo.Table  ON Num = Code
                                                      WHERE CodeEnsemble = @Code)  
 
         FETCH NEXT FROM c_curseur INTO @TCodeDoc
      END
      CLOSE c_curseur
      DEALLOCATE c_curseur

J'ai remplacé le nom de mes table par souci de confidentialité, mais l'idée est que j'ai 2 tables temporaires qui sont utilisés dans le curseur pour faire des insertions dans ma table. Je souhaiterais donc reussir a faire ces insertions sans passé par le curseur mais je ne sais pas comment réussir l'insertion sans parcourir une des deux tables ?

L'idée doit etre simple mais ne me vient pas comme ça, merci pour vos informations.

[aieeeuuuuu]

Bonjour,

il serait utiles de connaitre la structure des tables afin notamment de savoir d'où proviennent les colonnes.
La connaissance des éventuels index existant serait utile également, ainsi que des contraintes référentielles et d'unicité.

EDIT : une petite description de qu'est censée faire la procédure serait pas mal aussi : j'ai l'impression que vous avez commis quelques erreurs en anonymisant le noms des tables et des colonnes...

[StringBuilder]

+1 avec aieeuuuu : il y a des erreurs au niveau de ton anonymisation, et les colonnes n'utilisant pas la table d'origine en préfixe, on a du mal à comprendre l'agencement du bouzin.

Voici la partie que le comprends réécrite (je comprends même pas l'existence d'un curseur et de tables temporaires) :

Code sql :

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

1
2
3
4
5
6
7
8
 
INSERT INTO MaTableInsert (Colonne1, Colonne2, Colonne3, Colonne4) 
SELECT T3.Code, @CodeDoc, GETDATE(), @Pers
FROM Table3 T3
WHERE T3.CodeEnsemble = @Code
AND T3.Code NOT IN (SELECT Code FROM dbo.Table3 
                                                      JOIN dbo.TABLE  ON Num = Code
                                                      WHERE CodeEnsemble = @Code)

En revanche, après le "NOT IN", c'est copier/coller de ton code, donc pas corrigé. Tu as mal anonymisé, et ça fait n'importe quoi.
Accessoirement, la variable @TableTemp n'est jamais utilisée, je pense que c'est elle qui doit être dans le sous-select, mais sans les sous-titres, pas moyen de comprendre ce que tu cherches à faire...