如何提高代码的可读性

近日有在看Brain W. Kernighan & Rob Pike 共同编著的The practice of programming(强烈推荐看看这本书)。书中提到了很多实用的细节,大多来源于作者多年的编程经验,在关于代码风格这方面就是开篇,有不少篇幅,在此整理摘录一些个人认为重要的原则(rules),虽然细小,但很重要,必要的会附上相关代码。

一、命名(Name)

1.对于全局变量最好用更具描述性的名字,对于局部变量就用短一点的名字。(Use descriptive names for globals, short name for locals)

二、表达式和语句(Expressions and Statements)
1.用必要的括号消除歧义(Parenthesize to solve ambiguity)

因为许多操作符都有各自的优先级,如果没有用好括号或是完全忘了就会让程序出现逻辑问题无法运行。

以下是示例代码:

if (x&MASK == BITS)

如果没有括号,代码会执行为

if (x & (MASK == BITS))

但实际上想操作的却是

if ((x&MASK) == BITS)

2.将复杂的表达式拆开写(Break up complex expressions)

*x += (*xp=(2*k < (n-m) ? c[k+1] : d[k--]));

上面这段代码的可读性非常差,而如果我们拆成if else语句就会好很多。

if (2*k < n-m)
*xp=c[k+1];
else
*xp =d[k--];
*x += *xp;

书中有一句话说:“我们的目标是写出清晰的代码,而不是自认为聪明的代码。”

对于 ?: 操作符的使用建议是只适用于短小的表达式,如:

max = (a > b) ? a : b ;

代码并不是越短越好,逻辑清晰和可读性十分重要,如果是以削减可读性的代价去缩短代码是不建议的。最好是想想优化算法吧:P

3.注意“副作用” (Be careful with side effects)

str[i++] = str[i++] = ' ' ;
这段代码意在将空格储存在str的后两个位置,但这取决于 i 更新数据的时间,因此可能 i 只增加了1。为了避免这种情况,我们应该将它拆开

str[i++] = ' ' ;
str[i++] = ' ' ;

而即使只有一个变量在增加也可能有类似情况
a[i++] = i ;
如果初始值是3,结果可能会是3 或者4。

三、持续性和习惯(Consistency and Idioms)
1.在花括号的使用风格上保持持续性

这里又提到了一个经常争议的问题就是“花括号到底应该直接放语句后面还是换行再写。”对此作者的建议是:

选一个你喜欢的并且保持下去,不要花时间在争吵上面。(Pick one style, use it consistently,and don't waste time arguing.):D

有一个值得一提的细节是,当你在修改别人写的代码时,你应该按照他的代码风格来写,因为这样保持风格统一方便了以后来修改代码的人。:)

2,无穷循环的选择

对于无穷的循环,作者只建议以下两种表达
for (; ;)

while( 1 )

3.谨慎使用 gets

原话是" one shoud never use gets"。
因为用gets没有办法去限制它将读到的输入长度,这种缺陷会导致安全问题,而 fgets 就会是一个更好的选择。

4.对于多种情况,用else-if语句,尽量少一些 if 嵌套

对比以下两种代码风格

if 嵌套:

if (argc==3)
if((fin = fopen(argv[1], "r"))!= NULL)
if((fout = fopen(argv[2],"W"))!= NULL){
while( 
...........
)
}else
printf(" .........."); `
else
.............
else
..............

else-if语句:

if (argc != 3)
printf(" Usage: cp inputfile outputfile\n");
else if (...........)
else if(............){
......
....
}else{
.....
}

有良好的写代码的习惯能提高写代码的可读性,提高调试的效率,也是对自己和以后修改你程序的其他程序员的一种尊重吧:D

你可能感兴趣的:(如何提高代码的可读性)