2 INSERT dans un WHILE

Version imprimable

Bonjour :D,

Je souhaiterais savoir s'il est conseillé de faire 2 INSERT dans 2 tables différentes dans une boucle WHILE.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
	SET @Idx = 1
 
	WHILE (@Idx <= @Nombre_Copies)
	BEGIN
 
		INSERT INTO SOTRIGGERTAGHD(
		TGTSYSTEMCOD, TGTDOCTYP, TGTPARAMVALUE, TGTCOPIESNB, TGTPRINTER, 
		TGTSHSTATUS, TGTUPDATEDDT, TGTPROG, TGTUSERCOD)
		VALUES(
		@Prefixe_Id_Etiquette + CAST(@No_Etiquette AS VARCHAR), @Type_Document, @Parametres, @Nombre_Copies, @Identifiant_Imprimante, 
		@Edition_Statut_Attente, GETDATE(), @Dialogue, @Operateur)
 
		INSERT INTO SOTRIGGERDIALOGHD(TGDIPADDR, TGDMESSAGE, TGDSHSTATUS)
		VALUES('127.0.0.1', @Prefixe_Id_Etiquette + CAST(@No_Etiquette AS VARCHAR), @Edition_Statut_Attente)
 
		SET @Idx = @Idx + 1
	END

Que me conseillez-vous ?

:ccool:

16/06/2011, 10h52
aieeeuuuuu

Bonjour,

Il est plutôt conseillé d'éviter ce genre de boucles.

Vous n'avez pas mis le code entier, mais visiblement le but est d’insérer plusieurs fois (un nombre prédéfini) la même ligne...

préférez insérer 1 fois N lignes que N fois 1 ligne...
16/06/2011, 10h59
Sergejack
Citation:

Envoyé par aieeeuuuuu

Bonjour,

Il est plutôt conseillé d'éviter ce genre de boucles.

Vous n'avez pas mis le code entier, mais visiblement le but est d’insérer plusieurs fois (un nombre prédéfini) la même ligne...

préférez insérer 1 fois N lignes que N fois 1 ligne...

En complément d'information, vous devriez employer une table de nombres (il existe plusieurs façon d'en faire, cherchez, vous trouverez).

Votre query deviendrait, grosso-modo :
Code:

1 2 3 4 5 6 -- Rajout de 1000 lignes INSERT maTable ( colonne1, ...) SELECT valeur1, ... FROM TableDeNombre WHERE TableDeNombre.numero BETWEEN 1 AND 1000
16/06/2011, 11h18
WOLO Laurent
Une solution proposée par l'actuel responsable de la rubrique SQL Serveur que j'ai aimé.
Code:

1 2 3 4 SELECT DISTINCT Number FROM dbo.spt_values where Number between 0 and 1000 ORDER BY 1

oui,

voire aussi utiliser n'importe quelle table dont le nombre de lignes sera toujours plus grand que @Nombre_Copies (sinon, faire un produit cartésien) :

Sur le principe :

Code:

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
 
INSERT INTO SOTRIGGERTAGHD(
	TGTSYSTEMCOD, 
	TGTDOCTYP, 
	TGTPARAMVALUE, 
	TGTCOPIESNB, 
	TGTPRINTER, 
	TGTSHSTATUS, 
	TGTUPDATEDDT, 
	TGTPROG, 
	TGTUSERCOD
)
SELECT TOP(@Nombre_Copies)
	@Prefixe_Id_Etiquette + CAST(@No_Etiquette AS VARCHAR), 
	@Type_Document, 
	@Parametres, 
	@Nombre_Copies, 
	@Identifiant_Imprimante, 
	@Edition_Statut_Attente, 
	GETDATE(), 
	@Dialogue, 
	@Operateur
)
FROM sys.all_columns
CROSS JOIN sys.all_columns

Citation:

Envoyé par aieeeuuuuu

Bonjour,

préférez insérer 1 fois N lignes que N fois 1 ligne...

Vous avez tout à fait raison.

Effectivement la proc stockée gère ligne par ligne.
Comment pourrais - je en un insert, sélectionner les bonnes lignes.

Ci-dessous le code en entier :

Code:

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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
ALTER PROCEDURE [dbo].[G_Edition_Etiquette]
	@Type_Document			VARCHAR(10),
	@Parametres				VARCHAR(255),
	@Nombre_Copies			INT,
	@Identifiant_Imprimante	VARCHAR(60),
	@Dialogue				VARCHAR(10),
	@Operateur				VARCHAR(12)
AS
 
-- SET @ParamEtiqTRSP = 'TRSC' + '¤' + E0045051
 
--exec G_Edition_Etiquette 'TRSP',@ParamEtiqTRSP , 1, @ImpETQ, @Dialogue, @Operateur
--exec G_Edition_Etiquette 'TRSP',TRSC¤E0045051 , 1, 8, ESCP, 49808
 
--Constantes
DECLARE @Prefixe_Id_Etiquette					VARCHAR(50)	
 
--Variables
DECLARE @No_Etiquette							INT
DECLARE @TypeEtiquette							VARCHAR(5)
DECLARE @Edition_Statut_Attente					VARCHAR(3)
DECLARE	@Idx									INT
 
BEGIN
	--Initialisation des constantes
	SET @Prefixe_Id_Etiquette = 'ETIQ_'
 
	-- Lancement de la demande d'édition 
	EXEC G_Format_Compteur 'ETIQUETTE', 'P', '0', 0, @Dialogue, @Operateur, @No_Etiquette OUTPUT
 
	-- Sélection du N° d'étiquette
 
	select @No_Etiquette
 
 
	SET @Edition_Statut_Attente = '0'
 
	IF (@Type_Document = 'TRSP') AND (@Dialogue <> 'REED')
	BEGIN
		--Récupération du type de transporteur : GLS par rapport au numéro de colis 'E0045051'
		SELECT @TypeEtiquette = CNHPROP004 FROM SKCONTAINERHD WHERE CNHCONT = dbo.G_Decoupe_Chaine(@Parametres, 2, '¤')
		-- @TypeEtiquette = GLS
 
		-- Si transporteur = GLS 
		IF (@TypeEtiquette = 'GLS')
			SET @Edition_Statut_Attente = '5'
	END
 
	SET @Idx = 1
 
	WHILE (@Idx <= @Nombre_Copies)
	BEGIN
 
		INSERT INTO SOTRIGGERTAGHD(
		TGTSYSTEMCOD, TGTDOCTYP, TGTPARAMVALUE, TGTCOPIESNB, TGTPRINTER, 
		TGTSHSTATUS, TGTUPDATEDDT, TGTPROG, TGTUSERCOD)
		VALUES(
		@Prefixe_Id_Etiquette + CAST(@No_Etiquette AS VARCHAR), @Type_Document, @Parametres, @Nombre_Copies, @Identifiant_Imprimante, 
		@Edition_Statut_Attente, GETDATE(), @Dialogue, @Operateur)
 
		-- Lancement du dialogue d'édition par le SCAP COMMPC
		INSERT INTO SOTRIGGERDIALOGHD(TGDIPADDR, TGDMESSAGE, TGDSHSTATUS)
		VALUES('127.0.0.1', @Prefixe_Id_Etiquette + CAST(@No_Etiquette AS VARCHAR), @Edition_Statut_Attente)
 
		SET @Idx = @Idx + 1
	END		
 
END

16/06/2011, 12h34
WOLO Laurent

Si vous êtes sous SQL Serveur 2000 et 2005, vous pouvez passer en paramètre un tableau de paramètre via XML tandis qu'une méthode plus élégante existe sous SQL Serveur 2008, les TVP (Table Valued Parameters).
C'est à vous de voir.
16/06/2011, 13h40
Blacknounou

Citation:

Envoyé par aieeeuuuuu

Vous n'avez pas mis le code entier, mais visiblement le but est d’insérer plusieurs fois (un nombre prédéfini) la même ligne...

Le but est d'inséré des plusieurs lignes dans 2 tables différentes. :zoubi:

Exemple :

Ligne 1 dans Table 1 et Table 2
Ligne 2 dans Table 1 et Table 2
Ligne 3 dans Table 1 et Table 2
16/06/2011, 14h05
aieeeuuuuu

Insérer dans deux tables différentes = deux ordres INSERT (y a-t-il des contraintes sur vos tables, sinon vous pouvez jouer sur la clause OUTPUT, mais il me semble préférable de faire 2 insert)

par contre, vous pouvez insérer toutes vos lignes avec un seul ordre SQL. à partir de votre exemple, ce serait donc :

Insertion Ligne 1, ligne 2 et ligne 3 dans table 1
Insertion Ligne 1, ligne 2 et ligne 3 dans table 2

donc sans boucle.
16/06/2011, 14h52
Blacknounou

Citation:

Envoyé par aieeeuuuuu

par contre, vous pouvez insérer toutes vos lignes avec un seul ordre SQL. à partir de votre exemple, ce serait donc :

Insertion Ligne 1, ligne 2 et ligne 3 dans table 1
Insertion Ligne 1, ligne 2 et ligne 3 dans table 2

donc sans boucle.

C'est ce que j'ai fait et ça fonctionne !!!! :ccool: :D :ccool:

Merci encore pour vos réponse. :ccool::ccool:
16/06/2011, 15h00
WOLO Laurent

Effectivement ca ne change rien au principe de banir du procédure au profit de l'ensembliste.