Oracle动态SQL更新表存储过程报错（PLS-00103）求助

你的存储过程触发PLS-00103错误，是因为存在多处语法和逻辑问题，逐一排查如下：

错误点分析 #

• 命名冲突：存储过程的输入参数newjob和IS块内定义的局部变量newjob同名，PL/SQL不允许参数与局部变量重名，会导致编译器混淆。
• 字符串拼接语法错误：构建动态SQL时，query := 'update Tbl_Job_Desc set Jobname='||newjob' ;存在两处问题：
  1. 拼接字符串时缺少连接符||，末尾的单引号位置错误，正确的拼接需要用||连接所有片段。
  2. 字符串类型的newjob值没有用单引号包裹，生成的SQL会把输入值当成列名而非字符串常量，执行时会触发语法错误。
• 存储过程调用名称不匹配：定义的存储过程名为Ds_sampleTest，但调用时写的是Ds_sample，名称不一致会导致找不到对应的存储过程。
• SQL注入风险：直接拼接字符串生成动态SQL存在SQL注入隐患，应该使用绑定变量替代直接拼接。

修正后的存储过程代码 #

CREATE OR replace PROCEDURE Ds_sampleTest (
    newjob  IN VARCHAR2
)
IS
  query  VARCHAR2(1000);
BEGIN
    -- 使用绑定变量构建动态SQL，避免注入风险
    query := 'update Tbl_Job_Desc set Jobname = :1';
    
    dbms_output.Put_line(query);
    -- 执行动态SQL并传入绑定变量值
    EXECUTE IMMEDIATE query USING newjob;
END;
/

-- 正确调用存储过程
SET serveroutput ON
BEGIN
    Ds_sampleTest('doctor');
    COMMIT;
END;
/

额外说明 #

如果一定要用字符串拼接（不推荐），需要给newjob值添加单引号包裹，修正后的拼接语句如下：

query := 'update Tbl_Job_Desc set Jobname='''||newjob||''';';

注意这里用两个单引号表示SQL中的一个单引号，避免语法错误。但这种方式存在SQL注入风险，优先推荐绑定变量的写法。

内容的提问来源于stack exchange，提问作者secretgirl2023