用返回值返回錯誤真是一個錯誤�。�

lipingtababa

我最近參與的項目，用到同事之前寫的一個類，這個工具類是用來打包ＴＣＰ　Ｐacket的，有大量的public　member　function，形如“AddULong(), GetULong(), AddString()，GetString(), GetBodySize()。

這些函數(shù)我要大量的調(diào)用，每次組包都要調(diào)用，因此它們的調(diào)用散落在各處。

我現(xiàn)在很頭疼的是，這些函數(shù)可能因為內(nèi)存不夠而返回錯誤，而這些函數(shù)是通過返回false來告知錯誤的。它們是這樣定義的　bool AddULong(unsigned long l)；這樣我每次調(diào)用，都要檢查返回值。那么我的代碼就會象這樣難看：


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);
這樣的代碼就很危險了，如果函數(shù)因為內(nèi)存出錯，就可能導致不可預料的bug，而且還很難>重現(xiàn)

但是上面那種代碼實在太難看了，感覺程序員不用干別的，專門寫重復代碼去了

我現(xiàn)在把這個類重寫了，內(nèi)存由對象自增長，用異常處理new失敗的情況，改個名字，打算>用在我的新代碼里。

blizzard213

原帖由 lipingtababa 于 2008-11-24 01:43 發(fā)表
我最近參與的項目，用到同事之前寫的一個類，這個工具類是用來打包ＴＣＰ�。衋cket的，有大量的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 發(fā)表
throw還沒返回錯誤簡單

又來了,上次談引用的帖子您也是一句話"引用真是太好用了",然后什么也不多說.

這里,我又是拷貝代碼又是舉例,就是想說用返回值返回錯誤不方便使用,如果我說得片面了,歡迎指教.

可你老人家一句話就下結論"throw還沒返回錯誤簡單",沒有代碼,沒有解釋,你要真是Bjarne Stroustrup那我就服氣,可你又不是.