PL / SQL Вставка 1 строки для каждого результата в выборке - PullRequest
Новая
       34

PL / SQL Вставка 1 строки для каждого результата в выборке

17 голосов
/ 29 марта 2012

Я пишу процедуру 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 ... 'NAME_ID' является неверным идентификатором. Я пытался поместить это в кавычки, но без игры в кости. Он также жалуется, что использование переменных индекса цикла 'REC' недопустимо. Любая помощь очень ценится.

1 Ответ

47 голосов
/ 29 марта 2012

Здесь нет необходимости в динамическом 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, вам не следует объединять значения в него, а использовать переменные связывания: 

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;
...