Discussion :

[Aerofly]

Bonjour,

Voici la situation :

J'ai une procédure qui exécute entre autre un insert sur une table, là le trigger insert de la table prend le relai et effectue des tests, en cas de soucis il envoie un raiserror.

Mon problème c'est que je ne sais pas comment savoir dans ma procédure si mon insert via le trigger a renvoyé une erreur.

Dans ma procédure j'ai tenté un truc du style mais cela ne marche pas :

Code :

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

1
2
3
4
 
INSERT INTO MA_TABLE (x, y,z) values (a,b,c);
IF @@ERROR <> 0
 Return;

@@ERROR reste désespérément à 0.

J'ai trouvé une solution de contournement, je sauvegarde le prochain identifiant de ma table avant l'insert, le prochain identifiant après l'insert et je les compare, s'ils sont égaux, l'insert c'est pas bien passé et je sors de la procédure.

N'y a-t-il pas un moyen plus propre ?

Merci d'avance
Aerofly

[SQLpro]

Il faut utiliser un bloc TRY/CATCH. En effet un trigger est ensembliste et atomique par principe....

Démo :

Code :

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

1
2
CREATE TABLE T (I INT);
GO

Code :

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

1
2
3
4
5
6
CREATE TRIGGER E_I_T
ON T
AFTER INSERT
AS
   SELECT 1/0;
GO

Code :

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

1
2
3
4
5
6
7
8
9
CREATE PROCEDURE P
AS
BEGIN TRY
   INSERT INTO T VALUES (0);
END TRY
BEGIN CATCH
   RAISERROR('Erreur lors de l''insertion', 16, 1);
END CATCH
GO

Code :

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

1
2
-- test :
EXEC P

Msg*50000, Niveau*16, État*1, Procédure*P, Ligne*7
Erreur lors de l'insertion

A +

[Aerofly]

Merci pour la réponse

Effectivement je peux englober l'insert dans un try catch mais ça me bloque pour la suite :/
Le trigger renvoie une erreur personnalisée paramétrable avec un certain numéro, numéro dont je me sers plus tard. En fait la procédure est appelé par du code c# et c'est à ce niveau entre autre que je gère la transaction et que je traite l'erreur en fonction du code erreur. La si je renvoi un raiserror dans la procédure, je perds cette info.

Aerofly

[SQLpro]

Vous commettez deux erreurs :
1) commencez vos transaction côté client est la pire des choses... En effet cela allonge démentiellement les temps de blocage des tables du fait des aller et retour réseau. Bref de quoi pourrir les performances !
2) un trigger est au sein de la transaction. Il est donc inutile de démarrer une transaction pour englober le code procédural d'un trigger !

A +

[azur668]

Il faut utiliser un bloc TRY/CATCH.

La si je renvoi un raiserror dans la procédure, je perds cette info.

Il suffit de NE PAS mettre de Try/Catch, et l'erreur du Trigger remontera jusqu'au code appelant C#, avec ses infos personnalisés !
Ce qui n'empêche pas d'englober le tout dans une transaction coté serveur, avec set XACT_ABORT ON.
Et pour savoir dans le code T-SQL si le Trigger a bloqué l'insert,
on pourrait peut-être récupérer les lignes insérés avec OUTPUT pour les compter ensuite ?