Discussion :

[jim40]

Bonjour

Je suis en train de batailler sur un truc que je pense être simple mais pas moyen ...

J'ai deux tables.
Une table employé et une table département
Avant de faire l'insertion d'un employé dans la table employé, je dois faire un trigger pour vérifier si le département entré par l'utilisateur est présent dans la table département.

si le département n'existe pas, je dois créer un département qui se nommerait " non attribué"

Comment puis je faire ?

je galère

[Ikebukuro]

Qu'est-ce que tu as déjà fait? A quel endroit, dans ton trigger, tu as des problèmes?
A priori ça a l'air simple.

[jim40]

Pour le moment je galère vraiment
J'ai essayé ceci et je coince.

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
23
CREATE OR REPLACE TRIGGER essai
AFTER 
  INSERT or DELETE ON emp
  for each row
 
DECLARE 
nombre number(2) ;
bool varchar2 ;
 
BEGIN 
 
n := 0 ; 
select count(*) into nombre from emp where exists (select numdep from dpt where numdep = :new.numdep) ;
 
 
  if nombre >0 then 
  bool := 'Ce département est inconnu ! ' ;
  else
  bool := 'ajout enregistré' ;
  end if ;
 
 dbms_output.put_line (bool) ;
END;

[Ikebukuro]

A quoi sert le "or DELETE" de ton trigger? Tu dis que le test ne se fait que pour un INSERT donc je l'enlèverai.

Code :

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

n := 0;

==> tu n'as pas déclaré cette variable n.

Perso j'aurais éclaté en deux ce SELECT, je n'aime pas :

Code :

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

select count(*) into nombre from emp where exists (select numdep from dpt where numdep = :new.numdep) ;

J'aurais mis quelques chose du genre (on est d'accord, numdep est toujours > 0?).

Code :

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

1
2
3
4
5
6
select numdep into v_numdep from dpt where numdep = :new.numdep ;
if v_numdep >0 then
bool := 'Ce département est inconnu ! ' ;
else
bool := 'ajout enregistré' ;
end if ;

avec v_numdep une variable de type NUMBER.
Quand tu compiles, tu as un message d'erreur?

[tbc92]

Dans la 2ème ligne du trigger, tu as le mot clé AFTER insert. Et dans le titre du message, tu disais "trigger vérifier AVANT insertion".

[Ikebukuro]

Dans la 2ème ligne du trigger, tu as le mot clé AFTER insert. Et dans le titre du message, tu disais "trigger vérifier AVANT insertion".

Exact, j'avais pas vu...

[jim40]

A quoi sert le "or DELETE" de ton trigger? Tu dis que le test ne se fait que pour un INSERT donc je l'enlèverai.

Code :

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

n := 0;

==> tu n'as pas déclaré cette variable n.

Perso j'aurais éclaté en deux ce SELECT, je n'aime pas :

Code :

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

select count(*) into nombre from emp where exists (select numdep from dpt where numdep = :new.numdep) ;

J'aurais mis quelques chose du genre (on est d'accord, numdep est toujours > 0?).

Code :

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

1
2
3
4
5
6
select numdep into v_numdep from dpt where numdep = :new.numdep ;
if v_numdep >0 then
bool := 'Ce département est inconnu ! ' ;
else
bool := 'ajout enregistré' ;
end if ;

avec v_numdep une variable de type NUMBER.
Quand tu compiles, tu as un message d'erreur?

Ça fonctionne. Il me reste à créer le nouveau département. Merci beaucoup