如何编写出安全高效的代码


1.命名约定

主要包括变量名、常量名、函数名、符号等。

2.优化和效率

规则一：对于在中断函数／线程和外部函数中均使用的全局变量应用volatile定义。例如：

volatile int ticks;

void timer(void) interrupt 1 //中断处理函数

{

ticks++

}

void wait(int interval)

{

tick=0;

while(tick<interval);

}

如果未用volatile，由于while循环是一个空循环，编译器优化后（编译器并不知道此变量在中断中使用）将会把循环优化为空操作！这就显然不对了。

规则二：不要编写一条过分复杂的语句，紧凑的C++/C代码并不见到能得到高效率的机器代码，却会降低程序的可理解性，程序出错误的几率也会提高。

规则三：变量类型编程中应用原则：尽量采用小的类型（如果能够不用“Float”就尽量不要去用）以及无符号Unsigned类型，因为符号运算耗费时间较长；同时函数返回值也尽量采用Unsigned类型，由此带来另外一个好处：避免不同类型数据比较运算带来的隐性错误。

3其他

规则一：不要编写集多种功能于一身的函数，在函数

的返回值中，不要将正常值和错误标志混在一起。

规则二：不要将BOOL值TRUE和FALSE对应于1和0进行编程。大多数编程语言将FALSE定义为0，任何非0值都是TRUE。Visual C++将TRUE定义为1，而Visual Basic则将TRUE定义为-1。例如：

BOOL flag;

…

if(flag) { // do something } // 正确的用法

if(flag==TRUE) { // do something } // 危险的用法

if(flag==1) { // do something } // 危险的用法

if(!flag) { // do something } // 正确的用法

if(flag==FALSE) { // do something } // 不合理的用法

if(flag==0) { // do something } // 不合理的用法

规则三：小心不要将“= =”写成“=”，编译器不会自动发现这种错误。

规则四：建议统一函数返回值为无符号整形，0代表无错误，其他代表错误类型。

最后需要明确一点的是这里的模块是以一个.C文件为单位。

规则一：利用函数命名规则和静态函数

模块中不被其他模块调用的内部函数采用以下命名规则：用全部小写，单词间采用带下划线的形式。如底层图形函数：pixel、lineto以及读键盘函数get_key等。这些函数应定义为static静态函数，这样在其他模块错误地调用这些函数时编译器能给出错误（如BC编译器）。（注意：有些编译器不能报告错误，但为了代码风格一致和函数层次清晰，仍建议这样作）。

规则二：利用静态变量

模块中不能被其他模块读写的全局变量应采用static声明，这样在其他模块错误地读写这些变量时编译器能给出警告（C51编译器）或错误（BC编译器）。

规则三：引入OO接口概念和指针传参

模块间的数据接口（也就是函数）应该事先较充分考虑，需要哪些接口，通过接口需要操作哪些数据，尽量作到接口的不变性。

模块间地数据交换尽量通过接口完成，方法是通过函数传参数，为了保证程序高效和减少堆栈空间，传大量参数（如结构）应采用传址的方式，通过指针作为函数参数或函数返回指针，尽量杜绝extern形式的全局变量，请注意是extern形式的全局变量，模块内部的全局变量是允许和必须的。

传指针参数增加的开销主要是作参数的指针和局部指针的数据空间（嵌入式系统（如C51）往往由于堆栈空间有限，函数参数会放到外部RAM的堆栈中），增加的代码开销仅是函数的调用，带来的是良好的模块化结构，而且使用接口函数会比在代码中多处直接使用全局变量大大节约代码空间。

需注意一点的事物总有他的两面性，水能载舟，也能覆舟。对于需要频繁访问的变量如果仍采用接口传递，函数调用的开销是巨大的，这时应考虑仍采用extern全局变量。

规则四：有限的封装与多态

不要忘记C++的class源于C的struct，C++的虚函数机制实质是函数指针。为了使数据、方法能够封装在一起，提高代码的重用度，如对于一些与硬件相关的数据结构，建议采用在数据结构中将访问该数据结构的函数定义为结构内部的函数指针

。这样当硬件变化，需要重写访问该硬件的函数，只要将重写的函数地址赋给该函数指针，高层代码由于使用的是函数指针，所以完全不用动，实现代码重用。而且该函数指针可以通过传参数或全局变量的方式传给高层代码，比较方便