我的主题是,有时候知道一些细节会让你写出更好的代码。
============================================
之前学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