You can change your code to do:
v_lstmt := 'SELECT count(*) FROM userB.tableB WHERE id = '''||v_ret (i).id||''''
|| ' and ('||v_ret (i).col||' is null or '||v_ret (i).col||' = :val)';
EXECUTE IMMEDIATE v_lstmt INTO cDel using v_ret (i).val;
That checks that the column is null or matches the supplied val
, and uses a bind variable to supply the value to check to cut down parsing a bit.
However this still relies on implicit conversion, so if you had a date value in the table for instance you'd be relying on your NLS settings to convert it to match the target table column type.
You can use the all_tab_columns
view to find the data type of the target column and do explicit conversion of the val
to that type before binding. A more involved but possibly more robust approach would be to use dbms_sql
for the inner dynamic SQL instead of execute immediate
.
The outer query doesn't seem to need to be dynamic though, you coudl do:
declare
v_lstmt VARCHAR2(32000);
cDel number;
begin
for rec in (SELECT id, col, val FROM tableA) loop
v_lstmt := 'SELECT count(*) FROM tableB WHERE id = '''||rec.id||''''
|| ' and ('||rec.col||' is null or '||rec.col||' = :val)';
dbms_output.put_line(v_lstmt);
EXECUTE IMMEDIATE v_lstmt INTO cDel using rec.val;
If cDel > 0 Then
--some code
cDel := 0;
end if;
end loop;
end;
/
与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…