Discussion :

[randriano]

Bonjour,

Je ne comprends pourquoi il y a erreur sur mon appel de procédure suivant:

Code :

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

1
2
3
4
5
6
7
8
9
declare
begin
USER41.eed_SetGaugeForConnective('EED_OBJ_DEF_ConnectiveEqui');
end;
 
ERROR at line 1:
ORA-00933: SQL command not properly ended
ORA-06512: at "USER41.EED_SETGAUGEFORCONNECTIVE", line 1
ORA-06512: at line 3

Cette procédure existe et est valide mais pourquoi il y a "not properly ended"

ORA-00933
Cause: The SQL statement ends with an inappropriate clause

Ce qui n'est pas mon cas???

[mnitu]

Salut Randriano,

Montre nous le select qui commence à la ligne 3 de ta procédure. Sinon ci dessous la description de l'erreur en question qui semble à avoir disparu à partir d'Oracle 10R2 (R1 ?)

ORA-00933 SQL command not properly ended

Cause: The SQL statement ends with an inappropriate clause. For example, an ORDER BY clause may have been included in a CREATE VIEW or INSERT statement. ORDER BY cannot be used to create an ordered view or to insert in a certain order.

Action: Correct the syntax by removing the inappropriate clauses. It may be possible to duplicate the removed clause with another SQL statement. For example, to order the rows of a view, do so when querying the view and not when creating it. This error can also occur in SQL*Forms applications if a continuation line is indented. Check for indented lines and delete these spaces.

[randriano]

En fait, j'utilise 9i, même si c'est pas récent (2002), c'est fiable

La ligne 3 c'est:

Code :

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

USER41.eed_SetGaugeForConnective('EED_OBJ_DEF_ConnectiveEqui');

Je ne vois d'incorrect là dedans mais ça échoue quand même, c'est pour cela que je pose la question

[Waldar]

Je pense que mnitu faisait référence au code de la procédure USER41.eed_SetGaugeForConnective.

[randriano]

Je pense que mnitu faisait référence au code de la procédure USER41.eed_SetGaugeForConnective.

Le contenu de la procédure!!
Il ne semble que c'est pas dedans le problème, jusqu'à la ligne 20, il n'y a que des déclarations de variables !! Concernant cette procédure, j'utilise 3 curseurs différents, est-ce que c'est peut-être cela le problème? Je répète cette procédure est valide syntaxiquement

[Waldar]

Ah je me demande s'il ne faut pas mettre un / après, que donne :

Code :

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

1
2
3
4
begin
  USER41.eed_SetGaugeForConnective('EED_OBJ_DEF_ConnectiveEqui');
end;
/

[randriano]

Non ce n'est pas, cela causerait: Encountered the symbol "end-of-file" when expecting one of the following

[mnitu]

...Je répète cette procédure est valide syntaxiquement

Est-ce que la procédure utilise du SQL Dynamique (execute immediate ou autre) ?

[randriano]

En effet, elle utilise execute immediate, 10 fois même comme suit

Code :

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

1
2
vQueryUpdate := 'UPDATE ' || v_tablename || ' SET Capacity  = ''' || v_Capacity || ''' WHERE IDEquipment ='''|| v_IDEquipment ||'''';
EXECUTE IMMEDIATE vQueryUpdate;

[mnitu]

Affiche les update via dbms_output au lieur de les exécuter. T'as toujours l'erreur ?

[randriano]

Code :

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

1
2
3
4
5
6
7
8
9
	type cursor_eed is ref cursor;
	cIDEqu cursor_eed;
	cIDEqu2 cursor_eed;
BEGIN
 
    vQuerySelect := 'SELECT DISTINCT EDC.IDEquipment FROM ' || v_tablename || ' AS EDC INNER JOIN TB_FRAME AS ELE ON EDC.IDEquipment = ELE.IDEquipment WHERE EDC.Type <> ''Text''';
 
	OPEN cIDEqu FOR vQuerySelect;
.......................................

Enfin, j'ai identifié la ligne à problème c'est sur ce OPEN cIDEqu!!!
Mais je ne comprends pas pourquoi

[mnitu]

Code :

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

1
2
3
4
5
6
7
8
9
	type cursor_eed is ref cursor;
	cIDEqu cursor_eed;
	cIDEqu2 cursor_eed;
BEGIN
 
    vQuerySelect := 'SELECT DISTINCT EDC.IDEquipment FROM ' || v_tablename || ' AS EDC INNER JOIN TB_FRAME AS ELE ON EDC.IDEquipment = ELE.IDEquipment WHERE EDC.Type <> ''Text''';
 
	OPEN cIDEqu FOR vQuerySelect;
.......................................

Enfin, j'ai identifié la ligne à problème c'est sur ce OPEN cIDEqu!!!
...

C'est toujours du SQL dynamique

Mais je ne comprends pas pourquoi

C'est pourtant simple:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Connected to Oracle9i Enterprise Edition Release 9.2.0.7.0 
Connected as mni
 
SQL> SELECT DISTINCT EDC.ename FROM emp AS EDC INNER JOIN dept AS ELE ON EDC.deptno = ELE.deptno WHERE EDC.dnamer <> 'Text';
 
SELECT DISTINCT EDC.ename FROM emp AS EDC INNER JOIN dept AS ELE ON EDC.deptno = ELE.deptno WHERE EDC.dnamer <> 'Text'
 
ORA-00933: La commande SQL ne se termine pas correctement
 
SQL>

[randriano]

OK

C'est donc le AS qui n'est pas supporté par Oracle!!! Je portais ce code de SQL Server et je n'ai pas vérifié

Merci c'était donc le problème

[Waldar]

Pour aliaser les tables avec Oracle il ne faut pas utiliser "AS" :

Code :

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

1
2
3
4
5
6
7
8
select * from dual d;
 
DUMMY
-----
X
 
select * from dual as d;
ORA-00933: la commande SQL ne se termine pas correctement

[mnitu]

...Je portais ce code de SQL Server et je n'ai pas vérifié
...