Discussion :

[Pragmateek]

Bonjour,

est-il possible pour un trigger de renvoyer une valeur ?

Plus précisément voici le trigger :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
CREATE TRIGGER ExtendedATrigger
   ON ExtendedA
   INSTEAD OF INSERT
AS
BEGIN
    SET NOCOUNT ON;
 
    INSERT INTO TableA(x, y)
    SELECT x, y FROM inserted;
 
    INSERT INTO TableB(TableAId, n, m)
    SELECT @@IDENTITY, n, m FROM inserted;
END

Le problème est le suivant :
ce trigger remplace une insertion dans la vue "ExtendedA"; or la valeur de retour de l'insertion est interprétée par une méthode (de LINQ) comme la clé de la dernière valeur insérée.
Le problème est que cette méthode lève une exception si la valeur de retour est null.

Donc il faut lui renvoyer @@IDENTITY.

Merci.

[SQLpro]

ATTENTION : vous commettez une lourde faute. En effet vous supposez implicitement qu'un trigger ne porte que sur 1 ligne. Ce n'est pas le cas. Un trigger n'est déclenché qu'une seule fois quelque soit le nombre de lignes...

En particulier retourner le @@identity ,e concernera que la dernière ligne insérée.....

Votre raisonnement est donc faux et votre façon de faire à revoir.

A +

[Pragmateek]

Merci de votre réponse.

En effet, cette implémentation est très basique, mais, dans ce cas précis, l'application qui utilise la view n'effectue qu'une insertion à la fois.

De plus, elle est là pour illustrer la question principale : est-il possible de renvoyer un résultat ?

Merci.

[SQLpro]

Possible, mais cela n'a aucun intérêt dans les triggers. N'oubliez pas que le trigger est situé à l'intérieur d'une transaction. Vous allongez donc d'autant la durée de la transaction, dont les verrous posés, le blocage et par là même les risques de verrous mortels !
C'est donc une pratique extrêmement déconseillée et fort peu recommandable.

A +

[Pragmateek]

Dans le cas général cela n'a peut être aucun intérêt et peut poser des problèmes mais malheureusement c'est une contrainte imposée par la bibliothèque utilisée.

Et même si cette solution n'est pas pérenne elle permettra au moins de faire fonctionner la partie persistance, permettant ainsi de se concentrer sur les aspects métiers de l'application, quitte à la modifier plus tard.

Merci d'avance.