Discussion :

[KuRuVI]

Bonjour,

Je travaille sur les triggers et une question m'est venu. Je sais que devant new et old on doit mettre des : dans le bloc sql. Mais j'ai entendu du dire qu'on ne les mettais pas dans les requettes sql ou les if.
pouvez-vous me confirmer la chose et aussi me dire si dans le trigger suivant j'aurais du mettre les : devant les new.
Merci d'avance.

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
17
18
19
20
21
22
CREATE TRIGGER verif_vip
AFTER INSERT OR UPDATE
ON Activite
FOR EACH ROW
 
DECLARE
    somme_total Activite.tarifH%TYPE;
 
BEGIN
 
 SELECT sum(duree*tarifH) INTO somme_total FROM participe, Activite
  WHERE participe.IdAct = Activite.IdAct AND new.IdCli = participe.Cli;
 
    IF somme_total > 1000 THEN 
           IF (new.IdCli NOT IN (SELECT IdCli FROM VIP)) THEN
                INSERT INTO VIP VALUES (IdCli, sysdate):
           END IF;
 
    END IF;
 
END;
/

[SergioMaster]

Bonjour,

Je travaille sur les triggers et une question m'est venu. Je sais que devant new et old on doit mettre des : dans le bloc sql.
Mais j'ai entendu du dire qu'on ne les mettais pas dans les requettes sql ou les if.

cela dépend beaucoup du SGBD, ce que vous n'avez pas indiqué

Code :

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

1
2
3
 
 SELECT sum(duree*tarifH) INTO somme_total FROM participe, Activite
  WHERE participe.IdAct = Activite.IdAct AND new.IdCli = participe.Cli;

utilisez plutôt une forme plus moderne (normalisée ?) pour la jointure

Code :

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

1
2
SELECT sum(duree*tarifH) INTO somme_total FROM participe JOIN Activite  ON  participe.IdAct = Activite.IdAct
  WHERE participe.Cli = new.IdCli ;

Code :

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

1
2
3
4
IF somme_total > 1000 THEN 
           IF (new.IdCli NOT IN (SELECT IdCli FROM VIP)) THEN
                INSERT INTO VIP VALUES (IdCli, sysdate):
           END IF;

AMHA cela peut s'optimiser tout dépend si le NOT EXISTS est possible dans ce SGBD, en tout les cas le NOT IN c'est vraiment pas le top !

Code :

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

1
2
3
IF (SOMME_TOTAL>1000) AND (NOT EXISTS SELECT 1 FROM VIP WHERE IDCLI=NEW.IDCLI ) THEN
   INSERT INTO VIP VALUES (IdCli, sysdate):
 END IF;

[KuRuVI]

Bonjour,

Merci pour votre réponse.
Effectivement je pensais que la syntaxe des : était la même partout. Le SGBD que j'utilise est Oracle.

De plus pouvez-vous m'expliquer pourquoi une forme normalisée est mieux ici?

Code :

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

1
2
3
IF (SOMME_TOTAL>1000) AND (NOT EXISTS SELECT 1 FROM VIP WHERE IDCLI=NEW.IDCLI ) THEN
   INSERT INTO VIP VALUES (IdCli, sysdate):
 END IF;

Que représente le 1 après le SELECT ?


Merci encore pour votre réponse.

[SergioMaster]

Bonjour,

pour répondre à la question sur la "normalisation" je dirais que c'est plus une question de modernisation et de lecture qu'autre chose.
Cela permet de séparer les relations des conditions, et même si je présume que l'optimiseur d'oracle le fait de manière implicite c'est quand même mieux de l'écrire de manière explicite. Ici vous avez affaire à requête simple mais avec une dizaine de tables ?
De même, je n'ai pas osé, ne connaissant pas le SGBD, utiliser des alias de table mais je le conseille vivement, toujours pour des questions de lecture "humaine" (je pense toujours à la maintenance)

Pour le 1 , cela m'évite simplement de récupérer une colonne inutile la seule nécessité étant de savoir si la requête renvoi null ou non (ce que le EXISTS fait)


pour les ":" , je n'utilise pas Oracle donc je ne saurais être affirmatif, pourquoi ne pas demander dans le forum Oracle ?

[KuRuVI]

Bonjour,

Merci beaucoup pour votre réponse je comprends mieux.
Oui, je vais voir sur le forum d'oracle.

Je vous souhaite une bonne journée,

Cordialement.