Procédure stockée SQL Server

[kFedali]

Je veux créer une procédure stockée pour rendre la saisie d'un champs obligatoire. Voici ma procédure, mais, elle marche pas veuillez m'aider SVP:

Code :

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
USE [KIMOCE]
GO
/****** Objet :  StoredProcedure [dbo].[CTC_TEST]    Date de génération du script : 01/04/2011 08:44:41 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER PROCEDURE [dbo].[CTC_TEST]  
AS
 
DECLARE @DstCtcDsc VARCHAR(71)
DECLARE RqtEvent_Cursor CURSOR FOR 
SELECT DstCtcDsc
FROM p_rqt
WHERE RqtInCde > 0 
 
OPEN CtcEvent_Cursor 
FETCH NEXT FROM CtcEvent_Cursor INTO @DstCtcDsc
WHILE @@FETCH_STATUS = 0 
BEGIN
-- Recherche des demandes à traiter
IF @DstCtcDsc IS NULL
GOTO LBL_ERROR 
 
LBL_ERROR: 
RAISERROR('Contact obligatoire',11,2) WITH SETERROR 
END

[aieeeuuuuu]

Bonjour

Heu... il manque a un bout de la SP la non (sinon c'est normal qu'elle ne fonctionne pas ) ?

et sinon :

mais, elle marche pas

c'est dire ? postez le message d'erreur.

Avec seulement la moitié de la SP, et sans message d'erreur, cela va etre difficile de vous aider...

[Zabriskir]

Bonjour,

Oubliez les curseurs, ce n'est pas comme ça qu'on programme en SQL.
Pourquoi ne pas rendre NOT NULLABLE la colonne qui vous intéresse ?

@+

[kFedali]

J'ai pas d'erreur, mais j'arrive pas à contrôler le champs.
Je travail sur un outil dont j'ai pas le code source mais ilest paramétrable et je peux contrôler le champs par une procédure stockée.

[aieeeuuuuu]

je ne comprend pas bien

vous lancez la procédure tel quel ? avez vous posté la totalité de la procédure ?

il est normal qu'elle vous remonte tout le temps une erreur, il y a un RAISERRROR sur le chemin de code.

pourquoi utiliser un GOTO ?

pourquoi ne pas faire simplement

Code SQL :

1
2
3
 
IF @DstCtcDsc IS NULL
RAISERROR('Contact obligatoire',11,2) WITH SETERROR

Sinon mettez un autre GOTO devant votre label LBL_ERROR pour éviter que le raiserror ne soit exécuté s'il n'y a pas d'erreur...

[iberserk]

J'ai pas d'erreur, mais j'arrive pas à contrôler le champs.
Je travail sur un outil dont j'ai pas le code source mais ilest paramétrable et je peux contrôler le champs par une procédure stockée.

Il n'y a pas de champs dans une base de données mais des colonnes...

Vous voulez seulement contrôler lors de vos UPDATE/INSERT si la colonne est bien renseignée?

Si c'est le cas et que vous ne pouvez modifier la structure de la table (CONTRAINTE NOT NULL sur la colonne...) passez par des TRIGGERS.

[elsuket]

Bonjour,

J'ai pas d'erreur, mais j'arrive pas à contrôler le champs.

C'est normal, il manque le FETCH dans la boucle.
Pour le faire fonctionner il aurait fallu écrire :

Code :

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
ALTER PROCEDURE [dbo].[CTC_TEST]
AS
 
	DECLARE @DstCtcDsc VARCHAR(71)
 
	DECLARE RqtEvent_Cursor CURSOR FOR
		SELECT DstCtcDsc
		FROM p_rqt
		WHERE RqtInCde > 0
 
	OPEN CtcEvent_Cursor
	FETCH NEXT FROM CtcEvent_Cursor INTO @DstCtcDsc
	WHILE @@FETCH_STATUS = 0
	BEGIN
		-- Recherche des demandes à traiter
		IF @DstCtcDsc IS NULL
		GOTO LBL_ERROR
 
		LBL_ERROR:
		RAISERROR('Contact obligatoire',11,2) WITH SETERROR
 
		-- FETCH MANQUANT !
		FETCH NEXT FROM CtcEvent_Cursor INTO @DstCtcDsc
	END
	CLOSE CtcEvent_Cursor
	DEALLOCATE CtcEvent_Cursor

L'indentation n'a rien à voir, c'est juste pour la lisibilité.
Vous noterez de plus que vous ne dés-allouez pas le curseur (CLOSE + DEALLOCATE).

Oubliez les curseurs.
SQL est un langage ensembliste, donc il est conçu pour traiter très rapidement les données dans leur ensemble, et pas dans leur unité.

Pareil pour les LBL_ERROR: et les GOTO : c'est vieux et moche.
Vous auriez pu donc écrire, avec votre curseur :

Code :

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
ALTER PROCEDURE [dbo].[CTC_TEST]
AS
BEGIN
	SET NOCOUNT ON
 
	DECLARE @DstCtcDsc varchar(71)
 
	DECLARE RqtEvent_Cursor CURSOR FOR
		SELECT	DstCtcDsc
		FROM	p_rqt
		WHERE	RqtInCde > 0
 
	OPEN CtcEvent_Cursor
	FETCH NEXT FROM CtcEvent_Cursor INTO @DstCtcDsc
	WHILE @@FETCH_STATUS = 0
	BEGIN
		-- Recherche des demandes à traiter
		IF @DstCtcDsc IS NULL
		BEGIN
			RAISERROR('Contact obligatoire',11,2) WITH SETERROR
			RETURN
		END
	END
	CLOSE CtcEvent_Cursor
	DEALLOCATE CtcEvent_Cursor
END

Je ne vois pas non plus l'intérêt sur SETERROR pour le RAISERROR.

Je pense que c'est ce qu'il vous faut plutôt :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
ALTER PROCEDURE [dbo].[CTC_TEST]
AS
BEGIN
	IF EXISTS
	(
		SELECT	*
		FROM	dbo.p_rqt
		WHERE	RqtInCde > 0
		AND	DstCtcDsc IS NULL
	)
	BEGIN
		RAISERROR('Contact obligatoire', 16, 1)
	END
END

Ici la table p_rqt est recherchée selon le filtre.
Si au moins une ligne correspond, ce que SQL Server peut savoir très vite si votre table est bien indexée, l'erreur est levée.

Dans votre cas vous parcourez, un à un, les lignes de votre curseur, en espérant trouver un DstCtcDsc à NULL.

Autrement dit vous tournez encore les pages pour trouver celle qu'il vous faut, alors que je suis déjà en train de la lire !

SQL est aussi un langage déclaratif : vous dites ce que vous voulez obtenir, pas la façon dont vous voulez l'obtenir (comme dans un langage fonctionnel).

@++

[kFedali]

Je vais vous expliquer ce que je veux faire:
J'ai un outil de gestion qui permet de créer des demandes clients. Lors de la création d'une demande je veux obliger l'utilisateur à saisir un champs qui n'est pas obligatoire par défaut. L'outil me permer d'associer une procédure stockée à ce champs. C'est pourquoi en fait j'ai décider de faire ça.
Donc, comment faire, je suis vraiment perdue

[iberserk]

Pouvez vous passer la valeur qui va être insérée dans de la colonne en paramètre de cette procédure stockée?
Si oui rien de plus simple:

Code :

1
2
3
4
5
6
ALTER PROCEDURE [dbo].[CTC_TEST] (@VALUE VARCHAR(50))
AS
BEGIN
   IF @VALUE IS NULL OR LEN(LTRIM(RTRIM(@VALUE)))=0
       RAISERROR('Contact obligatoire',11,2)
END

[elsuket]

Non seulement ce devrait être implémenté comme vous le propose iberserk, mais vous devriez réaliser en plus ce contrôle côté applicatif !

@++

[alassanediakite]

Salut
Pour rendre obligatoire un champs d'une table vous avez deux choix:

1. mettre "not null" dans la définition du champ (lors de create/alter table)
2. ou faire un trigger qui vérifie le champ avant (instead of) insert ou update

Les procédures stockées sont faits pour des taches répétitives.
Bonne recherche

[elsuket]

La première proposition d'alassanediakite étant de loin la plus propre et la plus performante.

@++

[EMC51]

Est-ce que l'outil de gestion permet de définir ou même d'imposer un paramètre à la procédure stockée associée à ton champ ?

Ca peut sembler étrange de vérifier toute la table pour une seule entrée...

Je pense que ce genre de contrôle est destiné à implémenter des règles de gestion spécifiques.

Donc on a besoin de la valeur...

[iberserk]

La première proposition d'alassanediakite étant de loin la plus propre et la plus performante.

@++

A priori il ne peux pas modifier la structure des tables?

[alassanediakite]

Salut
Comment voulez-vous qu'une PS vérifie un champ?
Supposons que l'outil (mais de quel outil s'agit-il?) permet d'associer un évènement (après mise à jour par exemple) au champ ciblé: on appelle le PS et qui renvoi une erreur, erreur qu'il va falloir capter et interpréter dans l'outil.
Ma question est: si l'outil permet d'associer des évènement à un champ (je ne voie pas comment le faire autrement!!!), pourquoi ne pas laisser l'outil gérer le contenu du champ? autant que je sache, les PS s'executent sur le serveur?

A priori il ne peux pas modifier la structure des tables?

... s'il peut lancer des PS qu'est ce qui l'empêche d'en faire un avec alter table...?
@+

[Oliv-]

Envoyé par kFedali

Je veux créer une procédure stockée pour rendre la saisie d'un champs obligatoire. Voici ma procédure, mais, elle marche pas veuillez m'aider SVP:

As tu un exemple d'une ps proposée par l'éditeur et qui fonctionne, ce sera peut être plus simple d'en comprendre l'utilité et le fonctionnement.