Procédure stockée SQL Server

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

---

Bonjour

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

et sinon :

Citation:

---

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...

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 ?

@+

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.

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:

---

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...

Citation:

---

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.

Bonjour,

Citation:

---

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

@++ ;)

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

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

---

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 !

@++ ;)

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

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

@++ ;)

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...

Citation:

---

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?

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?

Citation:

---

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...?
@+

Citation:

---

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:cry::cry: 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.