ORA-00926: missing VALUES key

Version imprimable

Bonjour tout le monde,

Je suis confronté a un petit problème à la compilation d'une procedure avec SQL*Plus.

Voila ci-dessous la procédure avec les deux erreurs :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
SQL> edit
Wrote file afiedt.buf
 
  1  CREATE OR REPLACE PROCEDURE Y_INC_SAS_ENT(NUMCMD IN NUMBER)
  2  AS
  3  BEGIN
  4  INSERT INTO satcd (nuisatcd, etssatcd,clasatcd, numsatcd, snusatcd, fousatcd, mnesatcd, refsatc
  5  devsatcd, depsatcd, refsatcd, devsatcd, depsatcd, dcdsatcd, drdsatcd,lansatcd,etasatcd,
  6  extsatcd,obssatcd, ucrsatcd);
  7  SELECT nuisvcdv, substr(clisvcdv,0,3),'HPML', numsvcdv, snusvcdv, etssvcdv,'BL'||expsvcdv,
  8  clasvcdv||numsvcdv||snusvcdv, devsvcdv, refsvcli, dcdsvcdv, dedsvcdv,'FR','A',nuisvcdv,obssvcdv
  9  FROM svcdv, svcli
 10  where nuisvcdv = NUMCMD and clisvcdv = numsvcli and etssvcli = etssvcdv;
 11* END;
SQL> /
 
Warning: Procedure created with compilation errors.
 
SQL> show error
Errors for PROCEDURE Y_INC_SAS_ENT:
 
LINE/COL ERROR
-------- -----------------------------------------------------------------
4/1      PL/SQL: SQL Statement ignored
6/29     PL/SQL: ORA-00926: missing VALUES keyword
SQL>

Je vous remercie d'avance pour l'aide que vous allez pouvoir m'apporter.

Anis

bonjour,

dégagez le ";" avant le "SELECT" et mettez des parenthèses autour de votre "SELECT".

Et pendant que vous y êtes, utilisez des inner join pour vos jointures :)

Code:

1
2
3
4
5
6
7
8
9
 
INSERT INTO satcd (nuisatcd, etssatcd,clasatcd, numsatcd, snusatcd, fousatcd, mnesatcd, refsatc
devsatcd, depsatcd, refsatcd, devsatcd, depsatcd, dcdsatcd, drdsatcd,lansatcd,etasatcd,
extsatcd,obssatcd, ucrsatcd)
(SELECT nuisvcdv, substr(clisvcdv,0,3),'HPML', numsvcdv, snusvcdv, etssvcdv,'BL'||expsvcdv,
clasvcdv||numsvcdv||snusvcdv, devsvcdv, refsvcli, dcdsvcdv, dedsvcdv,'FR','A',nuisvcdv,obssvcdv
FROM svcdv 
INNER JOIN svcli on clisvcdv = numsvcli and etssvcli = etssvcdv
where nuisvcdv = NUMCMD);

29/04/2011, 14h00
Waldar

Les parenthèses autour du select sont facultatives !
Une bonne mise en page suffit en général.

Utilisez également des alias de tables pour faciliter la relecture.

j'ai bien enlever le ";" et et toujours ne fonctionne pas.
Après plusieurs test j'ai trouver que le nombre des champs dans l' INSERT et plus grand que dans la SELECT

Voila ca fonctionne comme ça!

Code:

1
2
3
4
5
6
7
8
9
10
11
12
CREATE OR REPLACE PROCEDURE Y_INC_SAS_ENT(NUMCMD IN NUMBER)
AS
BEGIN
INSERT INTO satcd( nuisatcd, etssatcd,clasatcd, numsatcd, snusatcd, fousatcd, mnesatcd,
refsatcd,devsatcd, depsatcd, dcdsatcd, drdsatcd,lansatcd,etasatcd,extsatcd,obssatcd, ucrsatcd) 
SELECT nuisvcdv, substr(clisvcdv,0,3),'HPML', numsvcdv, snusvcdv, etssvcdv,'BL'||expsvcdv,
clasvcdv||numsvcdv||snusvcdv, devsvcdv, refsvcli, dcdsvcdv, dedsvcdv,'FR','A',
nuisvcdv,obssvcdv,ucrsvcdv
FROM svcdv, svcli
where nuisvcdv = NUMCMD and clisvcdv = numsvcli and etssvcli = etssvcdv ;
END;
/

Merci.

J'espère juste que ce n'est pas pour un de vos clients, un peu de mise en page ne prend que deux minutes à réaliser et c'est tellement plus lisible et maintenable :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
create or replace procedure y_inc_sas_ent
(
  p$_numcmd in number
)
as
begin
  insert into <schema>.satcd
  (
    nuisatcd, etssatcd, clasatcd, numsatcd, snusatcd, fousatcd,
    mnesatcd, refsatcd, devsatcd, depsatcd, dcdsatcd, drdsatcd,
    lansatcd, etasatcd, extsatcd, obssatcd, ucrsatcd
  ) 
  select ???.nuisvcdv                                 as nuisatcd
       , substr(???.clisvcdv, 1, 3)                   as etssatcd
       , 'HPML'                                       as clasatcd
       , ???.numsvcdv                                 as numsatcd
       , ???.snusvcdv                                 as snusatcd
       , ???.etssvcdv                                 as fousatcd
       , 'BL' || ???.expsvcdv                         as mnesatcd
       , ???.clasvcdv || ???.numsvcdv || ???.snusvcdv as refsatcd
       , ???.devsvcdv                                 as devsatcd
       , ???.refsvcli                                 as depsatcd
       , ???.dcdsvcdv                                 as dcdsatcd
       , ???.dedsvcdv                                 as drdsatcd
       , 'FR'                                         as lansatcd
       , 'A'                                          as etasatcd
       , ???.nuisvcdv                                 as extsatcd
       , ???.obssvcdv                                 as obssatcd
       , ???.ucrsvcdv                                 as ucrsatcd
    from <schema>.svcdv cdv
         inner join <schema>.svcli cli
           on cli.numsvcli = cdv.clisvcdv
          and cli.etssvcli = cdv.etssvcdv 
   where cdv.nuisvcdv = p$_numcmd;
end;
/

bonjour,
j'ai besoin d'ajouter UPDATE OR INSERT si l enregistrement déjà existant je dois faire Update sinon INSERT, a la compilation j'ai l'erreur suivante :
7/3 PL/SQL: SQL Statement ignored
7/10 PL/SQL: ORA-00903: invalid table name
si je supprime "UPDATE OR" la procédure ça compile bien.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
CREATE OR REPLACE procedure y_inc_sas_ent
(
  NUMCMD IN number
)
AS
begin
  UPDATE OR INSERT INTO satcd
  (
    nuisatcd, etssatcd, clasatcd, fousatcd, mnesatcd, 
    refsatcd, devsatcd, depsatcd, dcdsatcd, drdsatcd,
    lansatcd, etasatcd, extsatcd, obssatcd, ucrsatcd
  ) 
 
  SELECT cdv.nuisvcdv                                 AS nuisatcd
       , substr(cdv.clisvcdv, 1, 3)                   AS etssatcd
       , case when movsvlcv = 'RVS' then'HPRM' 
              else 'HPML' end  			      AS clasatcd
       , cdv.etssvcdv                                 AS fousatcd
       , 'BL' || cdv.expsvcdv                         AS mnesatcd
       , cdv.clasvcdv || cdv.numsvcdv || cdv.snusvcdv AS refsatcd
       , cdv.devsvcdv                                 AS devsatcd
       , cli.refsvcli                                 AS depsatcd
       , cdv.dcdsvcdv                                 AS dcdsatcd
       , cdv.dedsvcdv                                 AS drdsatcd
       , 'FR'                                         AS lansatcd
       , 'A'                                          AS etasatcd
       , cdv.nuisvcdv                                 AS extsatcd
       , cdv.obssvcdv                                 AS obssatcd
       , cdv.ucrsvcdv                                 AS ucrsatcd
    FROM svcdv cdv
         INNER JOIN svcli cli
           ON cli.numsvcli = cdv.clisvcdv
          AND cli.etssvcli = cdv.etssvcdv
	 INNER JOIN svlcv svl
	  ON svl.nuisvlcv = cdv.nuisvcdv 
   WHERE cdv.nuisvcdv = NUMCMD;
end;
/

merci de votre aide.

01/06/2011, 10h34
Waldar

UPDATE OR INSERT, ça n'existe pas en SQL.
Par contre, vous avez l'instruction MERGE qui réalise cela.

Une petite recherche sur ce forum vous donnera de nombreux exemples !
01/06/2011, 15h06
orafrance

UPDATE OR INSERT INTO :roll:

On veut bien donner un coup de main mais il y a tout de même un minimum de recherche à faire :?
02/06/2011, 00h34
McM

Citation:

Envoyé par Waldar

J'espère juste que ce n'est pas pour un de vos clients, un peu de mise en page ne prend que deux minutes à réaliser et c'est tellement plus lisible et maintenable :

Tout est une question de goût.. je préfère un code plus compact qu'un insert qui me prend tout l'écran.. et encore dans l'exemple il n'y a que 15 colonnes qu'est ce que ce serait pour une centaine.
02/06/2011, 13h12
Pomalaix

Citation:

Envoyé par McM

Tout est une question de goût.. je préfère un code plus compact qu'un insert qui me prend tout l'écran..

Je crois qu'il est incontestable que l'indentation est une aide considérable à la lecture et à la compréhension de la structure.
Par contre je ne suis pas fervent des retours à la ligne intempestifs (chaque colonne du SELECT sur une ligne, les parenthèses sur une ligne, etc), car ça dilue le code sans le rendre spécialement plus lisible, alors que la "compacité verticale" me facilite beaucoup la vie.

Mais bien entendu, je préfère un code trop dilué qu'écrit au km. Il faut juste avoir un éditeur un peu intelligent, qui permette de replier les portions de code.
Notepad++ est un début dans ce sens, mais pas très au point quand même.
(Faudra qu'on se fasse un sondage sur les outils de ce genre, tiens...)
02/06/2011, 15h38
Invité

Si le code est déjà indenté, c'est déjà fort bien et j'avoue que je n'irais pas imposé ma méthode à tout prix...
Quand je vois le temps que je perds sur du code non indenté, non commenté, non historisé, je place mon combat ailleurs. (Surtout quand c'est ton boss qui l'a produit en mode geek il y a un certain temps !!!)
01/09/2011, 10h14
anis-2005

Probléme de Case when
Bonjour,
Lorsque j'ajoute l'instruction case when dans la procédure elle ne fonctionne plus. merci de m'aider pour trouver une solution pour ajouter des conditions dans la procédure:
si le champs movsvlcv = RVS alors clasatcd = 'HPRM' sinon clasatcd = 'HPML'
Code:

1 2 , case when movsvlcv = 'RVS' then'HPRM' else 'HPML' end AS clasatcd
merci de votre aide.
01/09/2011, 11h52
mnitu

Case n'existe pas avant Oracle 9 (qui est assez vieux en fait :)), utilisee decode à sa place.
02/09/2011, 11h53
StringBuilder

Citation:

Envoyé par McM

Tout est une question de goût.. je préfère un code plus compact qu'un insert qui me prend tout l'écran.. et encore dans l'exemple il n'y a que 15 colonnes qu'est ce que ce serait pour une centaine.

Encore un fan de ce précept :
http://thc.org/root/phun/unmaintain.html

Après, on comprend mieuxp pourquoi y'avait un ";" qui traînait au milieu et pas assez de champs dans le SELECT.

Je suis étonné de ne pas avoir vu un * se balader d'ailleurs :?