一个简单的sql审核案例

  1. 云栖社区>
  2. 博客>
  3. 正文

一个简单的sql审核案例

jeanron100 2016-05-05 23:02:43 浏览386
展开阅读全文
今天开发的同学发来一封邮件,希望我帮忙对一个sql语句做一个评估。他们也着急要用,但是为了稳妥起见,还是希望我来审核一下,这是一个好的习惯。
打开邮件,看到的语句是下面这样的形式。
select a.cout1+b.cout2 from (select count(*) as cout1 from TEST_ONLINE where CN='' and to_char(LOGIN_TIME,'yyyymmdd') = to_char(sysdate,'yyyymmdd') and rownum = 1) a,(select count(*) as cout2 from TEST_USER_CENTER where CN='' and to_char(LAST_LOGOUT,'yyyymmdd') = to_char(sysdate,'yyyymmdd') and rownum = 1) b;
看到这个语句,确实需要审核。
首先从sql语句结构上来说,实在不够好。
如果两个子查询的结果集条数大于1,很可能走笛卡尔积,貌似开发的同学也注意到了这一点,在两个子查询的末尾都加了rownum=1的字样,这样就肯定能够保证语句能够始终有1条以内的记录显示。所以这个语句看起来可以调整的空间不大。
但是我们做sql审核,也离不开表的属性信息。这两个表是OLTP的数据表,里面会有大量的实时数据变化,看看两个子查询中的过滤条件,是根据日期来作为单位统计的,而一个核心字段就是CN了。看到这种情况,如果每日存在大量的数据,使用to_char(LAST_LOGOUT,'yyyymmdd')这种方式肯定是有弊端,但是看需求是想精确到日为单位的数据,那么在这种情况下的关键就是CN了。
带着疑问继续查看,发现CN在两个表中都是主键,那么这种情况就好办多了。对于日期带来的困扰,其实影响不大,而且根据数据的分布,一个CN对应的数据是唯一性的,那么使用rownum=1就有些多余了,然后再来看日期的过滤,有了CN的唯一性约束过滤,数据要么有匹配的是1条,要么就是没有匹配的0条。
结果也是显而易见,明白了这一点,这个时候看起来思路就清晰多了,这个查询的结果应该是在0~2之间。
对于这个语句有了更深入一步的认识,我们就来简单的改造一下。
这样的形式:
select a.cout1+b.cout2 from (select count(*) as cout1 from TEST_ONLINE where CN='xxx' and to_char(LOGIN_TIME,'yyyymmdd') = to_char(sysdate,'yyyymmdd')) a,(select count(*) as cout2 from TEST_USER_CENTER where CN='xxx' and to_char(LAST_LOGOUT,'yyyymmdd') = to_char(sysdate,'yyyymmdd') ) b;
或者:
select  (select count(*) as cout1 from TEST_ONLINE where CN='xxx' and to_char(LOGIN_TIME,'yyyymmdd') = to_char(sysdate,'yyyymmdd') ) +(select count(*) as cout2 from TEST_USER_CENTER where CN='xxx' and to_char(LAST_LOGOUT,'yyyymmdd') = to_char(sysdate,'yyyymmdd') )  from dual;
或者使用with
with
a as (select count(*) as cout1 from TEST_ONLINE where CN='xxx' and to_char(LOGIN_TIME,'yyyymmdd') = to_char(sysdate,'yyyymmdd') and ),
b as (select count(*) as cout2 from TEST_USER_CENTER where CN='xxx' and to_char(LAST_LOGOUT,'yyyymmdd') = to_char(sysdate,'yyyymmdd') )
select a.count1+b.count2 from a,b;
在目前满足条件的情况下,性能差别应该不大。如果CN为非唯一性约束,这个问题还是需要好好斟酌一下了,如果在LOGIN_TIME,LOGOUT_TIME上有索引还是需要避免使用日期的二次格式化,而且在这个基础上,我应该在末尾使用group by而不是rownum=1了。
这样语句可能就变成了下面的形式。
select a.cout1+b.cout2 from (select count(*) as cout1 from TEST_ONLINE where CN='xxx' and LOGIN_TIME between trunc(sysdate) and to_date(sysdate,'yyyy-mm-dd hh24:mi:ss')  group by LOGIN_TIME) a,(select count(*) as cout2 from TEST_USER_CENTER where CN='xxx' and LOGOUT_TIME between trunc(sysdate) and to_date(sysdate,'yyyy-mm-dd hh24:mi:ss')
group by LOGOUT_TIME) b;
还有其它更多的改进方法,暂且讨论到这里。


网友评论

登录后评论
0/500
评论
jeanron100
+ 关注