管理感悟:代码审查做哪些事情?

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

 

 

红朝儒生

2016-10-9

 

关键字:代码审查 管理

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

 

 

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

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

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

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

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

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

  

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

你可能感兴趣的:(管理感悟,管理感悟)