一区二区欧美亚洲,久久久亚洲高清,亚洲大胆人体在线

需要判斷指針為空嗎

前幾天，KOK3客戶端因為我的資源配置錯誤而崩潰了。
調試器帶我到了出錯的代碼行，是一個空指針解引用。
代碼大致如下：

   Item* pItem = itemList.getItem(index);
   return *pItem;

getItem方法如下：

Item* ItemList::getItem(int index)
{
   if (index < 0) return NULL;
   if (index >= size()) return NULL;
   return _list[index];
}

錯誤是因為資源加載出錯，列表為空，所以getItem()返回NULL.

對于資源加載出錯，應該加載時就提示出錯，不應該在使用時才報錯。

getItem()做了索引越界出錯處理，所以它的調用者必須檢查其返回值。

假設ItemList的指針元素保證不為空，正常情況下，索引正確情況下都不會返回空指針。
這種情況下，添加斷言是必要的。

   Item* pItem = itemList.getItem(index);
   assert(pItem);
   return *pItem;

Item* ItemList::getItem(int index)
{
   ...
   assert(_list[index]);
   return _list[index];
}


對與索引值，還是用無符號數為好，可省去下界溢出斷。

posted on 2010-02-11 09:48 金慶閱讀(4524) 評論(11) 編輯收藏引用所屬分類: 1. C/C++ 、2. 網游開發

這個問題在我們組里也備受爭議。有人覺得對每個指針進行拿捏然后再決定判斷，很多余，所以索性對所有指針進行空判斷；有人覺得沒必要寫那么多冗余代碼，需要對每個指針進行上下文審查。

拿一個接口為例，接口實現里是否有必要對參數進行判斷?接口調用者是否需要對返回值做判斷?反正這種事情挺糾結的. 回復更多評論

# re: 需要判斷指針為空嗎 2010-02-11 11:19 Jimixu

簡簡單單的從兩個if不寫在一起，用||隔開就看出LZ功力非凡，可以節省很多不必要的判斷
Item* ItemList::getItem(int index)
{
if (index < 0) return NULL;
if (index >= size()) return NULL;
return _list[index];
}
回復更多評論

# re: 需要判斷指針為空嗎[未登錄] 2010-02-11 11:50 zz

@Jimixu
兩個if寫在一起用||隔開會增加很多判斷嗎？不解。
如果第一個條件成立，||的后一個條件不是不會判斷了嗎？回復更多評論

# re: 需要判斷指針為空嗎 2010-02-11 11:58 jQuery

@Jimixu

=_=
回復更多評論

# re: 需要判斷指針為空嗎[未登錄] 2010-02-11 12:37 絕對零度

@Jimixu
有這種講解嗎？回復更多評論

# re: 需要判斷指針為空嗎 2010-02-11 16:18 金慶

@kevin lynx
我的觀點是：只要有可能為空的，就一定要判斷是否為空; 不可能為空的，可以用一個assert斷言來表達。
關鍵在于判斷是否有可能為空。由函數調用者保證不為空的，就認為不可能為空，可在函數說明中注釋并在函數開始處加個斷言。由進程以外輸入決定的，如用戶輸入，文件輸入，或其他進程傳入數據決定的指針，即使外部接口規定了不會造成空指針，但仍應該對輸入判斷。外部輸入錯誤不應該造成程序錯誤。回復更多評論

# re: 需要判斷指針為空嗎 2010-02-13 17:01 陳梓瀚(vczh)

理論上，對于一個對象A，如果他不能在這個上下文里面取到所有類型T的值，那么A的類型就不能是T。

舉個例子，句柄可以是個指針。但是如果有規定是不能為空，那就去搞個句柄類。無論你在哪里使用，都可以在復制和構造的時候集中檢查。譬如，句柄的構造函數可以檢查傳入的指針，為空就崩潰。只要構造成功了，那么句柄里面的內部指針肯定不會是空的。回復更多評論

# re: 需要判斷指針為空嗎[未登錄] 2010-02-15 16:24 Steven

我是比較推崇這樣的寫法的
if (index < 0) return NULL;
if (index >= size()) return NULL;
雖然用 || 可以生成一樣的匯編代碼，但是對于調試維護來說是有區別的
如果我用 || 的話，并且在return 之前，輸出一些錯誤log
if (index < 0 || index >= size())
{
cout << "invalide index";
return null;
}
當我看到這個log的時候，你怎么才能知道到底是哪個判斷出錯了呢？index是太大還是太小呢？當然你可以說我應該把index的值也一并輸出出來，但是在實踐中很多判斷是很復雜的，很難輸出所有的值，比如長字符串的比較，總不能輸出那么多東西到log里吧。回復更多評論

# re: 需要判斷指針為空嗎 2010-05-12 22:00 classyk

索引值用無符號數并不能減少判斷，反而更加麻煩回復更多評論

# re: 需要判斷指針為空嗎 2010-05-13 09:38 金慶

@classyk
傳統的C程序中是int類型到處使用，而在C++中更傾向于各種功能用不同的類型，如容器大小用size_t. 用無符號數減少了小于0的判斷，“更加麻煩”在哪里？回復更多評論

刷新評論列表

只有注冊用戶登錄后才能發表評論。


相關文章: How are dtLinks created in NavMesh C++ parameter passing rules Naming Conventions for Accessors Visual Studio 2019 Compiler Hangs Fbx File Format Identifier 查找內存錯誤 std::thread 中的異常會丟失調用棧用賦值代替 protobuf CopyFrom() vs2017 linux 編譯輸出改成 vs 格式為 LiteIDE 添加選中標記

網站導航: 博客園 IT新聞 BlogJava 博問 Chat2DB 管理

青青草原综合久久大伊人导航_色综合久久天天综合_日日噜噜夜夜狠狠久久丁香五月_热久久这里只有精品

金慶的專欄

公告

留言簿(12)

隨筆分類(502)

隨筆檔案(423)

相冊

搜索

積分與排名

最新評論

閱讀排行榜

評論排行榜

評論