如何正确重写删除ARBITER表中4个月前数据的存储过程？

咱们先拆解下你遇到的问题和原存储过程的错误点：

原代码的几个问题： #

• 参数定义错误：你写的procedure RemoveOldData(CREATE_DATE)既没指定参数的数据类型，而且这个参数完全多余——咱们要用到的是表ARBITER里的CREATE_DATE列，根本不需要传入这个参数
• 条件逻辑完全写反：trunc(sysdate) < add_months(trunc(sysdate),-4)这个条件永远不会成立（当前日期不可能小于4个月前的日期），应该判断表中的CREATE_DATE是否早于4个月前的日期
• 语法结构不完整：Oracle存储过程需要完整的定义结构，原代码缺少必要的编译标记，导致解析时触发PLS-00103错误

正确的存储过程代码 #

CREATE OR REPLACE PROCEDURE RemoveOldData
AS
BEGIN
    -- 删除CREATE_DATE早于当前日期4个月前的数据，截断时间部分避免时分秒干扰
    DELETE FROM ARBITER
    WHERE TRUNC(CREATE_DATE) < ADD_MONTHS(TRUNC(SYSDATE), -4);
    
    -- 可选：如果业务需要在存储过程内提交事务，可取消下面注释
    -- COMMIT;
END RemoveOldData;
/

关键修改说明： #

1. 移除多余参数：直接操作表内的CREATE_DATE列，不需要外部传入参数
2. 修正过滤逻辑：把判断条件改成表中记录的创建日期早于4个月前的日期，确保只会删除符合要求的旧数据
3. 完善语法结构：加上CREATE OR REPLACE方便后续重新编译，结尾的/是Oracle环境中用来结束存储过程定义的必要标记
4. 事务控制建议：如果这个存储过程会被其他事务调用，建议不要在内部加COMMIT，由外层业务逻辑统一控制事务提交；如果是独立运行的定时任务，可以加上COMMIT确认删除操作

内容的提问来源于stack exchange，提问作者M.Ofek