C++编程规范 组织和策略问题 学习 (2) --做代码审查

做代码审查

审查代码：更多的关注有助于提高质量，亮出自己的代码，阅读别人的代码。相互学习，彼此都会受益。

一般修改代码，不论是实现需求还是修改故障，自己自测完成后到了该合入的时候，都是需要做代码审查的。一般都是针对新入职不久的员工或者对整体框架不熟悉的人较多，毕竟大家时间都有限，团队或者小组会指定某些人作为代码审核人，这些代码审核人，肯定是在代码能力、框架掌握都是很好很熟的员工，由他们来把关，降低代码合入风险。

通过能力较强的员工来对代码进行审核这有助于提高代码质量，当然这是需要一种考核机制，你的代码审核多次被退回修改，这虽然看似是一种压力，但是对被审核人和整体团队都是有益而无害的。

自己写的代码有时候思维会有一定的定式，或许你的办法不是最好或者最优，当然肯定是以满足要求为大前提的，毕竟你可能是个新手，给高手看你的代码，他会从最基本的高质量编程的角度来看你的代码有没有问题，比如内存的释放，指针、引用的使用方式等等，接着会从你的代码逻辑去考虑，有没有低级错误，if...else等控制语句加入的地方是否得当，另外还有波及影响，你修改的代码对整个功能或者其他模块功能有没有波及影响，至少不能引入问题。这些看似简单的一个代码合入审核机制，却包含很多实际工作中大家都要面对的问题。

新员工在此基础上，学习和领悟高手的指点，对自己也是一种快速培养和成长的办法，多和高手交流和沟通，对自己的设计能力和实现能力也是一种提高。

这样的工作对整个团队都能增强实力，提升整体软件的安全性，当然这个工作也要做的和谐，让大家不要有负担，太过于形式化和重惩罚制度对员工的心里压力是很大的，心情在高效工作中是很重要的，高手可以一而再再而三的辅助新员工提升代码能力，如若自己多留意几分，自己的工作能力也会势必上去，这个期间一定形成团队荣辱观，为什么别人的代码就质量好，审核通过率高，自己的却总被退回，几次修改，多从自身找找问题，不是他人在刁难，而是你的确做的不够好，不要总被人鄙视，要提高自己在团队中的信誉度。

高效、高质量、低成本的培训方式、增强团队实力，职业荣誉感，自身信誉度，这都与代码审查密不可分。