Discussion :

[Heisenberg5625]

Bonjour à tous !

Voici mon problème : Je souhaite mettre à jour une table A en fonction d'une table B uniquement si les données contenu dans A diffères de celle de B.
J'ai lu qu'il n'était pas conseillé de mettre à jour une table à l'aide d'une procédure stockée mais étant néophyte en l matière je n'ai pas trouvé d'autre solution...
Voici mon 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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 
 
ALTER PROCEDURE [dbo].[PS_RESTORE_PARAM_PROD] 
	-- Add the parameters for the stored procedure here
	@IdBase bigint 
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;
	DECLARE @Paragraphe varchar(50), @Variable varchar(50), @Valeur varchar(254), @Memo varchar(MAX), @Row uniqueidentifier, @IdBase_Connexion bigint
 
	DECLARE curseur_save CURSOR FOR
	SELECT Paragraphe, Variable,Valeur, Memo, Rowguid, Id_Base from dbo.Connexion_Serveur where Id_Base = @IdBase
 
	OPEN curseur_save
 
	FETCH curseur_save into @Paragraphe,@Variable, @Valeur, @Memo, @Row, @IdBase_Connexion
 
	WHILE @@FETCH_STATUS = 0
	BEGIN
 
		DECLARE @ParagraphePROD varchar(50), @VariablePROD varchar(50), @ValeurPROD varchar(254), @MemoPROD varchar(MAX), @RowPROD uniqueidentifier
		DECLARE curseur_PROD CURSOR FOR
		SELECT Paragraphe, Variable,Valeur, Memo, Rowguid from [ATH].[dbo].[CONNEXION_SERVEUR]
 
		OPEN curseur_PROD
 
		FETCH curseur_PROD into @ParagraphePROD,@VariablePROD, @ValeurPROD, @MemoPROD, @RowPROD
 
		WHILE @@FETCH_STATUS = 0
		BEGIN
 
			IF @Row = @RowPROD 
			BEGIN
				IF @Paragraphe != @ParagraphePROD
				SET @ParagraphePROD = @Paragraphe
 
				CLOSE curseur_PROD
				DEALLOCATE curseur_PROD
			END
 
		END
 
	END
 
END
CLOSE curseur_save
DEALLOCATE curseur_save

Je m'excuse d'ors et déjà auprès de ceux qui vont lire mon code car ça risque de leur faire mal aux yeux
Et si vous souhaitez plus de précision n'hésitez pas à me faire signe

Cordialement,

[al1_24]

Je pense que la commande MERGE devrait répondre de manière beaucoup plus élégante et efficace à ton besoin.

[Heisenberg5625]

En effet ça me semble bien mieux au première abord.
Je vais essayer ça, merci bien.

[aieeeuuuuu]

Bonjour,

Juste une précision sur ce point :

J'ai lu qu'il n'était pas conseillé de mettre à jour une table à l'aide d'une procédure stockée

Ce n'est pas le fait d'utiliser une procédure stockée, ce qui est au contraire plutôt une bonne pratique.
Ce qui est déconseillé, c'est d'utiliser des curseurs comme vous le faites. Ce ceux eux qui nuisent aux performances dans votre cas.

[Heisenberg5625]

Merci pour cette précision aieuuuu.

J'ai essayé le MERGE mais j'ai une SQLException qui est levée du côté de mon application : "Une exception non gérée du type 'System.Data.SqlClient.SqlException' s'est produite dans System.Data.dll

Informations supplémentaires : La cible d'une instruction MERGE ne peut pas être une table distante, une vue distante ou une vue sur des tables distantes."

Voici le MERGE que j'ai fait :

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
 
USE [Migration_Donnees]
GO
/****** Object:  StoredProcedure [dbo].[PS_RESTORE_PARAM_PROD]    Script Date: 20/06/2016 16:40:01 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
-- =============================================
-- Author:		<Author,,Name>
-- Create date: <Create Date,,>
-- Description:	<Description,,>
-- =============================================
ALTER PROCEDURE [dbo].[PS_RESTORE_PARAM_PROD] 
	-- Add the parameters for the stored procedure here
	@IdBase bigint
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;
 
	CREATE TABLE #Con ([Paragraphe] varchar(50), [Variable] varchar(50), [Valeur] varchar(254), [Memo] varchar(MAX), [Rowguid] uniqueidentifier, [Id_Base] bigint)
	BEGIN
		INSERT INTO #Con
		([Paragraphe], [Variable], [Valeur], [Memo], [Rowguid], [Id_Base])
		SELECT
		[Paragraphe], [Variable], [Valeur], [Memo], [Rowguid], [Id_Base]
		FROM
		[dbo].[Connexion_Serveur]
		WHERE Id_Base = @IdBase
	END
	BEGIN
 
		MERGE [ATH].[dbo].[CONNEXION_SERVEUR] as prod
		USING #Con
		ON (prod.rowguid = #Con.Rowguid)
		WHEN MATCHED THEN
		UPDATE SET prod.Paragraphe = #Con.paragraphe;
 
	END
END

[Lyche]

Bonjour,

Il me semble que le using ne s'utilise qu'avec une requête.

Code sql :

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

1
2
3
4
5
6
7
 
MERGE [ATH].[dbo].[CONNEXION_SERVEUR] as T
USING ( SELECT *
          FROM #Con ) AS S
  ON T.rowguid = S.Rowguid
WHEN MATCHED THEN
UPDATE SET Paragraphe = S.paragraphe;

Par contre je ne comprends pas. Dans la première version il y a 2 curseurs qui s'imbriquent, comment 1 seul merge? (vu le code, un update devrait suffir même)

Sinon, il est possible de se passer de la #Con en utilisant la requête qui la constitue dans la source du merge.

Code sql :

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

1
2
3
4
5
6
7
8
9
10
 
MERGE [ATH].[dbo].[CONNEXION_SERVEUR] as T
USING ( SELECT [Paragraphe], [Variable],
               [Valeur]    , [Memo]    ,
               [Rowguid]   , [Id_Base]
          FROM [dbo].[Connexion_Serveur]
         WHERE Id_Base = @IdBase) AS S
  ON T.rowguid = S.Rowguid
WHEN MATCHED THEN
UPDATE SET Paragraphe = S.paragraphe;

Cordialement,