Discussion :

[aminej76]

Bonjour ,

Merci de me guider afin d'optimiser ce PL/SQL :

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
24
create or replace TRIGGER FACTUREF_ANNEE_NUMERO
BEFORE INSERT ON FACTUREF for each row
 
DECLARE
    NEW_CLE_FAC VARCHAR2(8);
    NB NUMERIC;
    CPT NUMERIC;
 
BEGIN
 
SELECT count(CLE_FAC) INTO CPT FROM FACTUREF WHERE ANNEE_FAC = :new.ANNEE_FAC ;
 
if CPT > 0 then
 
     SELECT MAX(CLE_FAC) INTO NEW_CLE_FAC FROM FACTUREF WHERE ANNEE_FAC = :new.ANNEE_FAC ;
 
     NEW_CLE_FAC := SUBSTR(NEW_CLE_FAC, 3, 8);
     NB := cast(NEW_CLE_FAC as numeric ) ;
     NB := NB + 1;
      NEW_CLE_FAC := cast(nb as varchar2);
     :new.CLE_FAC := concat(:new.ANNEE_FAC , LPAD(NEW_CLE_FAC, 6, '0')); 
 ELSE
    :new.CLE_FAC := concat(:new.ANNEE_FAC , '000001' ) ; 
END IF;

[Michel.Priori]

Bonjour,

Quel est le type d'optimisation qui est recherché ?
Pour moi ce code est fonctionnellement faux, et ce, en dehors du fait qu'il n'est pas complet.

[McM]

Bonjour,

Merci d'éditer ton message pour encadrer ton code des balises CODE (le # dans le menu d'édition)

Normalement on pourrait simplifier comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
CREATE OR REPLACE TRIGGER FACTUREF_ANNEE_NUMERO
BEFORE INSERT ON FACTUREF FOR EACH ROW
DECLARE
  v_max VARCHAR2(8);
BEGIN
	SELECT :NEW.ANNEE_FAC || LPAD(NVL(TO_NUMBER(MAX(SUBSTR(CLE_FAC,3))), 0) + 1, 6, '0') INTO :NEW.CLE_FAC 
  FROM FACTUREF 
  WHERE ANNEE_FAC = :NEW.ANNEE_FAC ;
END IF;

Sauf que dans un trigger on ne peut pas faire de SELECT sur la table du trigger.
Le code doit être déporté dans le programme faisant l'insert

[aminej76]

Merci infiniment.
Je suis un débutant avec oracle.

Bon dév.

[pachot]

Attention, si le but est d'incrémenter un numéro de facture ce code est complètement faux. Il va amener des doublons (lorsqu'il y aura un autre insert en même temps) et des trous (lorsqu'il y aura une transaction rollbackée).