Discussion :

[hel2013]

Bonjour à tous,

Voila j'ai un petit problème avec un trigger :

Ce trigger s'applique sur ma table emprunt composé des champs suivants :
- NUM_ADHERENT : NUMBER
- DATE_EMPRUNT : NUMBER
- DATE_RETOUR : NUMBER

Je souhaite que mon trigger teste avant d'insérer une nouvelle ligne dans la base si le num_adherent n'est pas déjà présent 5 fois (En gros on ne doit pas avoir plus de 5 emprunts dans la base pour un même num_adhérent).

Voila mon trigger :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
create or replace trigger sumExemplaire before insert on EMPRUNT
declare ligne_emprunt EMPRUNT%rowType;
begin
select count(*),NUM_ADHERENT from EMPRUNT where NUM_ADHERENT = (select NUM_ADHERENT into ligne_emprunt from EMPRUNT) > 5;   
 
raise_application_error(-20001,'L adhérent à déjà 5 emprunt en cours');
 
exception
when no_data_found then null;
END;

Le message d'erreur

Code :

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

1
2
3
4
 
 
Warning: exécution terminée avec avertissement
trigger sumExemplaire Compiled.

Merci d'avance

[hel2013]

Bonjour à tous,

Voila j'ai un petit problème avec un trigger :

Ce trigger s'applique sur ma table emprunt composé des champs suivants :
- NUM_ADHERENT : NUMBER
- DATE_EMPRUNT : NUMBER
- DATE_RETOUR : NUMBER

Je souhaite que mon trigger teste avant d'insérer une nouvelle ligne dans la base si le num_adherent n'est pas déjà présent 5 fois (En gros on ne doit pas avoir plus de 5 emprunts dans la base pour un même num_adhérent).

Voila mon trigger :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
create or replace trigger sumExemplaire before insert on EMPRUNT
declare ligne_emprunt EMPRUNT%rowType;
begin
select count(*),NUM_ADHERENT from EMPRUNT where NUM_ADHERENT = (select NUM_ADHERENT into ligne_emprunt from EMPRUNT) > 5;   
 
raise_application_error(-20001,'L adhérent à déjà 5 emprunt en cours');
 
exception
when no_data_found then null;
END;

Le message d'erreur

Code :

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

1
2
3
4
 
 
Warning: exécution terminée avec avertissement
trigger sumExemplaire Compiled.

Merci d'avance

[orafrance]

déjà avec un COUNT tu peux pas avoir de NO_DATA_FOUND donc l'exception est inutile. En plus, sans show err pour voir l'erreur tu risques pas de t'en sortir

Sinon un INTO dans une sous-requête ça risque pas de marcher

[orafrance]

SELECT count(*),NUM_ADHERENT FROM EMPRUNT WHERE NUM_ADHERENT = (SELECT NUM_ADHERENT INTO ligne_emprunt FROM EMPRUNT) > 5;

t'as pas l'impression qu'il y a des opérandes en trop ?

[hel2013]

Merci pour ton aide, j'ai essayé d'une façon différente :

Code :

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

1
2
3
4
5
6
7
8
9
 
create or replace trigger sumExemplaire before insert on EMPRUNT
declare ligne_emprunt EMPRUNT%rowType;
begin
select NUM_ADHERENT from EMPRUNT GROUP BY NUM_ADHERENT HAVING count(*) > 5;   
 
raise_application_error(-20001,'L adhérent à déjà 5 emprunt en cours');
 
END;

Mais ça me met toujours une erreur que bien sûr je ne comprends pas :

Code :

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

1
2
3
 
 
3/1            PLS-00428: une clause INTO est attendue dans cette instruction SELECT

[Mathusalem]

il ne manque pas un IF ?

[mnitu]

Comme d'habitude on commence par les triggers en ignorant le SQL

Code :

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

1
2
3
...
SELECT count(*),NUM_ADHERENT FROM EMPRUNT WHERE NUM_ADHERENT = (SELECT NUM_ADHERENT INTO ligne_emprunt FROM EMPRUNT) > 5;   ...

[McM]

Code :

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

= (SELECT) > 5

pas mal !

Bon, après y'aura forcement l'erreur MUTATING TABLE. Et là, ça va être dur de contourner.

[orafrance]

merci d'éviter de créer 2 discussions sur le même sujet surtout pour un problème de syntaxe qu'une simple relecture du code et une bonne compréhension de l'erreur pourrait résoudre

Mais ça me met toujours une erreur que bien sûr je ne comprends pas :

une clause INTO est attendue dans cette instruction SELECT

Ca te met pas sur la voie ça ? Lis donc la doc et regarde comment INTO est utilisé en PL/SQL

[hel2013]

Merci à tous pour votre aide !

[hel2013]

Pour ceux que ça interesse je poste la solution :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
create or replace trigger sumExemplaire after insert or update on EMPRUNT
declare ligne_emprunt emprunt.NUM_ADHERENT%type;
begin
select NUM_ADHERENT into ligne_emprunt from EMPRUNT GROUP BY NUM_ADHERENT HAVING count(*) > 5;   
raise_application_error(-20902,'L adhérent à déjà 5 emprunt en cours');
exception
when no_data_found then null;
END;

[McM]

Et si 2 adhérents ont plus de 5 lignes, tu sortiras en TOO_MANY_ROWS

[Invité]

Code :

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

raise_application_error(-20902,'L''adhérent a déjà 5 emprunts en cours.');

Corrige le message d'erreur, ça t'évitera l'ouverture d'un bug par le client...

Et une clause WHERE serait sûrement avantageux dans ta requète comme précisé ci-dessus.

[Vld44]

et puis la gestion d'exceptions est inutile puisque dans un cas (when no data found) alors null mais dans les autres le fait que tu ne spécifies rien impliques null aussi ... il te manque in when others then raise ou quelque chose du genre.