这样写存储过程有错吗？该如何处理

这样写存储过程有错吗？
create or replace procedure yyp_cwdh(dhhm varchar) is
begin
 update tb_tel_prediallist set FD_ERRORTEL='1' where FD_PREDIALNO=dhhm;
end yyp_cwdh;


------解决方案--------------------
执行一下不就知道了？

提示 1 ，没有 commit;
------解决方案--------------------
字符型最好用varchar2 ,没看出来参数用varchar的必要性在哪里。
------解决方案--------------------
温馨提示，修改后 需要 commit;