1

現在、pl/sqlコードに次のステートメントがあります。

-- vList looks like '1,2,3,4'     
vStatement := 'SELECT NAME FROM T_USER WHERE ID IN ( ' || vList || ' ) '; 
Execute Immediate vStatement BULK COLLECT INTO tNames;

クエリの連結は悪い習慣だと思うので、文字列を使わずにこのクエリを作成したいと思います。これを書き換える方法は何ですか？

PS たぶん、このスタイルが悪いことを証明する十分な理由がないため、クエリの連結が悪い理由を指摘できるかもしれません。

4

2 に答える 2

2

私の推測では、以前に vList id を区切られた文字列にするためにいくつかの手順を実行したと思います (vList がどのように入力されたかはわかりません)。1つのクエリとして保持しないのはなぜですか?

begin
...
select name
bulk collect into tNames
from t_user
where id in (select id from some_table where ...);
...

何度も実行するときのコンテキストの切り替えは面倒ですが、私にとって最悪の部分は、実際には何でもかまいませんが、パラメーター入力を数値のリストとして盲目的に受け入れていることです。(無邪気に) '1,2,X' である可能性があり、実行時エラー「無効な数値」が発生します。さらに悪いことに、SQL インジェクション攻撃である可能性もあります。一般的には悪い習慣です（動的SQLにはその場所があります）が、それをどのように使用しているかは間違いありません。

次のようなことを試してください：

create or replace type t_num_tab as table of number;

create or replace procedure test_proc(i_list in t_num_tab) as
  type t_name_tab is table of varchar2(100);
  l_names t_name_tab;
begin
  -- get names
  select name
  bulk collect into l_names
  from user_table
  where id in (select * from table(i_list));

  -- do something with l_names
  dbms_output.put_line('Name count: ' || l_names.count);

end;

数値のリストよりも複雑なものが必要な場合は、オブジェクト タイプを作成できます。

于 2013-02-15T14:43:16.747 に答える
1

連結が遅いだけではありません。plsql の動的クエリが非常に遅いということです。これを行う方法と理由の両方の良い記事を次に示します。

トムに聞く: 「リスト内」で変数を使用するにはどうすればよいですか?

于 2013-02-15T15:08:31.280 に答える