Discussion :

[paintbox]

Bonjour,

j'écris une procédure stockée(non terminée !) qui reçoit différents paramètres dont 1 Table Value Parameters dont le type est définit comme ceci :

Code :

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

1
2
3
CREATE TYPE [dbo].[CoursType] AS TABLE(
	[CoursId] [int] NULL,
	[CoursNom] [varchar](50) NULL

Ma procédure stocké reçoit le paramètre @CoursSuivis qui est de ce type.

Le but de cette procédure est de modifier une fiche élève et de parcourir la Table EleveCours et modifier les cours d'un même élève. Elle se présente comme ceci:

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
CREATE PROCEDURE [dbo].[sp_EditerEleve]
	@idEleve int,
	@nom varchar(50),
	@prenom varchar(50),
	@age int,
	@idstatut int,
	@annee int,
	@actif int,
	@cheminPhoto varchar(350),
	@CoursSuivis CoursType READONLY
 
AS
BEGIN
IF(@age BETWEEN 12 AND 20)
	BEGIN
		UPDATE Eleve SET Nom=@nom , Prenom=@prenom, Age=@age, IDStatut=@idstatut, Annee=@annee, Actif=@actif, Photo=@cheminPhoto
		WHERE IDEleve = @idEleve 
	END
 
 
	BEGIN
	DECLARE @NbreCoursActuel int
	DECLARE @NbreCours int
	BEGIN
		set @NbreCoursActuel = (SELECT COUNT(*) FROM EleveCours WHERE IDEleve = @idEleve);
		set @NbreCours = (SELECT COUNT(*) FROM @CoursSuivis);
	END
	IF @NbreCoursActuel = @NbreCours
	PRINT 'OK_1'
		BEGIN
		DECLARE @identifiantEleve int;
		SET @identifiantEleve  = ( SELECT IDEleve FROM EleveCours WHERE IDEleve = @idEleve);
		While (@identifiantEleve = @idEleve)
			BEGIN
			UPDATE EleveCours SET IDCours = (SELECT CoursId FROM @CoursSuivis), CoursNom = (SELECT CoursNom FROM @CoursSuivis) WHERE IDEleve = @idEleve;
			END
		END
	END
END

Mon problème est que lorsque j'exécute ma procédure, ma sous requète retourne visiblement plusieurs valeurs ce qui n 'est pas permis lorsqu'elle suit un =, !=, <... . Le message d'erreur est :

Code :

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

1
2
Msg 512, Niveau 16, État 1, Procédure sp_EditerEleve, Ligne 38
Subquery returned more than 1 value. This is not permitted when the subquery follows =, !=, <, <= , >, >= or when the subquery is used as an expression.

Auriez-vous une idée?

Merci

[iberserk]

Mon problème est que lorsque j'exécute ma procédure, ma sous requète retourne visiblement plusieurs valeurs ce qui n 'est pas permis lorsqu'elle suit un =, !=, <... . Le message d'erreur est :

JE ne vois pas ce que l'on peut vous dire de plus, vous connaissez manifestement la réponse... inclue dans votre question

L'une de ces deux sous-requêtes retourne plusieurs résultat... a vous de voir pourquoi

[SQLDev]

Bonjour,

C'est cette ligne là qui doit poser problème :

Code :

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

UPDATE EleveCours SET IDCours = (SELECT CoursId FROM @CoursSuivis), CoursNom = (SELECT CoursNom FROM @CoursSuivis) WHERE IDEleve = @idEleve;

Si la variable de type udt @CoursSuivis contient plusieurs valeurs, ce qui est fort probable sinon pas la peine d'utiliser ce type de données, cette erreur ce produira.
Vous tentez de mette à jour chaque ligne de la table EleveCours où votre condition est remplie (filtre sur IDEleve) avec plusieurs valeurs (le nombre de ligne trouvée dans @CoursSuivis).

Il va falloir gérer vos traitements plus finement, en comparant les données de la table EleveCours à celles de la variable @CoursSuivi et faire les traitements qui s'imposent (UPDATE, DELETE, INSERT)

D'un point de vue général, vous allez avoir quelques problèmes pour obtenir le résultat que vous attendez, cette partie de code est très critique :

Code :

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

1
2
3
4
5
6
7
		DECLARE @identifiantEleve int;
		SET @identifiantEleve  = ( SELECT IDEleve FROM EleveCours WHERE IDEleve = @idEleve);
		While (@identifiantEleve = @idEleve)
			BEGIN
			UPDATE EleveCours SET IDCours = (SELECT CoursId FROM @CoursSuivis), CoursNom = (SELECT CoursNom FROM @CoursSuivis) WHERE IDEleve = @idEleve;
			END
		END

Boucle infinie très probable.
Déjà @identifiantEleve sera toujours égal à @idEleve s'il existe dans la table EleveCours.
Ensuite votre boucle While teste une condition sur 2 variables qui ne varient jamais dans votre boucle, la condition n'évoluera donc jamais, ce qui fait que si vous rentrez dans la boucle vous n'en ressortirez pas.

Vous mettez également beaucoup de BEGIN / END qui ne servent pas.

[SQLpro]

Déjà on enlève toutes les BEGIN END inutiles et les parenthèses qui servent à rien et on indente... (on se croirait en primaire)...

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
CREATE PROCEDURE [dbo].[sp_EditerEleve]
	@idEleve int,
	@nom varchar(50),
	@prenom varchar(50),
	@age int,
	@idstatut int,
	@annee int,
	@actif int,
	@cheminPhoto varchar(350),
	@CoursSuivis CoursType READONLY
 
AS
IF @age BETWEEN 12 AND 20
		UPDATE Eleve 
        SET    Nom=@nom ,  
               Prenom=@prenom, 
               Age=@age, 
               IDStatut=@idstatut, 
               Annee=@annee, 
               Actif=@actif, 
               Photo=@cheminPhoto
		WHERE  IDEleve = @idEleve 
 
DECLARE @NbreCoursActuel int, @NbreCours int
BEGIN
	SELECT @NbreCoursActuel = COUNT(*) 
    FROM EleveCours WHERE IDEleve = @idEleve;
    SELECT @NbreCours = COUNT(*) 
    FROM @CoursSuivis;
END
IF @NbreCoursActuel = @NbreCours
	PRINT 'OK_1'
 
DECLARE @identifiantEleve int;
SELECT @identifiantEleve  = IDEleve 
FROM   EleveCours 
WHERE IDEleve = @idEleve;
WHILE @identifiantEleve = @idEleve
   UPDATE EleveCours 
   SET IDCours =  (SELECT CoursId  
                   FROM @CoursSuivis), 
       CoursNom = (SELECT CoursNom 
                   FROM @CoursSuivis) 
   WHERE IDEleve = @idEleve;

Ensuite vous utilisez des sous requêtes là ou des jointures devraient être faites et du procédural là ou des requêtes devrait être...

A +

[SQLpro]

De plus ce code ne sert à rien dans votre PS :

Code :

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

1
2
3
4
5
6
7
8
9
10
DECLARE @NbreCoursActuel int, @NbreCours int
BEGIN
	SELECT @NbreCoursActuel = COUNT(*) 
    FROM EleveCours 
    WHERE IDEleve = @idEleve;
    SELECT @NbreCours = COUNT(*) 
    FROM @CoursSuivis;
END
IF @NbreCoursActuel = @NbreCours
	PRINT 'OK_1'

Ces variables ne sont pas utilisées ailleurs...

A +

[SQLpro]

En sus cette boucle est une boucle infinie :

Code :

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

1
2
3
4
5
6
7
WHILE @identifiantEleve = @idEleve
   UPDATE EleveCours 
   SET IDCours =  (SELECT CoursId  
                   FROM @CoursSuivis), 
       CoursNom = (SELECT CoursNom 
                   FROM @CoursSuivis) 
   WHERE IDEleve = @idEleve;

A +