这个存储过程有什么问题？

Question

create or replace procedure pro_addstu
(p_id in graduate_info.id%type,
p_name in graduate_info.name%type,
p_sex in graduate_info.sex%type,
p_birthday in graduate_info.birthday%type,
p_classno in graduate_info.classno%type,
p_major in graduate_info.major%type,
p_director in graduate_info.director%type,
p_source in graduate_info.source%type,
p_score in graduate_info.score%type,
p_bonus in graduate_info.bonus%type,
v_re out number)
as
v_count number;
v_expt exception;
v_expt1 exception;
cursor exit_cursor is select count(*) from graduate_info where name=p_name;begin
if p_name is null then
    raise v_expt;--学生姓名不能为空
end if;
open exit_cursor;
fetch exit_cursor into v_count;
if v_count>0 then
      raise v_expt1;--学生名称已经存在
 else
     
insert into graduate_info values(id.NEXTVAL,p_name,p_sex,p_birthday,p_classno,p_major,p_director,p_source,p_score,p_bonus);
v_re:=1;--增加成功返回1
     end if; 
close exit_cursor;
exception
     when v_expt then
         v_re:=0;
     when v_expt1 then
         v_re:=-1;
end;)

Answer 1

你select count(*)干吗用个游标啊，直接定义个number select into不就好啦。个人建议，观望中。

Answer 2

游标绝对是多余的，而且这样会影响性能的；
你的过程最后end；后面还有半个括号不知道是什么意思？

Answer 3

id.NEXTVAL中id是一个sequence，如果没有建立的话肯定会报错，具体什么问题都不贴出来，咋个改呢、

Answer 4

1.程序比较奇怪，游标多余，if v_count>0 then 引发异常，但是在异常处理用未关闭游标。
2. name字段建立unique约束，直接野蛮插入数据，成功后返回1，捕获异常时，说明数据已经存在，返回0；

Answer 5

create or replace procedure pro_addstu 
(p_id in graduate_info.id%type, 
p_name in graduate_info.name%type, 
p_sex in graduate_info.sex%type, 
p_birthday in graduate_info.birthday%type, 
p_classno in graduate_info.classno%type, 
p_major in graduate_info.major%type, 
p_director in graduate_info.director%type, 
p_source in graduate_info.source%type, 
p_score in graduate_info.score%type, 
p_bonus in graduate_info.bonus%type, 
v_re out number) 
as 
v_count number; 
v_expt exception; 
v_expt1 exception; cursor exit_cursor is select count(*) from graduate_info where name=p_name; --可以不定义该游标begin if p_name is null then 
    raise v_expt;--学生姓名不能为空 
end if; open exit_cursor; fetch exit_cursor into v_count; 
if v_count>0 then 
      raise v_expt1;--学生名称已经存在 
else 
insert into graduate_info values(id.NEXTVAL,p_name,p_sex,p_birthday,p_classno,p_major,p_director,p_source,p_score,p_bonus); 
v_re:=1;--增加成功返回1 
end if; close exit_cursor; exception 
    when v_expt then 
        v_re:=0; 
    when v_expt1 then 
        v_re:=-1; 
    when others  then --在此处可以捕获一下其他异常信息，并打印出来
    dbms_output.put_line(sqlcode||sqlerrm);
end;
) 
好像在最后多了个右括号。