我的Oracle匹配函数是否正确？有无更优化的实现方案？

原代码能覆盖大部分场景的正确返回，但存在三个明显缺陷：

1. 逻辑冗余：自定义异常完全没有必要，完全可以直接在分支赋值，异常捕获逻辑额外增加了运行开销
2. 边界场景异常：当service_id_map表为空时，v_result没有被初始化，最终会返回NULL，不符合返回0/1的需求
3. 性能较差：逐行循环遍历表数据匹配，表数据量较大时运行效率很低

推荐直接使用原生SQL的EXISTS判断完成需求，不需要循环和异常处理，性能更高逻辑更稳定：

CREATE OR REPLACE FUNCTION find_option_func(p_balances IN VARCHAR2)
  RETURN NUMBER IS
  v_result NUMBER;
BEGIN
  SELECT CASE
           WHEN EXISTS (
             SELECT 1 
               FROM service_id_map t 
              WHERE INSTR(p_balances, '|' || t.npararam1 || '|') > 0
           ) THEN 1
           ELSE 0
         END
    INTO v_result
    FROM dual;
  RETURN v_result;
END find_option_func;

优化点说明 #

• 去掉冗余的循环和异常逻辑，代码可读性大幅提升
• EXISTS查询匹配到第一条符合条件的记录就会终止检索，比全表循环遍历的效率高得多，数据量越大性能优势越明显
• 无论service_id_map是否有数据，都会稳定返回1或0，不会出现NULL的异常返回值
• 如果你使用的是Oracle 12c及以上版本，可以省略FROM dual的写法，直接在RETURN后写CASE WHEN判断逻辑，代码更简洁

内容的提问来源于stack exchange，提问作者Umid Umaraliev