Discussion :

[noobyyy]

Bonsoir,
j'ai du mal à utiliser les procédures stockées.

Dans cette exemple je dois retourner une liste de format FOR_ID et FOR_NOM
mais dans un sqlserver j'ai un soucis...

exemple de retour:

carte publicitaire 5
flyer 6

Sur sqlserver:

Code :

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

1
2
3
4
5
 
create procedure  SP_FORMAT
(@produit int , @nom char(50) output, @id int output)
as
select distinct(for_nom), for_id from refpf, forma where refpf_for=for_id AND refpf_pro=@produit

Code :

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

1
2
3
4
5
 
exec SP_FORMAT 3
 
Serveur : Msg 201, Niveau  16, État 4, Procédure SP_FORMAT, Ligne 0
La procédure 'SP_FORMAT' attend le paramètre '@nom', qui n'a pas été fourni.

je veux juste retourner mes deux listes FOR_ID et FOR_NOM a partir d'un paramètre, mais comment faire ?

[elsuket]

Bonjour,

Dans la mesure où tous les membres de ce forum participent librement, je trouve assez malvenu de commander la réécriture d'une procédure stockée pour mardi.
Même si telle n'était pas l'intention, c'est souvent de cette façon que c'est perçu par les participants réguliers à ce forum.
Néanmoins je vais répondre ...

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
CREATE PROCEDURE SP_FORMAT
(
	@produit INT,
	@nom CHAR(50) = NULL OUTPUT,
	@id INT = NULL OUTPUT
)
AS
BEGIN
	IF @produit IS NULL
	BEGIN
		RAISERROR('SP_FORMAT : Le paramètre @produit doit être valué !', 16, 1)
	END
	ELSE
	BEGIN
		SELECT DISTINCT(F.for_nom),
				F.for_id
		FROM dbo.refpf AS R
		JOIN dbo.forma AS F ON R.refpf_for = F.for_id
		WHERE R.refpf_pro = @produit
	END
END

1. Placez vos jointures dans la clause FROM/JOIN plutôt que dans la clause WHERE, c'est la norme, c'est plus lisible, et dans certains cas plus performant

2. qualifiez vos noms d'objet, cela évite au moteur de bases de données de devoir chercher dans les tables systèmes à quel schéma ils appartiennent, donc votre requête commencera à s'éxécuter plus tôt

3. pour effectuer des appels de procédures avec des paramètres optionnels, il suffit de les déclarer en tant que tel, c'est à dire en ajoutant la clause "= NULL" derrière le type de chacun des paramètres qui doivent l'être. Un paramètre déclaré comme OUTPUT est d'entrée/sortie, et non pas seulement de sortie.

@++

[agemis31]

Bonsoir,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
CREATE procedure  SP_FORMAT
@produit int, @nom char(50) output, @id int output
AS
BEGIN
SELECT TOP 1 
@nom = for_nom, 
@id = for_id 
FROM refpf, forma 
WHERE refpf_for=for_id AND refpf_pro=@produit
END 
 
-- Vous devez déclarer et passer les variables output
DECLARE @nom VARCHAR(50); DECLARE @id INT
EXEC SP_FORMAT 3, @nom OUTPUT, @id OUTPUT

J'ai ajouté un TOP 1 car votre DISTINCT laisse à penser que vous pouvez avoir plusieurs lignes en retours. Dans ce cas, les variables en output telles que vous les avez déclarées ne fonctionneront pas, car vous ne pouvez mettre plusieurs valeurs dans un variable scalaire.
Si c'est le cas, vous pouvez renvoyer un jeu de résultat sans passer par des variables en sortie, ou créer une fontion de table.

En règle génerale, on évite de nommer ses procédures stockées sp_xxx car les procédures stockées étendues commencent aussi par sp_

Enfin, demander une réécriture de requête pour mardi sur un forum tel que celui ci me semble limite. Nous avons aussi un boulot !

Par contre, je n'avais pas lu correctement votre post, puisque vous précisez que vous devez retourner une liste :-)

@+

[elsuket]

Ha ben on a répondu en même temps, et approximativement la même chose à propos du "pour mardi"

@++

[agemis31]

Oui, sauf que tu as mieux lu que moi :-)

[elsuket]

Ben il se débrouillera, il a tout demain pour trouver

@++

[noobyyy]

merci c'est gentil ! donc je ne peux pas retourner une liste avec une procédure stockée je dois faire une fonction de table c'est bien sa ?

[agemis31]

Bonsoir,

Non, ce n'est pas ça, vous pouvez le faire avec les deux, dans votre cas, la procédure stockée est sans doute toute indiquée.

Voici un cours utile sur TSQL
Regardez le chapitre 4.2

PS: On ne parle pas de liste mais de jeu de resultat

@+