Oracle：リラックスできる条件付きのPLSQLクエリ - 実現するためのエレガントな方法

Question

次のクエリを見てください。どのように私はそれをより読みやすい方法で書くことができますか？ 私はまあ、私は常に、バインド変数を使用するコードではなく、値の連結読みやすくを見つける（そして、それははるかに良い習慣です）IF

vSQL := ' SELECT loc.RMV_REMI_VIRTUALE_COD, loc.LOC_DATA_VER_FIN FROM ENI_SAG_TSF_LOCALITA_DEF loc WHERE 1=1 '; 
    IF P_LOC_LOCALITA_COD IS NOT NULL THEN 
    vSQL := vSQL||' AND LOC.LOC_LOCALITA_COD = '''||P_LOC_LOCALITA_COD||''' '; 
    END IF; 
    IF P_ISTAT_CITTA IS NOT NULL THEN 
    vSQL := vSQL||' AND loc.COMB_ISTAT_COD = '''||P_ISTAT_CITTA||''' '; 
    END IF; 
    IF P_PLAY_PLAYER_COD IS NOT NULL THEN 
    vSQL := vSQL||' AND LOC.PLAY_PLAYER_COD = '''||P_PLAY_PLAYER_COD||''' '; 
    END IF; 
    IF P_LOC_DATA IS NOT NULL THEN 
    vSQL := vSQL||' AND TO_TIMESTAMP ('''||P_LOC_DATA||''' , ''DD/MM/YYYY HH24:MI:SS.FF3'') BETWEEN LOC.LOC_DATA_VER_INI AND LOC.LOC_DATA_VER_FIN '; 
    END IF; 

Answer 1

あなたの答えを読むこの製法が見つかりました。あなたはそれについてどう思いますか？

SELECT loc.RMV_REMI_VIRTUALE_COD, loc.LOC_DATA_VER_FIN 
    FROM ENI_SAG_TSF_LOCALITA_DEF loc 
WHERE 1 = 1 
     AND (LOC.LOC_LOCALITA_COD = :P_LOC_LOCALITA_COD OR :P_LOC_LOCALITA_COD IS NULL) 
     AND (loc.COMB_ISTAT_COD = :P_ISTAT_CITTA OR :P_ISTAT_CITTA IS NULL) 
     AND (LOC.PLAY_PLAYER_COD = :P_PLAY_PLAYER_COD OR LOC.PLAY_PLAYER_COD IS NULL) 
     AND (TO_TIMESTAMP (:P_LOC_DATA, 'DD/MM/YYYY HH24:MI:SS.FF3') BETWEEN LOC.LOC_DATA_VER_INI AND LOC.LOC_DATA_VER_FIN OR :P_LOC_DATA IS NULL) 

Answer 2

の使用量を削減したいと思います。あなたは、SQLを実行する方法を言うことはありませんが、REF CURSORと仮定すると、あなたはこのようにそれを行うことができます：

vSQL := ' SELECT loc.RMV_REMI_VIRTUALE_COD, loc.LOC_DATA_VER_FIN FROM ENI_SAG_TSF_LOCALITA_DEF loc WHERE 1=1 '; 
    IF P_LOC_LOCALITA_COD IS NOT NULL THEN 
    vSQL := vSQL||' AND LOC.LOC_LOCALITA_COD = :P_LOC_LOCALITA_COD '; 
    ELSE 
    vSQL := vSQL||' AND (1=1 OR :P_LOC_LOCALITA_COD IS NULL)'; 
    END IF; 
    IF P_ISTAT_CITTA IS NOT NULL THEN 
    vSQL := vSQL||' AND loc.COMB_ISTAT_COD = :P_ISTAT_CITTA '; 
    ELSE 
    vSQL := vSQL||' AND (1=1 OR :P_ISTAT_CITTA IS NULL)'; 
    END IF; 
    IF P_PLAY_PLAYER_COD IS NOT NULL THEN 
    vSQL := vSQL||' AND LOC.PLAY_PLAYER_COD = :P_PLAY_PLAYER_COD '; 
    ELSE 
    vSQL := vSQL||' AND (1=1 OR :P_PLAY_PLAYER_COD IS NULL)'; 
    END IF; 
    IF P_LOC_DATA IS NOT NULL THEN 
    vSQL := vSQL||' AND TO_TIMESTAMP (:P_LOC_DATA, ''DD/MM/YYYY HH24:MI:SS.FF3'') BETWEEN LOC.LOC_DATA_VER_INI AND LOC.LOC_DATA_VER_FIN '; 
    ELSE 
    vSQL := vSQL||' AND (1=1 OR :P_LOC_DATA IS NULL)'; 
    END IF; 

    OPEN refcur FOR vSQL USING P_LOC_LOCALITA_COD, P_ISTAT_CITTA, P_PLAY_PLAYER_COD, P_LOC_DATA; 

文のバインド変数の数が持っている固有の動的SQLのためので、私はELSE句を追加しました固定する。 DBMS_SQLパッケージを使用する場合は、その必要はありません。

vSQL := ' SELECT loc.RMV_REMI_VIRTUALE_COD, loc.LOC_DATA_VER_FIN FROM ENI_SAG_TSF_LOCALITA_DEF loc WHERE 1=1 '; 
    || CASE WHEN P_LOC_LOCALITA_COD IS NOT NULL THEN 
       ' AND LOC.LOC_LOCALITA_COD = :P_LOC_LOCALITA_COD ' 
      ELSE 
       ' AND (1=1 OR :P_LOC_LOCALITA_COD IS NULL)' 
      END 
    || CASE WHEN P_ISTAT_CITTA IS NOT NULL THEN 
       ' AND loc.COMB_ISTAT_COD = :P_ISTAT_CITTA ' 
      ELSE 
       ' AND (1=1 OR :P_ISTAT_CITTA IS NULL)' 
      END 
... etc. 

はもちろん、あなたが今、代わりにケースを持っていますが、少なくとも、あなたはすべての vSQL := vSQL ||ビットを失う：

のIFを避けるためとして、あなたはこれを行うことができます。

あなたはあなたのような機能のロジックをラップすることができ、同様の条件の多くを追加する場合：

vSQL := ' SELECT loc.RMV_REMI_VIRTUALE_COD, loc.LOC_DATA_VER_FIN FROM ENI_SAG_TSF_LOCALITA_DEF loc WHERE 1=1 '; 
    || and_condition ('LOC.LOC_LOCALITA_COD', 'BV1', P_LOC_LOCALITA_COD) 
    || and_condition ('loc.COMB_ISTAT_COD', 'BV2', P_ISTAT_CITTA) 
    ... etc. 

（それは当然のBETWEEN条件では動作しません）。あなたはIF THENブロックである好きではない場合

Answer 3

「より読みは」少し主観が、オプションのカップルです：

vSQL := ' SELECT loc.RMV_REMI_VIRTUALE_COD, loc.LOC_DATA_VER_FIN FROM ENI_SAG_TSF_LOCALITA_DEF loc WHERE 1=1 '; 

vSQL := vSQL|| 
     NVL2(P_LOC_LOCALITA_COD, 
      ' AND LOC.LOC_LOCALITA_COD = '''||P_LOC_LOCALITA_COD||''' ', 
       NULL); 

vSQL := vSQL|| 
     NVL2(P_ISTAT_CITTA, 
       ' AND loc.COMB_ISTAT_COD = '''||P_ISTAT_CITTA||''' ', 
       NULL); 

vSQL := vSQL|| 
     NVL2(P_PLAY_PLAYER_COD, 
       ' AND LOC.PLAY_PLAYER_COD = '''||P_PLAY_PLAYER_COD||''' ', 
       NULL); 

vSQL := vSQL|| 
     NVL2(P_LOC_DATA, 
       ' AND TO_TIMESTAMP ('''||P_LOC_DATA||''' , ''DD/MM/YYYY HH24:MI:SS.FF3'') BETWEEN LOC.LOC_DATA_VER_INI AND LOC.LOC_DATA_VER_FIN ', 
       NULL); 

または

vSQL := ' SELECT loc.RMV_REMI_VIRTUALE_COD, loc.LOC_DATA_VER_FIN FROM ENI_SAG_TSF_LOCALITA_DEF loc WHERE 1=1 ' 
     ||NVL2(P_LOC_LOCALITA_COD, 
       ' AND LOC.LOC_LOCALITA_COD = '''||P_LOC_LOCALITA_COD||''' ', 
       NULL) 
     ||NVL2(P_ISTAT_CITTA, 
       ' AND loc.COMB_ISTAT_COD = '''||P_ISTAT_CITTA||''' ', 
       NULL) 
     ||NVL2(P_PLAY_PLAYER_COD, 
       ' AND LOC.PLAY_PLAYER_COD = '''||P_PLAY_PLAYER_COD||''' ', 
       NULL) 
     ||NVL2(P_LOC_DATA, 
       ' AND TO_TIMESTAMP ('''||P_LOC_DATA||''' , ''DD/MM/YYYY HH24:MI:SS.FF3'') BETWEEN LOC.LOC_DATA_VER_INI AND LOC.LOC_DATA_VER_FIN ', 
       NULL); 

はそれが役に立てば幸い...