从事技术开发的人员，对别人来走查代码可以说是很排斥的，我就是其中之一，代码写的如何跟当时的背景有关系，有时很急，有时需求就是不清不楚，反反复复变更等。
然代码走查是真有必要，不管是新开发模块还是维护变更，不管是大模块还是只改一行代码，都要去走查。
我遇到过一个很小很小的变更，就是加了个条件，结果当时的开发人员自测场景写的不全，更新生产后出现了严重的生产事故。
再者还有一种经历，在排查故障时，发现一行代码不知道为啥变更，查找变更记录无原因，当事人已经离职，怎么办？只能全部捋一遍，代价之大。
可以说代码变更引发的惨案代价是无法形容的，所以代码走查要认真对待，不要走过过场，玩形式主义。
这边分享下常见的需要改进地方:
注释，真的很重要，但最容易被忽视的点，尤其是变更来源，变更原因
空指针，真的很重要，简单，但也是被开发人员想当然以为不会出现，而未加判断
异常捕获，很重要，务必记录到日志文件，关键时刻，可是救命的药，
数组越界，经常出现数组未判断大小，就直接用
分页求总数，直接先获取结果集，然后求结果集大小，内存溢出，毕业生最容易犯这个错误
资源未释放，比如数据库资源，文件io资源，未及时释放，运行一段时间，会来一份大礼资源不足的底层错误，毕业生常犯这样的错误
查询数据时，语句没有参数控制，出现全表返回，乖乖内存溢出。
sql注入，拼接语句查询，职场新人常犯的错误
超时时间默认值，习惯了百度，直接粘贴过来，未设置连接超时，读取超时，结果对方出问题时，自身系统受牵连
