管理感悟：代码审查做哪些事情？

管理感悟：代码审查做什么

　

　

红朝儒生

2016-10-9

　

关键字：代码审查 管理

简介：代码审查，不审查方案流程，而是代码对齐、注释、模块化、依赖关系。

　

　

　　代码审查，是提高工作质量的必经之路。那么，代码审查做什么？审查方案流程？陷入代码大海里，汝要花多少时间，才能搞明白方案、流程？汝智商比作者高多少，能看出门道？都已经代码审查了，还要检查方案流程，那要人命了，项目前期高手都哪里去也？这代码审查怎么执行下去？

　　所以，代码审查，做的不是方案流程的大事，而是代码方面的小事。以吾工作体会，应该包括以下几方面的工作：

　　代码对齐。这指的代码格式方面。以吾体验，做到代码对齐是第一位的，特别是相邻的相似代码语句，必须对齐！对齐了看起来又整齐又漂亮，阅读的时候只关心不同点，等于节省了大量工作。

　　说明注释。各公司管理人员天天鼓噪要多加注释，实际上都是起哄。注释的代码要清理；由谁修改的这种垃圾注释，对于工作没有任何用处，不能要；只有解释为什么要这么做的注释，才是好注释。具体参考《管理感悟：需要什么样的注释》。

　　模块化。什么叫模块化？就是访问关系局部化，比如类、函数、字段，能私有的都不要公开。也叫“内聚性”，完成同一功能（不同层次）的类放在一起。

　　依赖关系。大多数程序员很随意，不经过思考，就直接调用别的类。这样就会多了很多不必要的依赖，之间关系非常复杂。实际上依赖关系跟模块化有很大关系。此外要注意的是，一些类似服务的功能，如常量、工具包、数据库、标准组件等等，严禁对外依赖。

　　

　　函数、变量命名？这听起来很简单，实际上呢？要做到命名的用词准确，要一定天赋。这一点，专门搞外语的都不一定行。比如吾自认没有外语天赋，但是在命名的用词上，经常能挑出别人的错误。