mthoutai

  博客园  :: 首页  :: 新随笔  :: 联系 :: 订阅 订阅  :: 管理

我的主题是,有时候知道一些细节会让你写出更好的代码。

============================================

之前学coocs2d-x的时候,发现有非常多do...while(0)的写法,一開始不明确为什么如此写,(起不到循环的作用),然后找了一下资料,发现这东西用处还蛮多的。如今来总结一下:

do...while(0)的妙用。

1.避免goto的使用

比方说我们须要在函数中处理一些错误。遇到错误则退出函数。当然退出之前我们须要释放一下资源。比方以下这样子:

bool HelloWorld::init()
{
   // 分配资源
   int *p = new int;
   bool bRet = true;

   // 运行并进行错误处理
   bRet = func1();
   if(!bRet) 
   {
      delete p;   
      p = NULL;
      return false;
   }

   bRet = func2();
   if(!bRet) 
   {
      delete p;   
      p = NULL;
      return false;
   }

   bRet = func3();
   if(!bRet) 
   {
      delete p;   
      p = NULL;
      return false;
   }

   // ..........

   // 运行成功,释放资源并返回
    delete p;   
    p = NULL;
    return true;
}

这样写最大的问题就是代码冗余。每次添加一个操作,就要处理一次,不灵活。所以这时候哦我们想到了goto,然后上面的版本号变成例如以下:

bool HelloWorld::init()
{
   // 分配资源
   int *p = new int;
   bool bRet = true;

   // 运行并进行错误处理
   bRet = func1();
   if(!bRet) goto error;

   bRet = func2();
   if(!bRet) goto error;

   bRet = func3();
   if(!bRet) goto error;

   // 运行成功。释放资源并返回
    delete p;   
    p = NULL;
    return true;

error:
    delete p;   
    p = NULL;
    return false;
}
当然,goto这东西尽管灵活方便,可是非常危急(这就跟红颜祸水这是一个意思,咦。怎么扯上这个了?)所以大部分的书籍都会建议你尽量程序中不要使用这个东西。

所以我们能够使用do...while来消除它。

例如以下:

bool HelloWorld::init()
{
	// 分配资源
	int *p = new int;
	bool bRet = true;

	do
	{
		// 运行并进行错误处理
  	    bRet = func1();
        if(!bRet) break;

        bRet = func2();
        if(!bRet) break;

        bRet = func3();
        if(!bRet) break;

	}while(0);

	// 运行成功,释放资源并返回
    delete p;   
    p = NULL;
    return bRet;  
}

2.宏定义中的do...while(0)


常常我们能在宏定义中看到这货的影子,比方说cocos2d-x中的

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


那么又没有循环,也仅仅运行了一次,这样子写有什么意义呢?

假如我们去掉do...while。例如以下:

#define CC_SAFE_DELETE(p)    delete (p); (p) = nullptr; 

if (p != NULL)
CC_SAFE_DELETE(p);

//上面会转换成这样子
if (p != NULL)
	delete (p); 
(p) = nullptr;  

如上。第二句永远运行,明显不合意思。有人可能会说,假设再加个括号呢,比方:

#define CC_SAFE_DELETE(p)    {delete (p); (p) = nullptr; }

//有可能碰到这样的情况
if (p != NULL)
	CC_SAFE_DELETE(p);
else
	//do else sth

//上面会转换成这样
if (p != NULL)
{
	delete (p); 
	(p) = nullptr; 
};
else
	//do else sth

else直接就编译错误了。


或许你会说,我们代码的习惯是在每一个推断后面加上{}, 就不会有这样的问题了,也就不须要do...while了,如:

if(...) 
{
}
else
{
}

诚然,这是一个好的,应该提倡的编程习惯。可是我们须要做的是让代码具有通用性,强壮性,因此我们更提倡do...while(0)的使用方法。

========================================


转载请注明出处:http://blog.csdn.net/shun_fzll/article/details/37776429



posted on 2017-05-01 16:30  mthoutai  阅读(236)  评论(0编辑  收藏  举报