一．使用#define宏应注意的问题
1.使用宏定义表达式的时候，加括号是一个好习惯
首先我们来看一段简短的代码，并试着分析其输出：
#include <iostream>
#define Add(a,b) a+b
int main()
{
std::cout<<Add(1,2)*Add(2,3)<<std::endl;
return 0;
}
结果输出：
很显然，当我们看到这么一段代码的时候，肯定会呲之以鼻，认为这么简单的错误怎么可能发现不了？别急，这还只是开始，下面我们在对上述代码进行改进，便有了下面的代码：
#include <iostream>
#define Add1(a,b) (a+b)
#define Add2(a,b) (a) + (b)
#define Mul(a,b) (a*b)
int main()
{
std::cout<<Add2(1,2)*Add2(2,3)<<std::endl; //理论上应该是 15
std::cout<<Mul(Add2(1,2),3)<<std::endl;//理论上应该输出 9
system("pause");
return 0;
}
结果输出：
上述结果表明，即使是整体加上括号或者分别单独加上括号都不能很好的解决问题，最完备的解决方案就是不要吝啬你的括号，用完备的括号完备的保护每一个宏参数，也就是说，针对上述案例，应该这样：
#define Add(a,b) ((a)+(b))
#define Mul(a,b) ((a)*(b))
2.使用宏的时候，参数不能变化
首先，照例我们还是来看一段问题代码：
#include <iostream>
#define CUBE(a) ((a)*(a)*(a))
inline int Cube(int a)
{
return a*a*a;
}
int main()
{
int base1 = 2,base2=2;
int nCube1 = CUBE(++base1);
int nCube2 = Cube(++base2);
std::cout<<"nCube1 = "<<nCube1<<std::endl;
std::cout<<"nCube2 = "<<nCube2<<std::endl;
system("pause");
return 0;
}
再看一下输出结果：
其实这样的结果也很好理解，因为宏是单纯的替换，每一次替换都自增了一次，这就是宏在展开时对其参数的多次取值替换所带来的副作用，为避免这种情况，最简单的方法就是保证使用宏的时候参数不能变换。
３.用大括号将宏所定义的多条表达式括起来
我们还是来看一段示例：
#include <iostream>
typedef struct Cube
{
int x;
int y;
int z;
};
#define INITIAL(a,b,c) a=1; b=2; c=3;
int main()
{
int x,y,z;
INITIAL(x,y,z);
std::cout<<"x = "<<x<<" y = "<<y<<" z = "<<z<<std::endl;//OK no problem
Cube cube[10];
int i=0;
for (;i<10;++i)
INITIAL(cube[i].x,cube[i].y,cube[i].z);
for (i=0;i<10;++i)
std::cout<<cube[i].x<<" "<<cube[i].y<<" "<<cube[i].z<<std::endl;
system("pause");
return 0;
}
结果显示：
其实说来说去还是一个宏替换的范围问题，改正很简单，只要在宏定义的时候加上大括号就行了，其实宏是一个很强大的工具，但是我们在用的时候要格外的小心，千万不要吝啬你的括号。