良好编程习惯很重要

      最近在调整一个查询模块的性能，该模块的前端封装得太不友好，是一次性把所有数据都拿到前台，不管当前有没有用到的，也就是缺少一种延迟加载，不过这样子问题应该也不是很大，毕竟传输的数据不多，而且调整起来也不容易。就接着看后台，后台代码写得更不友好，Action方法就写得比较简洁，从Action到Service，就像从星级酒店到了垃圾堆旁的感觉，让人一种无从适应的感觉，一个信息获取的方法，没什么业务的，负责把数据拿出来就行了，可为什么里面有这么多逻辑呢？接着做进一步分析，发现该信息获取涉及到主从表的信息的获取，先是获取主表的数据，进行遍历，再一次次地获取子表的数据，有两个字表，这完全是一种for循环里面连接数据库的典型反而教材。

 

      问题找到了，应该怎么调整呢？性能调整的目的是够用，而不是最好，如果说调整实现，用ORM的一对多之类的，可以实现目的，考虑到工作量大，而且带来的影响应该不是很大，所以先不考虑这种方法，继续分析该问题。

 

      对数据库模型做分析的时候，发现，两个从表都很简单，仅仅有两个字段，一个是ID，一个是Name，前台显示的时候，要求以“name1, name2...nameN。。。”的形式显示出来，所以就有了前面的for循环嵌套查询。经过样的分析，我发现，这个是设计的问题了，从性能的角度出发，这样做确实不好。有两种方法可以处理：

 

1、不要从表，把ID、Name都保存在主表里面，如(ID1、ID2。。。）和（Name1，Name2。。。）的形式，这样子主表会变大，从表就没有了，避免关联查询。 

2、从表里面保存记录跟主表是一一对应的，保存成(ID1、ID2。。。）和（Name1，Name2。。。）的形式，再通过外健与主表关联。从表还存在，还需要关联查询，并且修改时会涉及到多个表的操作。 

  

      分析两种情况后，我采取了第一种处理方法，很快就调整好了，代码量少了一半，看起来很简洁，性能方面也有了很大的提长。

 

      这个问题造成的影响很大的，一些客户因为该模块的查询速度慢，说“不再用你们系统了”之类的话，回头想想，这个问题在设计时，多想想，就可以避免这种问题的发生。如果这个问题是刚毕业的人写的，还可以原谅，但是却不是。

 

      本文主要是想从一个日常的安全说明良好习惯的重要性。良好习惯的养成，要从一开始就要注意，要刻意要求自己养成程序员该有的良好习惯，多从系统的使用、性能等方面去考虑设计、开发。我带的实习生，我都是要求他们要养成一个良好的编程习惯，也经常检查他们写的代码，发现有不好的就以上提出来，不管是包名称大写这类的小问题还是打开资源不关闭这类大问题，都是不好的。不能因为自己的原因，给后面的人维护、升级带来麻烦，所以要切记，良好编程习惯很重要。