编程人员怎样才能写出美丽的代码(1)(2)

6. 改善循环的效率

对于字符串name,看下面的循环：

for (i = 0; i < strlen(name); i++)效率明显差于下面的循环：  
n = strlen(name);  
for (i = 0; i < n; i++) 

后者只要计算name的长度一次。

7. 少用、慎用goto语句，并不禁用

goto语句能从多重循环体或者代码堆里一下子跳到外面， 例如：

{ …  
{ …  
{ …  
on error goto errorhandler;  
}  
}  
}  
errorhandler:  
…    

在Visual Basic中这一招是常用的。

8. 消除魔鬼数

魔鬼数者，没名字的常数也，你若看英文资料，它们的说法是magic data，我们的一些作品将其翻译为“魔术数”，我更愿意将其翻译为“魔鬼数”，因为它是一个导致代码的可读性极差的“魔鬼”。

假使你在程序里写下下面的一段代码：

for (i=0; i < 100; i++);for (i=0; i < 99; i++); 

谁都不知道100、99是个什么玩意，你可能意味着100是范围的边界(最大值)，就应该给出定义，代码的读者才能明白你的意思：

#define MAX 100 /* C语言的宏常量 */  
const int MAX = 100; // C++ 语言的const常量  
for (i=0; i < MAX; i++);for (i=0; i < MAX-1; i++)的意思就很清楚了。 

并且如果某一常量与其它常量密切相关，应在定义中包含这种关系，而不应给出一些孤立的值。

例如：

const float RADIUS = 100;  
const float DIAMETER = RADIUS * 2;  

9. 函数返回值

函数名字与返回值类型在语义上不可冲突,C标准库函数getchar违反了这一规则。

例如：

char c;  
c = getchar();  
if (c == EOF)　  

按照getchar名字的意思，将变量c声明为char类型是很自然的事情。但不幸的是getchar的确不是char类型，而是int类型，其原型如下：

int getchar(void); 

10. 乱指一气的指针

“野指针”者，乱指一气的指针也，它不是NULL指针，是指向“垃圾”内存的指针。野指针是很危险的，是经常导致bug的原因，它的成因主有两种：

一是指针变量没有被初始化。在C/C++中任何指针变量刚被创建时不会自动成为NULL指针，它的缺省值是随机的。所以，指针变量在创建的同时应当被初始化，要么将指针设置为NULL，要么让它指向合法的内存。例如：

char *p = NULL;  
char *str = (char *) malloc(100);  

二是指针p被free或者delete之后，没有置为NULL，让人误以为p是个合法的指针。

关于编码的风格，笔者还有许多需要讲解的，本文笔者暂时讲到这里，希望能对编程者有所帮助。