用返回值返回错误真是一个错误啊！

lipingtababa

我最近参与的项目，用到同事之前写的一个类，这个工具类是用来打包ＴＣＰ　Ｐacket的，有大量的public　member　function，形如“AddULong(), GetULong(), AddString()，GetString(), GetBodySize()。

这些函数我要大量的调用，每次组包都要调用，因此它们的调用散落在各处。

我现在很头疼的是，这些函数可能因为内存不够而返回错误，而这些函数是通过返回false来告知错误的。它们是这样定义的　bool AddULong(unsigned long l)；这样我每次调用，都要检查返回值。那么我的代码就会象这样难看：


bool ret=true;

ret=AddULong(1000L);
if (ret=false)
        Log("Failed to AddULong!");
ret=AddUShort(10);
if (ret=false)
        Log("Failed to AddULong!");
ret=AddUString("root");
if (ret=false)
        Log("Failed to AddULong!");
ret=AddULong(1000L);
if (ret=false)
        Log("Failed to AddULong!");

我看了其他同事的代码，大家基本不检查返回值，代码如下：
AddULong(1000L);
AddUShort(10);
AddUString("root");
AddULong(1000L);
这样的代码就很危险了，如果函数因为内存出错，就可能导致不可预料的bug，而且还很难>重现

但是上面那种代码实在太难看了，感觉程序员不用干别的，专门写重复代码去了

我现在把这个类重写了，内存由对象自增长，用异常处理new失败的情况，改个名字，打算>用在我的新代码里。

blizzard213

原帖由 lipingtababa 于 2008-11-24 01:43 发表
我最近参与的项目，用到同事之前写的一个类，这个工具类是用来打包ＴＣＰ　Ｐacket的，有大量的public　member　function，形如“AddULong(), GetULong(), AddString()，GetString(), GetBodySize()。

这些函 ...

感觉就是个serializer类嘛

samon_fu

linux里面喜欢这样：

1. 
   
2. 
   
3. if ( ((ret = AddULong(1000L) ) == false )
   
4.         Log("Failed to AddULong!");
   
5. 
   

复制代码

[ 本帖最后由 samon_fu 于 2008-11-24 02:42 编辑 ]

snick

写个宏好了, 代码简洁多了

#define JUDGED_ADDULONG(l,msg) { if  (AddULong(l)  == false )    Log(msg); }

system888net

xinglp

汗, 我就这么写

lipingtababa

#define JUDGED_ADDULONG(l,msg) { if  (AddULong(l)  == false )    Log(msg); }

这个方法也不好改动啊,比如我要写成
if  (AddULong(l)  == false )  
{
    Log(msg);
    return -1;
}
就没办法了.

lipingtababa

if ( ((ret = AddULong(1000L) ) == false )
        Log("Failed to AddULong!");

这个写法少写了一行,但是代码很难看,单步跟踪的时候也不方便.

cugb_cat

throw还没返回错误简单

lipingtababa

原帖由 cugb_cat 于 2008-11-24 18:21 发表
throw还没返回错误简单

又来了,上次谈引用的帖子您也是一句话"引用真是太好用了",然后什么也不多说.

这里,我又是拷贝代码又是举例,就是想说用返回值返回错误不方便使用,如果我说得片面了,欢迎指教.

可你老人家一句话就下结论"throw还没返回错误简单",没有代码,没有解释,你要真是Bjarne Stroustrup那我就服气,可你又不是.