Discussion :

[freud]

Bonjour,

J'ai une petite question sur la boite de message qui s'affiche dans une app cliente à l'aide de RAISERROR figurant dans une procedure stockées par exemple :

Code :

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

1
2
3
4
5
6
7
 
 IF (@@error!=0)
    BEGIN
        RAISERROR  20000 'message quelconque '
        ROLLBACK TRAN
        RETURN(1)
    END

Et bien j'aimerais savoir si ce ROLLBACK TRAN et le RETURN(1)
s'executent aprés que l'utilisateur ai cliquer sur le bouton du RAISERROR ?
Je vous remercie.

[Madinico]

Normalement oui. Les procédures stockées SQL Server s'exécutent jusqu'à ce qu'elles trouvent un mot-clé de sortie (type RETURN) ou qu'elles se terminent.
Donc une fois que tu auras validé ta boîte ton code continuera à s'exécuter.

[freud]

donc c'est pénalisant dans un contexte multi-utilisateur si l'utilisateur ne clique pas sur le bouton il y a un ROLLBACK TRAN en attente et ca pourrait bloquer certaines choses. Peut-être une variable de type varchar en output à la place de RAISERROR est à préférer ainsi elle sera récuperer dans l'app cliente ? n'y a-t-il pas mieux ?

[Madinico]

Effectivement, il vaut mieux privilégier les variables de sortie ou la définition d'un code retour explicite.

Solution 1 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
CREATE PROCEDURE USP_MA_PROC @var INT, @sortie VARCHAR(255) OUTPUT AS
SET @sortie ='traitement correct'
BEGIN TRAN T1
INSERT INTO .....
IF @@ERROR <> 0 
BEGIN
SET @sortie ='Erreur numero d insertion'
ROLLBACK TRAN T1
END
COMMIT TRAN T1

Solution 2

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CREATE PROCEDURE USP_MA_PROC @var INT AS
BEGIN TRAN T1
INSERT INTO .....
IF @@ERROR <> 0 
BEGIN
ROLLBACK TRAN T1
RETURN 'Erreur numero d insertion'
END
COMMIT TRAN T1

[freud]

Ainsi c'est mieux merci Madinico

[SQLpro]

pas de panique. Une erreur SQL. De quelque niveau que ce soit ne sera jamais vu qu'après que la proc stock ne se soit entièrement déroulée.

Autrement dit le code posté est presque parfait... En effet la valeur de retour est le numéro d'erreur. Et 0 si tout va bien. Donc je n'aurais pas mis return (1) qui fait croire au client que c'est l'erreur n°1 et non l'erreur 20000

Enfin, une remarque, le signe différence n'est pas != (ça c'est du Oracle) mais <> (ça c'est la norme et SQL Server la respecte).

Code :

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

1
2
3
4
5
6
7
 
IF @@error <> 0
    BEGIN
        RAISERROR  20000 'message quelconque '
        ROLLBACK TRAN
        RETURN
    END

Voila qui serait mieux !

A +

[freud]

Merci SQLpro, juste une petite question ..... si j'ai bien compris RAISERROR affichera la boite de message et je dois alors récuperer dans l'application cliente l'erreur 20000 contenu dans @@error ?

Merci

[SQLpro]

RAISERROR n'affiche rien, il positionne la valeur de la variable de retour à la valeur de l'erreur.
Sur le client vous faîtes le traitement que vous voulez.
La première chose consisterait donc à se poser la question de la valeur du code retour de votre fonction et s'il est différent de zéro c'est qu'il y a peut être une erreur. Mais attention certaines valeurs du code de retour ne sont pas des erreurs mais des messages. Dans ce cas il faut aussi compter sur le niveau de l'erreur. C'est pourquoin certains langage client permettent de mettre uen oeuvre un gestionnaires d'erreur. Mais ceci n'est plus au niveau de SQL Server !

A +

[freud]

Merci SQLpro