Discussion :

[kheironn]

Bonjour
Je dois réaliser quelques triggers et je me pose un question sur la forme et la performance

syntaxe 1

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
CREATE OR REPLACE TRIGGER MonTrigger
AFTER INSERT OR UPDATE
ON com_e
FOR EACH ROW  -- pour chaque ligne parceque ce n'est pas implicite !
DECLARE
  vIdSuivi NUMBER := -1;
BEGIN  
-- si mon select ne trouve rien, alors, je fais la suite, sinon dodo !
  FOR crs IN (SELECT idsuivi FROM MaTable WHERE flowtype = 'MonFlux' AND key = :new.NUMCOMMANDE AND stateid = 0)
  LOOP
    vIdSuivi := crs.idsuivi;
  END LOOP;
  IF vIdSuivi = -1 THEN
    INSERT INTO MaTable(idsuivi,flowtype,key,company,stateid,process) VALUES(MaSequence.nextval,'MonFlux',:new.NUMCOMMANDE,'',0,'Toto');
  END IF;
END ;

syntaxe2

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
CREATE OR REPLACE TRIGGER MonTrigger
AFTER INSERT OR UPDATE
ON com_e
FOR EACH ROW  -- pour chaque ligne parceque ce n'est pas implicite !
DECLARE
  vIdSuivi NUMBER := -1;
BEGIN  
-- si mon select ne trouve rien, alors, je fais la suite, sinon dodo !
  SELECT idsuivi INTO vIdSuivi FROM MaTable WHERE flowtype = 'MonFlux' AND key = :new.NUMCOMMANDE AND stateid = 0;
EXCEPTION
  WHEN NO_DATA_FOUND THEN
    INSERT INTO MaTable(idsuivi,flowtype,key,company,stateid,process) VALUES(MaSequence.nextval,'MonFlux',:new.NUMCOMMANDE,'',0,'Toto');
END ;

Quelle syntaxe choisir ?
Ou une autre ???
Pour info mon sélect ne renvoie qu'une ligne unique.

Merci d'avance

[Waldar]

L'idée c'est de faire directement l'insertion dans MaTable sans tester l'existence de la ligne au préalable, et de gérer l'exception si la ligne existe déjà.

Par contre, si je comprends la logique, quand vous faites un insert dans COM_E vous insérer une ligne dans MaTable, mais si vous faites un update, la ligne existant déjà vous n'allez rien faire.
Dans ce cas un déclencheur after insert doit être suffisant.

[kheironn]

L'idée c'est de faire directement l'insertion dans MaTable sans tester l'existence de la ligne au préalable, et de gérer l'exception si la ligne existe déjà.

Non, Cette ligne ne doit être insérée que si elle n'existe pas, il s'agit d'un ordre d'export de données qui sera traité par un batch.
=>Si une ligne pour monflux et maclef existe déjà en état à traiter (stateid = 0), on ne remet pas un deuxième ordre, les données de com_e ont été créées/modifiées et n'ont pas encore été exportées.
=>S'il existe en état traité, etc. (stateid != 0) on remet un nouvel ordre : les données de la table con_e ont été modifiée et doivent être exportées.

Par contre, si je comprends la logique, quand vous faites un insert dans COM_E vous insérer une ligne dans MaTable, mais si vous faites un update, la ligne existant déjà vous n'allez rien faire.
Dans ce cas un déclencheur after insert doit être suffisant.

Comme dit plus haut, la ligne insérée aura changée de stateid et poura être insérée de nouveau.

Dans tous les cas une ligne dans matable avec un FlowType et une Key donnée ne peut exister qu'une seule fois en état stateid = 0, par contre elle existera de nombreuses fois en stateid > 1.

[mnitu]

D’une manière générale l’algorithme qui ne gère pas d’exception est plus rapide, parce que gérer des exceptions ajout un coût supplémentaire. Mais cela dépende vraiment de la proportion des cases où l’exception sera levée. C’est-à-dire si l’exception arrivé que dans 2% des cas sur 100 000 exécutions il sera assez difficile de voir la différence. Et pour qu’une différence y’en a il faut de dizaines des milliers d’exécutions.
Tout ça c’est juste pour dire que sauf situation exceptionnelle il y a des autres chats à fouetter dans la quête de la performance.