plsql - PL/SQL 在选择中为每个结果插入 1 行

Question

我正在编写一个 PL/SQL 过程，它根据输入变量执行选择，然后为选择中的每个结果插入一行。由于我对 PL/SQL 的陌生，我无法调试我的查询出了什么问题。我知道这一定很容易，但由于某种原因我被困在这里。谢谢你的帮助!

CREATE OR REPLACE PROCEDURE setup_name_map(ranking_id IN NUMBER, class_string IN VARCHAR2) 
IS
BEGIN

    FOR rec IN (SELECT NAME_ID FROM PRODUCT_NAMES WHERE NAME = class_string)
    LOOP
        EXECUTE IMMEDIATE 'INSERT INTO NAME_RANKING (NAME_ID, RANKING_ID) VALUES (' || rec.NAME_ID || ', ' || ranking_id || ')';
    END LOOP;
END;

根据 Oracle Developer Compiler...'NAME_ID' 是无效标识符。我试过把它放在引号中，但没有骰子。它还提示循环索引变量“REC”的使用无效。任何帮助深表感谢。

Answer 1

这里不需要动态 SQL:

BEGIN

    FOR rec IN (SELECT NAME_ID FROM PRODUCT_NAMES
                WHERE NAME = class_string)
    LOOP
        INSERT INTO NAME_RANKING (NAME_ID, RANKING_ID)
        VALUES (rec.NAME_ID, ranking_id);
    END LOOP;
END;

更好的是，您可以避免像这样缓慢的逐行游标方法:

BEGIN
    INSERT INTO NAME_RANKING (NAME_ID, RANKING_ID)
    SELECT NAME_ID, ranking_id FROM PRODUCT_NAMES
    WHERE NAME = class_string;
END;

如果您确实需要动态 SQL，则不应将值连接到其中，而应使用绑定(bind)变量:

BEGIN

    FOR rec IN (SELECT NAME_ID FROM PRODUCT_NAMES
                WHERE NAME = class_string)
    LOOP
        EXECUTE IMMEDIATE 'INSERT INTO NAME_RANKING 
                           (NAME_ID, RANKING_ID) VALUES (:b1, :b2)
        USING rec.NAME_ID, ranking_id;
    END LOOP;
END;