oracle sqlplus 写的一个存储过程，看看有什么问题

Question

create or replace procedure InsertPointData 
(
jl_POINTID in varchar2,
jl_POINTNAME in varchar2,
jl_POINTTYPE in varchar2,
jl_VIDEOIP in varchar2,
jl_METERTYPE in varchar2,
jl_status out integer,
jl_hint out varchar2)
is
pointid varchar2(88),
pointname varchar2(30),
pointtype varchar2(10),
videoip varchar2(20),
metertype varchar2(20);
begin
insert into jl_pointinfo
(pointid, pointname,pointtype,videoip,metertype) 
values
(jl_POINTID, jl_POINTNAME,jl_POINTTYPE,jl_VIDEOIP,jl_METERTYPE);
commit;
  jl_status:=0;
  jl_hint:='添加成功';
exception
  when others then
    o_status:=-1;
    o_hint:='添加失败';
end ;

Answer 1

一、这几行变量定义结尾应该全是;，而不是,
二、这个异常里，最好加个rollback吧。

Answer 2

create or replace procedure InsertPointData  
(
jl_POINTID in varchar2,
jl_POINTNAME in varchar2,
jl_POINTTYPE in varchar2,
jl_VIDEOIP in varchar2,
jl_METERTYPE in varchar2,
jl_status out integer,
jl_hint out varchar2)
is
--pointid varchar2(88),
--pointname varchar2(30),
--pointtype varchar2(10),
--videoip varchar2(20),
--metertype varchar2(20);
--这些变量没有用到
begin
insert into jl_pointinfo
(pointid, pointname,pointtype,videoip,metertype)  
values
(jl_POINTID, jl_POINTNAME,jl_POINTTYPE,jl_VIDEOIP,jl_METERTYPE);
commit;
  jl_status:=0;
  jl_hint:='添加成功';
exception
  when others then
  jl_status:=-1;
  jl_hint:='添加失败';--原来变量名写错了
end ;

Answer 3

这里的o_status和o_hint没有定义，应该是想写：jl_status和jl_hint吧。

Answer 4

听你兰兰姐的,异常后记得rollback