一个简单的sql审核案例

今天开发的同学发来一封邮件，希望我帮忙对一个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;
还有其它更多的改进方法，暂且讨论到这里。