压缩AND子句Oracle sql

Question

我在我们的一个应用程序的一个 sql 脚本中遇到了这个问题。我注意到它在其他各个地方都有使用，但它不只是检查项目的存在吗？

AND INSTR((SELECT (',' || REPLACE('OWN, JO', ' ', NULL) || ',') b FROM DUAL),
         (',' || aao.AcctRoleCd || ',')) > 0

它在哪里寻找'OWN' 或'JO' 是否在aao.AcctRoleCd 中。如果是，那么INSTR 将导致它在字符串中的索引，所以它会大于一。所以AND 子句将是true。

这样检查一个项目是否存在不是很糟糕吗？ IN 子句的更多内容会更好吗？

AND aao.AcctRoleCd IN ('OWN', 'JO');

Answer 1

差不多：

• 'OWN, JO' 是文本字面量。
• REPLACE('OWN, JO', ' ', NULL) 只是从字符串中去掉空格，给出'OWN,JO'。
• ',' || 'OWN,JO' || ',' 只是将逗号连接到字符串的开头和结尾，给出',OWN,JO,'。
• (SELECT ',OWN,JO,') b FROM DUAL) 是多余的，你可以使用前面的文本文字。
• INSTR( ',OWN,JO,', (',' || aao.AcctRoleCd || ',') ) > 0 正在寻找等于 aao.AcctRoleCd 的子字符串开头和结尾的逗号，因此可以匹配 'OWN'、'JO' 或 'OWN,JO'。

所以你可以将其替换为：

AND aao.AcctRoleCd IN ( 'OWN', 'JO', 'OWN,JO' )

现在，'OWN,JO' 可能不是您期望的匹配项（或者甚至可能不是有效值），您可以将其从列表中删除，但这是您需要确定的。

Answer 2

您是对的，但感谢您分享这么糟糕的代码。

(SELECT (',' || REPLACE('OWN, JO', ' ', NULL) || ',') b FROM DUAL)

是一个完全不必要的标量子查询。可以换成

',' || REPLACE('OWN, JO', ' ', NULL) || ','

但是，由于 sn-p 只有文字，因此可以进一步替换为结果：

,OWN,JO,

是的，似乎整个 INSTR 可以用您建议的代码替换，除非 aao.AcctRoleCd 可能包含“N，J”或类似的内容，在这种情况下，原始代码和您的代码会得到不同的结果。我严重怀疑这是个问题。

最好的问候，炖菜