Discussion :

[Cyrille36]

bonjour à tous,

Je galère encore sur les PS...

Pourriez vous me dire :
1/ Pourquoi cette PS ne fonctionne pas
2/ Pourquoi attend t'elle des paramètres d'entrées à l'execution !?

Obectif : Récupérer tous les enreg. de la table Articles avec 3 champs :
référence, stock et un troisième qui n'existe pas mais qui contient une chaine de caractères conditionnée ('OK' ou 'PAS OK').

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
ALTER PROCEDURE test4
 
 
(@St int,
@Statut nchar(20) output)
 
 
AS
    SELECT référence,
               @St= stock
    from articles
 
BEGIN
 
 if  @St<200
 select @Statut='OK'
 else
 select @Statut='PAS OK'
 
END

Merci de votre aide, Cyrille.

[Aka Guymelef]

Bonjour,

Le SELECT doit se trouver entre le BEGIN et END de la procédure, pas après le AS.

Code sql :

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
ALTER PROCEDURE test4
 
(@St int,
@Statut nchar(20) output)
 
AS
 
BEGIN
    SELECT référence,
               @St= stock
    from articles
 
 if  @St<200
 select @Statut='OK'
 else
 select @Statut='PAS OK'
 
END

[Cyrille36]

Désolé, ça ne marche pas.
Voici le message d'erreur : Une instruction SELECT qui affecte une valeur à une variable ne doit pas être combinée avec des opérations d'extraction de données.

De plus, je doit ecrire : execute Test4 XX,XX pour la lancer !!! Je ne comprends pas. il n'y a pas de parmatres d'entrées.

Qu'est ce qui m'echappe ?

[Aka Guymelef]

Bien sûr que si qu'il y a des paramètres d'entrées !

Code sql :

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

ALTER PROCEDURE test4 (@St int, @Statut nchar(20) output)

Dont un qui est en sortie justement donc tu dois déclarer une variable avant l'exécution de ta requête qui va recevoir cette valeur de sortie soit :

Code sql :

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

1
2
3
4
5
DECLARE @SORTE AS NChar(20)
 
EXEC test4( 10, @SORTE OUTPUT )
 
SELECT @OUTPUT

Enfin pour la première erreur que tu cites c'est parce qu'il faut remplacer les SELECT par des SET.

Code sql :

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

1
2
3
4
IF  @St<200
  SET @Statut='OK'
else
   SET @Statut='PAS OK'

[Franck SORIANO]

En fait, ce que tu veux c'est :

Code sql :

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

1
2
3
4
5
6
7
8
 
alter procedure Test4
as
  select
    [Référence],
    [Stock],
    [Status] = case when Stock < 200 then 'OK' else 'PAS OK' end
  from articles

Visiblement, tu mélanges plusieurs choses :
- Ce que tu déclares entre parenthèse après le nom de la SP, ce sont des paramètres d'appels. Donc des paramètres qu'il faut renseigner au moment de l'appel. Dans ta SP, tu sembles utiliser ces paramètres comme des variables locales de la SP.
- Si tu as besoin de variables locales dans la SP, il faut les déclarer avec DECLARE dans le corps de la SP (donc après le AS).
- Le mot clé SELECT peut avoir plusieurs usages :
* Tu fais un SELECT normal au sens SQL du terme. Dans ce cas, tu va renvoyer un jeu de résultats à l'appelant.
* Tu peux aussi t'en servir pour affecter une valeur à une ou plusieurs variables (select @MaVariable1 = UneValeur, @MaVariable2 = Valeur2). Dans ce cas, il vaut mieux que ton select retourne une et une seule ligne. Et tu dois obligatoirement affecter tous les champs retournés par le select à une variable.
* Tu n'as pas le droit de faire un select qui retourne à la fois des champs et qui fait des affectations à une variable.

Depuis MSSQL 7, Si tu veux simplement affecter une valeur immédiate à une variable et que cette valeur ne provient pas d'un calcul à partir d'une valeur d'une table, tu peux l'affecter directement avec le mot clé SET, au lieu d'utiliser SELECT :

Code sql :

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

1
2
 
set @MaVariable = 5