Discussion :

[kekesar]

Bonjour à tous, j'ai un petit souci, et malgré des recherches et autres je plante x) ...

Voici le contexte,

J'ai une table qui est alimenté cependant cette table à une contrainte d'unicité et etrangère avec une table d'utilisateur.
C'est pourquoi je souhaite crée un trigger permettant de juste récupéré le nom de l'utilisateur que nous allons inséré dans notre table afin de crée un utilisateurs avant dans l'autre table.

Et voici ce à quoi j'en suis arrivé :

Code :

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

1
2
3
4
5
6
7
CREATE OR REPLACE TRIGGER add_pool_contact
BEFORE INSERT ON POOL 
FOR EACH ROW
BEGIN
    insert into CONTACT (nom, prenom,mdp, userdroit) 
    SELECT New.nom, 'test','test',1 FROM POOL;
END;

Cependant impossible de crée le trigger une erreur me bloque :

Code :

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

1
2
3
>[Error] Script lines: 7-7 --------------------------
 ORA-00900: instruction SQL non valide
 Script line 7, statement line 1, column 0 

Des explications ou aides à offrir que je puisse comprendre et avance ?

[Michel.Priori]

Bonsoir,

ça va être compliqué

Les contraintes déclaratives (null, check, PK, FK) sont évaluées AVANT les triggers.
Donc, c'est mort.
Il vaut mieux faire une procédure qui tente de faire l'ajout et sur erreur ajoute le contact et rejoue l'insert.

Ensuite pour l'erreur :
On a accès aux curseurs NEW et OLD à l'intérieur de d'un trigger FOR EACH ROW.
Du coup New.nom est un scalaire qu'on va pouvoir utiliser unitairement ; pas exemple dans INSERT ... VALUES (New.nom);

Le fait de requêter la table pour laquelle on fait le trigger n'est pas une bonne pratique.
... surtout sans clause WHERE !

[Garuda]

2 3 4 5 6 7

CREATE OR REPLACE TRIGGER add_pool_contact BEFORE INSERT ON POOL FOR EACH ROW BEGIN insert into CONTACT (nom, prenom,mdp, userdroit) VALUES (:New.nom, 'test','test',1); END;

[Michel.Priori]

Les contraintes déclaratives (null, check, PK, FK) sont évaluées AVANT les triggers.

je répond à mon erreur : https://docs.oracle.com/cd/B19306_01...0/triggers.htm

Oracle uses the following execution model to maintain the proper firing sequence of multiple triggers and constraint checking:
..Run all BEFORE statement triggers that apply to the statement.
..Loop for each row affected by the SQL statement.
......Run all BEFORE row triggers that apply to the statement.
......Lock and change row, and perform integrity constraint checking. (The lock is not released until the transaction is committed.)
......Run all AFTER row triggers that apply to the statement.
..Complete deferred integrity constraint checking.
.. Run all AFTER statement triggers that apply to the statement.

Donc conformément à la réponse de Garuda, il faut utiliser un trigger BEFORE et FOR EACH ROW pour bénéficier des curseurs NEW: et OLD:

Et on est bien d'accord qu'il faille utiliser INSERT ... VALUES en lieu et place de INSERT ... SELECT

NB: Je maintient que requêter la table pour le quel on crée le trigger est une mauvaise pratique.
Vu que les -1 ne permettent pas de justifier la note, je suis ouvert à la conversation même en MP.