最近在调整一个查询模块的性能,该模块的前端封装得太不友好,是一次性把所有数据都拿到前台,不管当前有没有用到的,也就是缺少一种延迟加载,不过这样子问题应该也不是很大,毕竟传输的数据不多,而且调整起来也不容易。就接着看后台,后台代码写得更不友好,Action方法就写得比较简洁,从Action到Service,就像从星级酒店到了垃圾堆旁的感觉,让人一种无从适应的感觉,一个信息获取的方法,没什么业务的,负责把数据拿出来就行了,可为什么里面有这么多逻辑呢?接着做进一步分析,发现该信息获取涉及到主从表的信息的获取,先是获取主表的数据,进行遍历,再一次次地获取子表的数据,有两个字表,这完全是一种for循环里面连接数据库的典型反而教材。
问题找到了,应该怎么调整呢?性能调整的目的是够用,而不是最好,如果说调整实现,用ORM的一对多之类的,可以实现目的,考虑到工作量大,而且带来的影响应该不是很大,所以先不考虑这种方法,继续分析该问题。
对数据库模型做分析的时候,发现,两个从表都很简单,仅仅有两个字段,一个是ID,一个是Name,前台显示的时候,要求以“name1, name2...nameN。。。”的形式显示出来,所以就有了前面的for循环嵌套查询。经过样的分析,我发现,这个是设计的问题了,从性能的角度出发,这样做确实不好。有两种方法可以处理:
1、不要从表,把ID、Name都保存在主表里面,如(ID1、ID2。。。)和(Name1,Name2。。。)的形式,这样子主表会变大,从表就没有了,避免关联查询。 2、从表里面保存记录跟主表是一一对应的,保存成(ID1、ID2。。。)和(Name1,Name2。。。)的形式,再通过外健与主表关联。从表还存在,还需要关联查询,并且修改时会涉及到多个表的操作。
分析两种情况后,我采取了第一种处理方法,很快就调整好了,代码量少了一半,看起来很简洁,性能方面也有了很大的提长。
这个问题造成的影响很大的,一些客户因为该模块的查询速度慢,说“不再用你们系统了”之类的话,回头想想,这个问题在设计时,多想想,就可以避免这种问题的发生。如果这个问题是刚毕业的人写的,还可以原谅,但是却不是。
本文主要是想从一个日常的安全说明良好习惯的重要性。良好习惯的养成,要从一开始就要注意,要刻意要求自己养成程序员该有的良好习惯,多从系统的使用、性能等方面去考虑设计、开发。我带的实习生,我都是要求他们要养成一个良好的编程习惯,也经常检查他们写的代码,发现有不好的就以上提出来,不管是包名称大写这类的小问题还是打开资源不关闭这类大问题,都是不好的。不能因为自己的原因,给后面的人维护、升级带来麻烦,所以要切记,良好编程习惯很重要。