Discussion :

[Devlin111]

Bonjour
J'ai fais une procédure stockée pour insérer des données dans ma base, et j'ai une erreur que je ne comprend pas et n'arrive pas à résoudre.
Must declare the scalar variable "@id". 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
50
51
52
53
54
 
CREATE PROCEDURE [dbo].[Insert_Langue] 
	-- Add the parameters for the stored procedure here
        @id int = null,
	@lvl int = null,
	@name nvarchar (100)= null
 
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
 
	declare @la_id int;
	select   @la_id = la_id from LANGUE where la_name = @name;
 
	declare @Query nvarchar(MAX);
	declare @Query2 nvarchar(MAX);
	declare @Param nvarchar(MAX);
 
	BEGIN TRANSACTION 	
	SET NOCOUNT ON;
 
	set @Query ='';
	Set @Param = @Param  + ',@id int '	
	Set @Param = @Param  + ',@lvl int '
	Set @Param = @Param  + ',@name nvarchar(100) '
 
 
 
    -- Insert into DB
            set @Query = 'INSERT INTO [dbo].[USER_LANGUE] ';
			set @Query = + @Query + '([id] ' ;
			set @Query2 =  ' VALUES (@id';
 
			if (@la_id <> NULL ) BEGIN 
				set @Query = @Query + ',[la_id] ';
				set @Query2 =@Query2 + ', @la_id';
			END
 
			if (@la_id <> NULL ) BEGIN 
				set @Query = @Query + ',[lvl] ';
				set @Query2 =@Query2 + ', @lvl';
			END
			set @Query = + @Query + ') ' + @Query2+ ')'
 
			IF (@Query <> '' ) 
			begin
 
			EXECUTE sp_executesql  @Query, @Param, @id = @id 
                                                          @la_id = @la_id,
                                                          @lvl = @lvl
                        end
COMMIT TRANSACTION
END

Merci d'avance pour votre aide

Dev

[SQLDev]

Bonjour,

Dans un premier temps, il manque la déclaration de la création ou mise à jour de la procédure stockée : CREATE PROC dbo.nom_de_la_procédure
(Peut être un mauvais copier/coller)

Ensuite, ne pas utiliser IF (@la_id <> NULL ) mais @la_id IS NOT NULL

Pour le problème initial, c'est une virgule dans le code ci-dessous qui est manquante

EXECUTE sp_executesql @Query, @Param, @id = @id ,
@la_id = @la_id,
@lvl = @lvl

[Devlin111]

Salut,

Oui, c'était un mauvais copié collé pour le nom de la procédure. J'ai corrigé.

J'ai changé mes IF (@la_id <> NULL ) en @la_id IS NOT NULL

Mais pourquoi en fait? <> NULL et IS NOT NULL n'est ce pas la même chose?

Merci

EDIT : J'ai aussi corrigé la virgule, mais le problème persiste Toujours la même erreur

[SQLDev]

Mais pourquoi en fait? <> NULL et IS NOT NULL n'est ce pas la même chose?

Je vous propose de lire cet article complet qui vous donnera toutes les informations nécessaires et bien plus :
http://sqlpro.developpez.com/cours/null/

EDIT : J'ai aussi corrigé la virgule, mais le problème persiste Toujours la même erreur

Hum, bizarre quand je mets la virgule je n'ai plus l'erreur, voici le code complet :

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
50
51
52
53
 
CREATE PROCEDURE [dbo].[Insert_Langue] 
 
	@id int = NULL,
	@lvl int = NULL,
	@name nvarchar (100)= NULL
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
 
	declare @la_id int;
	SELECT   @la_id = la_id FROM LANGUE WHERE la_name = @name;
 
	declare @Query nvarchar(MAX);
	declare @Query2 nvarchar(MAX);
	declare @Param nvarchar(MAX);
 
	BEGIN TRANSACTION 	
	SET NOCOUNT ON;
 
	SET @Query ='';
	SET @Param = @Param  + ',@id int '	
	SET @Param = @Param  + ',@lvl int '
	SET @Param = @Param  + ',@name nvarchar(100) '
 
   -- Insert into DB
            SET @Query = 'INSERT INTO [dbo].[USER_LANGUE] ';
			SET @Query = + @Query + '([id] ' ;
			SET @Query2 =  ' VALUES (@id';
 
			IF (@la_id IS NOT NULL ) BEGIN 
				SET @Query = @Query + ',[la_id] ';
				SET @Query2 =@Query2 + ', @la_id';
			END
 
			IF (@la_id IS NOT NULL ) BEGIN 
				SET @Query = @Query + ',[lvl] ';
				SET @Query2 =@Query2 + ', @lvl';
			END
			SET @Query = + @Query + ') ' + @Query2+ ')'
 
			IF (@Query <> '' ) 
			begin
 
			EXECUTE sp_executesql  @Query, @Param
					, @id = @id,
					  @la_id = @la_id,
					  @lvl = @lvl
            end
COMMIT TRANSACTION
 
END

[Devlin111]

Merci pour l'article.
J'ai exactement le même code que le tien, et pourtant l'erreur persiste

[SQLDev]

Etrange, je viens de copier/coller le code que j'ai posté au dessus, j'ai exécuté à nouveau ce code et tout passe correctement.

Tu peux faire la même manipulation si ce n'est déjà fait, au cas où il y aurait une petite différence non visible au premier coup d'oeil ?

C'est bien le même message d'erreur ?

[Devlin111]

Oui, j'ai fais un copié collé, mais lorsque j'éxécute la PS, toujours l'erreur "Must declare the scalar variable "@id" "

[SQLDev]

Ne pouvant expliquer pourquoi ça ne s'exécute pas chez toi, je me suis un peu plus penché sur ce que tu cherches à faire.

Je passe sur les noms de procédures, table...
Il semblerait que tu cherches à insérer une ligne dans la table [USER_LANGUE], avec soit :
- 1 valeur pour 3 colonnes différentes (id, la_id et lvl) si tu trouves une valeur pour la_id dans la table [Langue] (en filtrant sur la colonne name)
- 1 valeur pour une seule colonne (id) si tu ne trouve pas de valeur pour la_id

Je te propose donc plutôt d'utiliser le script suivant :

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
CREATE PROCEDURE [dbo].[Insert_Langue] 
 
	@id int = NULL,
	@lvl int = NULL,
	@name nvarchar (100)= NULL
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
 	SET NOCOUNT ON;
 
	DECLARE @la_id INT; 
	DECLARE @Query NVARCHAR(MAX);
	DECLARE @Query2 NVARCHAR(MAX);
	DECLARE @Param NVARCHAR(MAX);
 
	SELECT   @la_id = la_id FROM LANGUE WHERE la_name = @name;
 
	-- Si la variable << @la_id >> contient une valeur, 
	-- insertion d'une ligne dans la table [USER_LANGUE] en renseignant les 3 valeurs connues : 
	--	. [id], 
	--	. [la_id], 
	--	. [lvl]
	IF (@la_id IS NOT NULL ) 
	BEGIN 
		INSERT INTO [dbo].[USER_LANGUE] ([id], [la_id], [lvl])
		VALUES (@id, @la_id, @lvl);
	END
	-- Sinon, insertion d'une ligne dans la table [USER_LANGUE] en renseignant uniquement la value connue : 
	--	 . [id]
	ELSE
	BEGIN 
		INSERT INTO [dbo].[USER_LANGUE] ([id]) VALUES (@id);
	END
END

[iberserk]

Ton problème initial vient uniquement du fait que tu utilise @id en tant que texte dans du sqldynamique (tu executes un texte...)
Quand la requete est lancée elle l'est dans son propre contexte donc @id n'est pas trouvé.

[Devlin111]

Re bonjour

iberserk, j'ai trouvé mon erreur finalement


SQLDev, non, ce n'est pas cela que je cherche à faire, mais je me suis aperçu en reprenant ligne par ligne que j'avais fais quelques erreurs de copié collé, dont une, celle qui posait problème :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
declare @Query nvarchar(MAX);
	declare @Query2 nvarchar(MAX);
	declare @Param nvarchar(MAX);
 
	BEGIN TRANSACTION 	
	SET NOCOUNT ON;
 
	SET @Query ='';
	SET @Param = @Param  + ',@id int '	
	SET @Param = @Param  + ',@lvl int '
	SET @Param = @Param  + ',@name nvarchar(100) '

Ici, je n'ai pas initialisé @Param, du coup, je démarrais par ','.
Je devais soit mettre

Code :

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

1
2
3
4
5
6
declare @Param nvarchar(MAX) = ' ';
 
SET @Query ='';
	SET @Param = @Param  + ',@id int '	
	SET @Param = @Param  + ',@lvl int '
	SET @Param = @Param  + ',@name nvarchar(100) '

soit

Code :

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

1
2
3
4
5
6
declare @Param nvarchar(MAX);
 
        SET @Query ='';
	SET @Param = '@id int '	
	SET @Param = @Param  + ',@lvl int '
	SET @Param = @Param  + ',@name nvarchar(100) '

Voilà, problème résolu (et j'ai aussi corrigé les petites erreurs de c/c, ma PS fonctionne). Merci pour l'aide

[Devlin111]

Desolé je me suis trompé