#define 宏实现函数功能可能存在的问题

#define 宏实现函数功能的问题

情形1

#define free_ptr(p) \
	if(p) delete p; p = nullptr;

在调用free_ptr(p)的地方展开看这段代码：

if(p)
	delete p;
	p = nullptr;

其实判断条件if(p)，只对delete p 有效。不管if(p)是否满足执行条件p =nullptr都会执行。这么写可能直观一点

if(p)
	delete p;
p = nullptr；

显然这种操作和我们的意图是不一样的。

情形2

#define free_ptr(p) \
	delete p; p = nullptr;

Object* p = new Object;
if(p) free_ptr(p);

展开代码：

Object* p = new Object;
if(p)
    delete p;
p = nullptr;

显然这种操作也是不符合我们意图的。

情形3

#define free_ptr(p) \
	if(p){\
		delete p; p = nullptr;\
	}

Object* p = new Object;
free_ptr(p);

展开调用代码：

Object* p = new Object;
if(p){
    delete p;
    p = nullptr;
}

可以实现意图。

情形4

#define free_ptr(p) \
	do{\
		delete p; p = nullptr;\
	}while(0);

Object* p = new Object;
if(p) free_ptr(p);

展开调用代码：

Object* p = new Object;
if(p)
	do{
		delete p;
        p = nullptr;
	}while(0)

显然使用do{...}while(0)， 可以实现意图。

使用宏来实现一个函数的功能是危险的，这么做的时候需要十分小心。相比于宏，可能使用内联函数更为妥当。

template<typename T>
inline void  free_ptr(T** p)
{
	if(*p)
	{
		delete *p;
		*p = nullptr;
	}
}