Procédure stockée

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:

---

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:

---

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 ?

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:

---

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.

@++ ;)

Bonsoir,

Code:

---

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 :-)

@+

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

@++ ;)

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

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

@++ ;)

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 ?

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 ;)

@+