就算你熟记并正确使用了，写出来的代码也轻易产生歧义而使其可读性较差。 好的风格 if ((a b) && (a & c)) 坏的风格 if (a b && a & c) 虽然后者和前者功能一样，但后者是很恐怖的，难以阅读。 3）不要编写太复杂的复合表达式。 复合表达式使用在适当的场合可以使代码更加简洁，但不能因为这个简洁而带来理解的复杂。 例如： max = a > b ？(a > c ? a : c) : (b > c ? b : c) // 复合表达式过于复杂 应该修改为： max = a; if(max < b) { max = b; } if(max < c) { max = c; }
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++)的意思就很清楚了。
int getchar(void); 10)乱指一气的指针 “野指针”者，乱指一气的指针也，它不是NULL指针，是指向“垃圾”内存的指针。野 指针是很危险的，是经常导致bug的原因，它的成因主有两种：一是指针变量没有被初始化。在C/C++中任何指针变量刚被创建时不会自动成为NULL指针，它的缺省值是随机的。所以，指针变量在创建的同时应当被初始化，要么将指针设置为NULL，要么让它指向合法的内存。例如
