Discussion :

[Flastick]

Bonsoir,

J'ai écrit un déclencheur dans le but de vérifier si un vétérinaire est encore en charge d'un animal avant que le vétérinaire ne soit supprimé.

N'ayant pas le moyen d'exécuter le code et n'étant vraiment pas certain que ma manière de faire soit correcte pour le test, je vous demande votre avis :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
-- Début du code
CREATE OR REPLACE TRIGGER Integrite_personnel BEFORE DELETE ON Personnel
DECLARE
	requete VARCHAR(1000); -- 1000 Pour s'assurer qu'il n'y ait pas d'overflow
	pb_veterinaire EXCEPTION;
BEGIN
	requete = (SELECT animal_id FROM Animal WHERE son_veterinaire_id = :old.personnel_id) ;
	IF NOT requete THEN RAISE pb_veterinaire; --si la requête n'a obtenu aucun résultat
        ELSE animal_id = TO_NUMBER(requete) ; --si oui, on prend l'id de l'unique animal trouvé (animal_id étant une PRIMARY KEY)

J'accepte volontiers toutes les critiques !


Flavien

[Invité]

Votre interprétation du PL/SQL est originale.
Commencez par regarder une documentation : http://sheikyerbouti.developpez.com/pl_sql/
Pour les déclencheurs, c'est ici : http://sheikyerbouti.developpez.com/...?page=Chap6#L6

[Flastick]

Originale ? hem..Je prendrai ça pour un compliment!

Oui je connais ce tutoriel, je l'ai déjà parcouru longuement mais il est si vaste que je m'y suis un peu perdu.
Pourriez-vous me donner une syntaxe-type à utiliser pour effectuer une telle vérification? Ou alors des indices ?

Merci pour votre aide !

Flavien

[Z3phur]

Votre interprétation du PL/SQL est originale.

comme cité ci-dessus tu ne fermes pas ta boucle if, ton trigger...

Pour les déclencheurs, c'est ici : http://sheikyerbouti.developpez.com/...?page=Chap6#L6

tu as un bel exemple de trigger before delete dans ce lien

[Flastick]

Oui on est d'accord ; C'est le début du code seulement, après il vient un
END IF;
EXCEPTION
...
END;

Mais j'ai simplifié car ce que je voulais montrer c'était ma manière de faire un test sur une requète en utilisant une variable 'requete'.

Est-ce que c'est la bonne manière de réaliser le test ?

Je remets seulement les lignes cruciales :

Code :

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

1
2
3
4
5
 
requete VARCHAR(1000);
...
requete = (SELECT animal_id FROM Animal WHERE son_veterinaire_id = :old.personnel_id) ;
IF NOT requete THEN ... --si la requète ne donne aucun résultat

En vous remerciant pour votre patience,

Flavien

[McM]

Code :

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

requete = (SELECT

?? C'est du php que tu codes ?
Un trigger, c'est comme une procédure, d'ailleurs, tu peux créer la procédure et envoyer en paramètre ton :old. depuis le trigger.
Faut vérifier les erreurs de compil.

[Garuda]

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
 
v_animal_id animal.animal_id%TYPE;
BEGIN
....
BEGIN 
SELECT animal_id
INTO v_animal_id
FROM Animal
WHERE son_veterinaire_id = :old.personnel_id;
EXCEPTION
WHEN NO_DATA_FOUND
THEN
RAISE pb_veterinaire;
WHEN TOO_MANY_ROWS
THEN
RAISE pb_veterinaire;
END;
...
END;

[Flastick]

Ahh on fait comme ça !
Parfait ! J'ai tout compris. Merci beaucoup !!

Flavien