Insert into Select

Bonjour,

Je souhaite faire un INSERT INTO SELECT.
Particularité, je n'utilise pas les identity. Pour des faits historique j'ai un générateur unique.

J'ai donc cette procédure pour incrémenter et me retourner mon ID

Code:

---

1 2 3 4 5 6 7 8 9 10 11

create procedure dbo.getClePrimaire @iClePrim int OUTPUT as begin -- insert into bidon pour générer un identity insert into TABLE_CLE (CODE_CLE) values (1);   -- retour de l'identity en paramÞtre de sortie SELECT @iClePrim = @@IDENTITY; end;

---

Par contre je ne vois pas comment faire simplement un insert into select du genre

Code:

---

1 2 3

INSERT INTO A (CODE_A, NOM_A) SELECT "truc pour générer mon ID", NOM_B FROM B

---

Si quelqu'un a une idée je suis preneur :D

ça ne marche pas ça ?

Code:

---

NSERT INTO A (CODE_A, NOM_A) SELECT "truc pour générer mon ID", NOM_B FROM B

---

Quel est le message d'erreur ?

Vous ne pouvez pas faire un INSERT qui encapsule une autre commande. Ce n'est pas transactionnellement cohérent !

A +

Bonjour, je fais quelque chose comme ça pour faire l'équivalent de ce que tu veux faire:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

--définit la structure de la table qui stocke la liste des données create table #T_temp_code_erreur ( id_regle_erreur uniqueidentifier not null, code_erreur varchar(18) not null, description_erreur varchar(4000) not null, regle_erreur varchar(4000) )   --initialise la table #T_temp_code_erreur insert into #T_temp_code_erreur select [r].[id_configuratorEngine] as [id_regle_erreur], [r].[v_codeArticle] as [code_erreur], [r].[v_commentaire] as [description_erreur], '' as [regle_erreur] from [ServSql1].[ConfiguratorEngine].[dbo].[T_configuratorEngine] as [r] inner join [ServSql1].[ConfiguratorEngine].[dbo].[T_module] as [m] on [m].[id_module]=[r].[id_module] where [m].[v_module_display]=@v_module_display

---

Tu noteras que j'alias tous les champs de mon select avec les noms des champs cibles.

Par contre, je ne peux pas te garantir que ça puisse marcher avec une table non-temporaire. Le DBA de mon boulot veut pas me laisser en écriture.
(Entre nous il a raison d'être prudent).

Bonjour,
Nous avons la même problématique que "Joc02".
On doit avoir un ID unique pour chaque enregistrement quelque soit la table.
Nous débutons en MS-SQL.
Si quelqu'un a une solution pour obtenir un id unique et permettre de l'utiliser directement dans une requête d'insertion, nous sommes preneur. Cela fonctionne très bien avec firebird/interbase et les générateurs.

Bonjour,

Pourquoi ne pas implémenter une table de compteur de cas là ?

- Je récupère le dernier enregistrement + 1 = dernier ID
- Je mets à jour la table avec le dernier ID généré
- Je mets à jour le dernier enregistrement

etc ...

- Je récupère le dernier enregistrement + 1 = dernier ID
- Je mets à jour avec le dernier ID généré
- Je mets à jour le dernier enregistrement

Et ceci pour chaque transaction de mise à jour de vos tables.

++

C'est un peu la solution de Joc02 mais elle ne permet pas l'insert into select , ce qui nous bloque.
La réponse "Ce n'est pas transactionnellement cohérent !" sans + de détail m'interpelle également ...

Pourrait-on utiliser une fonction pour générer et récupérer un ID unique ?
Sans passer par une UDF (fonction externe développée hors SQL) puisque cela ne sera plus supporté par les prochaines versions de MS-SQL. Cela nous permettrait son utilisation dans un insert select. Mais étant débutant en MS-SQL, nous n'avons pas réussi !

Bonjour,

Citation:

---

La réponse "Ce n'est pas transactionnellement cohérent !" sans + de détail m'interpelle également ...

---

C'est parce que toute transaction est atomique (une des propriétés ACID, donc vous ne pouvez pas générer un identifiant dans la même instruction que celle qui spécifie un INSERT, UPDATE ou DELETE.

Citation:

---

Pourrait-on utiliser une fonction pour générer et récupérer un ID unique ?

---

Non, car il n'est pas possible d'effectuer des modifications de données à partir d'une UDF
Vous devez pour cela utiliser une procédure stockée qui vous retourne la valeur générée en OUPUT, de façon à pouvoir affecter une variable locale à toute procédure stockée appelante.

Citation:

---

Sans passer par une UDF (fonction externe développée hors SQL) puisque cela ne sera plus supporté par les prochaines versions de MS-SQL

---

UDF = User Defined Function, que vous pouvez toujours écrire en T-SQL, et qui ne sera pas supprimée dans les prochaines versions de SQL Server.
Vous confondez probablement avec les procédures stockées étendues (C et C++), qui ont été dépréciées avec SQL Server 2008, et qu'il faut donc réécrire en .NET pour profiter de l'intégration de CLR dans SQL Server (depuis SQL Server 2005) ;)

Je peux donc vous proposer ceci, mais sachez que vous vous éviteriez ce tarabiscotage si vous ajoutiez une colonne de type entier avec la propriété de compteur auto-incrémenté ...

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

CREATE TABLE dbo.TbCompteur ( compteur INT NOT NULL CONSTRAINT CHK_TbCompteur_compteur CHECK (compteur >= 0) ) GO   INSERT INTO dbo.TbCompteur VALUES (0) GO   --------------------------------------------------------- -- Incrémente et retourne une nouvelle valeur du compteur --------------------------------------------------------- CREATE PROCEDURE PsGetNouveauCompteur @nouvelleValeurCompteur INT = NULL OUTPUT AS BEGIN -- pour ne pas retourner le nombre de lignes affectées -- par les instructions à l'application appelante SET NOCOUNT ON   -- Incrémentation du compteur UPDATE dbo.TbCompteur SET compteur = compteur + 1   -- Affectation de la variable de sortie SELECT @nouvelleValeurCompteur = compteur FROM dbo.TbCompteur END

---

Ensuite vous pouvez appeler la procédure stockée PsGetNouveauCompteur dans tout autre module SQL (excepté les UDFs) comme suit :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

DECLARE @compteur INT EXEC dbo.PsGetNouveauCompteur @compteur OUTPUT   INSERT INTO dbo.maTable ( colonneCompteurDeMaTable, autresColonnesDeMaTable ) SELECT @compteur, mesColonnes ... FROM dbo.uneAutreTable JOIN ...

---

Une solution peut-être plus souple et plus propre serait probablement de passer tous les paramètres des lignes que vous devez insérer à une procédure stockée, et de générer la valeur de compteur dans celle-ci.
Supposons que nous devons insérer des lignes dans la table suivante :

Code:

---

1 2 3 4 5

CREATE TABLE maTable ( compteur INT NOT NULL CONSTRAINT PK_maTable_compteur PRIMARY KEY, maColonne VARCHAR(20) NOT NULL CONSTRAINT UQ_maTable_maColonne UNIQUE )

---

Nous conservons la table précédente :

Code:

---

1 2 3 4 5 6 7 8

CREATE TABLE dbo.TbCompteur ( compteur INT NOT NULL CONSTRAINT CHK_TbCompteur_compteur CHECK (compteur >= 0) ) GO   INSERT INTO dbo.TbCompteur VALUES (0) GO

---

Voici donc la procédure stockée :

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

CREATE PROCEDURE PsAjouteUneLigneA_maTable @maColonne VARCHAR(20) AS BEGIN -- pour ne pas retourner le nombre de lignes affectées -- par les instructions à l'application appelante SET NOCOUNT ON   -- Incrémentation du compteur UPDATE dbo.TbCompteur SET compteur = compteur + 1   -- Affectation de la variable de sortie DECLARE @nouvelleValeurCompteur INT SELECT @nouvelleValeurCompteur = compteur FROM dbo.TbCompteur   INSERT INTO dbo.maTable ( compteur, maColonne ) VALUES ( @nouvelleValeurCompteur, @maColonne ) END

---

Citation:

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11

CREATE procedure dbo.getClePrimaire @iClePrim int OUTPUT AS begin -- insert into bidon pour générer un identity INSERT INTO TABLE_CLE (CODE_CLE) VALUES (1);   -- retour de l'identity en paramÞtre de sortie SELECT @iClePrim = @@IDENTITY; end;

---

---

N'utilisez pas @@IDENTITY, car cela vous retourne la valeur de toute table de la base de données qui a été insérée et qui a une colonne avec la propriété d'auto-incrémentation : si donc j'insère une ligne dans une telle table, et que vous faites de même dans une autre table, et que mon INSERT se termine avant le votre, il est possible que vous récupériez la valeur de la colonne avec la propriété d'auto-incrémentation de la table que je viens d'insérer.
Préférez la fonction SCOPE_IDENTITY(), qui vous retournera la valeur de la colonne avec la propriété d'auto-incrémentation du module en cours d'exécution, et pas des autres ;)

@++ ;)

Ok, merci pour toutes ces informations.
On va sans doute mixer les 2 solutions : on mettra un auto incrément lorsqu'on pourra le faire et le restant avec une procédure pour récupérer un ID.