软件开发实践:如何编写整洁的代码

最近在工做中, 常常会维护一些已经存在的代码,常常Review别人的代码,也常常请别人Review代码.

感受Review代码真是一个很累人的工做.感谢那些为咱们Review代码的同事.

不少时候,感受Review的效果并很差,难以深刻下去.

如何Review好代码,当前尚未太好的思路,结对编程是一个方法,可是本身没有实践过,仅仅在思考的层面上,因此也难以有发言权.

可是,如何方便别人Review,如何提升本身的代码质量,却是总结了一些内容,本我大打算主要站在C/C++开发人员的角度写一下本身的总结.

有本书叫<代码整洁之道>, 主要就是讲述怎么写整洁代码的.一直以来,感受本身写代码还算是整洁,可是看了这本书后,仍是颇有收获.

别人的东西毕竟是别人的,本身理解的不必定到位,在这里我把本身理解的整洁的代码整理一下.想到哪儿写到哪儿,条目未必都在一个层次上,有点乱,也不全,之后会持续整理:

1控制可见范围

我以为代码中,控制可见范围最重要.

l  能放在源文件中的不要放在头文件中

由于头文件的可见范围比源文件大,可能被别处使用.

能被别处使用的代码，之后就不敢轻易修改, 会使代码愈来愈僵化.

看代码时, 增长阅读代码负担, 会关心是否被其余地方使用.追溯代码时，扩大追溯范围.

头文件中信息太多, 会使头文件更复杂.

l  能用static的，必定要用static

在源文件中,static的函数或者变量,咱们能够随便改,由于这些变量或函数的范围只在不会超出文件.可是非static的,要改动时就要当心了. 阅读代码时,也会考虑它为何不是static的,会不会被其余地方调用.

l  在类中,能用private的不用protected,能用protected的,不用public.

2 头文件

l  抽象层次不一样的代码不要放在一块儿，特别是头文件

不这样作，在使用 写makefile时， 可能须要include一大堆不相干的文件夹.甚至可能出现类型冲突.

l  常量，枚举若是可能，最好和结构体分离，特别是与具体业务联系密切的结构体， 最好对结构体也划分好层级

若是用于ioctl之类的头文件，最好不要出现函数

l  能不include的头文件不要引用

l  若是是库， 对外接口能写成C接口的，尽可能写成C接口，而不是C++接口.

3 变量

l  不要对外暴露变量，而是接口

变量未通过抽象，描述性差.

变量暴露了细节. 例若有些类型能够是模块私有的，若是暴露变量，则该类型可能须要设置为外部可见的

l  变量必定要初始化

l  尽可能以只读常量代替宏

4 函数

l  不要有长函数

长函数有不少的不方便:

1难以阅读

2 不知道逻辑段长短

有些逻辑段特别长

3 if/else一大堆

4 有些变量声明的地方距离使用的位置比较远

特别是C代码

5 缩进太多

逻辑不明显

代码列数增加

6 没有对代码进行抽象

全部代码都是细节

描述性差

7 未分层，分函数

函数名具备描述性

8 拆分大函数是重构，模式灵感的重要来源之一

从新抽象的过程

9 长的函数难以写单元测试

可能违反单一职责原则,一个函数干多件事

没有好的分层,没法方便的mock

没法分多个层测试

5注释

l  尽可能不添加多余的注释

让代码具备自描述性.这要求咱们在代码的自描述性上下功夫.例如名称,格式等.

l  有些注释要加

例如假设,限制条件,依赖或调用顺序,不说别人不知道的知识.

调用顺序尽可能让代码作到不顺序调用没法使用(例如<代码整洁之道>中的’ 把逻辑依赖改成物理依赖’),可是有时候也不得不使用注释来解决,可是这不是好的方法.

6 不要将C++当C使用

l  局部变量不须要再最前面声明

使用时才声明

l  有种参数叫引用参数&

不少地方可使用引用参数或const type&代替指针参数

l  不须要类时，不要刻意用类

毕竟增长复杂度

l  能用泛型时尽可能用泛型

泛型能解决不少不用泛型没法进行的抽象.

使用泛型不会有太大的代价.

编译速度可能会变慢

l  C++新标准中， function对象和Lamda表达式是很好用的东西，尽可能用好

有种范式是函数式编程,使用好function对象和Lamda表达式是一种函数式编程的实验.

l  尽可能不使用继承实现扩展

l  对外的接口最好写成C的

l  数据和操做最好在一个类中，而不是分开

这一点我是有些疑问的.

l  若是一个类中的全部成员都是static的，建议使用namespace代替类

同一命名空间， 能够放在多个文件中

使用时，可使用using namespace 节省输入

l  使用#program once代替头文件中的宏判断和定义来保证头文件的惟一性

l  类名有描述性,不须要在成员变量中再重复类名相关的描述文字

l  使用bool类型代替BOOL

BOOL是用户自定义类型,可能存在TRUE为0的状况,bool不会存在这种状况.

若是使用BOOL,请不要将TRUE当作true,FALSE当作FALSE.不要从新定义TRUE和FALSE