IT咨询顾问:group by与join引发的项目救火
我又一次进行了项目救火,这次的原因是group by与join胡乱的堆彻导致的整个业务系统审核流程发生严重的错误。基础的sql表关联,group by,子表都理不清,我也只能对你面带微笑,不想对你解析原因,你就按照我提供给你的模板改你全部的业务sql层(XML文件的sql)吧。
很简单的一对多表关联
用户表,customerId代表用户的唯一id,insertTime代表这条数据何时存入的。与之对应的是表单表,外键customerId表明这个保单是哪个客户的,同时unit表明出该保单的机构,insertTime也是插入时间。
业务背景
前端列表显示出每一天投保的客户信息(客户连续两天投保,则显示两条该客户数据,如上面intsmaze客户),点击每条记录的详情可以查看该客户当天的保单详情。
审核权限划分:一个客户一天投保的多个保单可能会来自多个出单机构,比如上面intsmaze,06-21的三个保单出自上海,北京,深圳,那么怎么划分,借助group by的”随缘法则”,intsmaze 06-21号就取数据插入顺序最前的上海,特朗普 06-21 就取杭州。
他们的原产sql
SELECT * from customer c LEFT JOIN insurance i on c.customerId=i.customerId and c.insertTime=i.insertTime where c.flow='0' GROUP BY c.customerId
查询的结果是两条数据,很显然少了一条intsmaze 06-22号的
虽然有问题,但是感觉很难爆出给用户的,它是怎么出现的了?
这要说审核流程了,默认数据进来是初审,用户表的flow是0。
当用户审核intsmaze的这一条数据后,数据变得如下
我们可以看到flow=‘0‘初审,intsmaze的unit由上海变成了北京,那是因为这条问题sql隐藏的数据终于出现了。
此时flow=‘1‘复审的数据如下
然后有趣的事情来了,审核人员此时又将初审的intsmaze 北京 提交到复审,这个时候复审应该有两条数据,但是他到复审那里还是就看到一条数据。然后就出大事了,最后我就马革裹尸过来救火了。
解决方案
join的时候是几个字段,group by就几个字段,加上insertTime即可
SELECT * from customer c LEFT JOIN insurance i on c.customerId=i.customerId and c.insertTime=i.insertTime GROUP BY c.customerId,c.insertTime
unit判断,导致同一条数据两个机构均可审核
他的unit的判断放在join后的where条件上
SELECT * from customer c LEFT JOIN insurance i on c.customerId=i.customerId and c.insertTime=i.insertTime where c.flow='1' and i.unit='北京' GROUP BY c.customerId
这导致一个有趣的问题就是,比如intsmaze 06-21 这个客户,它分别能被北京,上海,深圳三个机构看到,其他机构是看不到。然后就出现一个有趣的现象:”谁动了我的数据,我明明没有审核,为什么到复审了,北京复审页面看到这条数据初审提交人事上海,这是怎么一回事嘛?”。
这个问题我只显示结果,不想解释,最后附上解决方案。
SELECT * from customer c LEFT JOIN insurance i on c.customerId=i.customerId and c.insertTime=i.insertTime where c.flow='1' GROUP BY c.customerId
这个客户本来是该上海机构去审核
可是我发现,如果北京机构人登录,也是可以看到这条记录进行审核
SELECT * from customer c LEFT JOIN insurance i on c.customerId=i.customerId and c.insertTime=i.insertTime where c.flow='1' and i.unit='北京' GROUP BY c.customerId
如何解决,子表呗
SELECT * from ( SELECT c.customerId,c.insertTime,unit,money from customer c LEFT JOIN insurance i on c.customerId=i.customerId and c.insertTime=i.insertTime where c.flow='1' GROUP BY c.customerId )temp where temp.unit='北京'
最终我提供的sql模板如下:
SELECT * from ( SELECT c.customerId,c.insertTime,unit,money from customer c LEFT JOIN insurance i on c.customerId=i.customerId and c.insertTime=i.insertTime where c.flow='1' GROUP BY c.customerId,c.insertTime )temp where temp.unit='北京'
模板提供后,剩下的事情当然不是我去改了,毕竟我已经吐了几口血了。我只负责找出问题,提供解决方案,然后功能顾问就全方位的改自己的sql。我则一旁陪伴直至改完后,系统没有出现他们无法解决的毛病后就离场修身养性咯。
ps:救了几次火后,我晓得当初为什么我被面试,别人问我你开发中遇到什么奇怪的bug没,我当时真的没有啊,我确实没有遇到什么奇怪的bug,因为你如果真的按照语法规则开发,其实很多问题都是不会出现的。之所以会出现,往往是基础太薄弱,然后也不理解上来就模仿,然后就会出错。