2009年2月16日星期一

#define的坏处

在C++编程中应该尽量使用const和inline来代替#define,尽量做到能不用#define就不用。

#define常见的用途有"定义常量"以及"定义宏",但其中存在诸多的弊病。

第一,查错不直观,不利于调试。Define的定义是由预处理程序处理的,作的是完全的文本替换,不做任何的类型检查。在编译器处理阶段,define定义的东西已经被完全替换了,这样在debug的时候就看不到任何的相关信息,即跟踪时不能step
into宏。例如,把ASPECT_RATIO用define定义成1.653,编译器就看不到ASPECT_RATIO这个名字了。如果编译器报1.653错,那么就无从知道此1.653来自于何处。在真正编码的时候应该使用如下的语句来定义:
static const double ASPECT_RATIO = 1.653;

第二,没有任何类型信息,不是type
safe。因为它是文本级别的替换,这样不利于程序的维护。

第三,define的使用很容易造成污染。比如,如果有两个头文件都定义了ASPECT_RATIO,
而一个CPP文件又同时包含了这两个头文件,那么就会造成冲突。

更难查的是另外一种错误,比如有如下的代码:

// in header file def.h

#define Apple 1 #define Orange 2 #define Pineapple 3 …

// in some cpp file that includes the def.h

enum Colors {White, Black, Purple, Orange};

在.h文件中Orange被定义成水果的一种,而在.cpp文件中Orange又成为了一种颜色,那么编译器就会把此处的Orange替换成2,编译可能仍然可以通过,程序也能够运行,但是这就成了一个bug,表现出古怪的错误,且很难查错。

再比如定义了一个求a与b哪个数大的宏,

#define max(a,b) ((a) > (b) ?(a) : (b))

int a = 5, b = 0; max(++ a, b); max(++ a, b + 10);

在上面的操作中,max(++ a, b); 语句中a被++了两次,

而max(++ a, b + 10); 语句中a只加了一次,

这样在程序处理中就很有可能成为一个bug,且此bug也非常的难找。在实际编码时可以使用如下的语句来做:

template inline const T& max(const T& a, const T& b) { return a > b ? a :
b; }

没有评论: