牽著老婆滿街逛

嚴以律己,寬以待人. 三思而后行.
GMail/GTalk: yanglinbo#google.com;
MSN/Email: tx7do#yahoo.com.cn;
QQ: 3 0 3 3 9 6 9 2 0 .

記錄兩個錯誤

都是弱智錯誤。
頭文件：

class Subsystem;

class Application

{

std::vector<Subsystem> mSubsystems;

}

cpp文件：

#include "Application"

Application::~Application()

{

for (size_t i=mSubsystems.size()-1; i>=0; --i)

{

delete mSubsystems[i];

}

在這里有兩個錯誤：
1.Subsystem在頭文件里面作了向前聲明，但是在cpp文件里面沒有包含其頭文件，因此，其析構函數是不會被調用的。

2.數組的反向遍歷，使用size_t也就是unsigned int，如果當i為0的時候，--i就為負數，為負數這是有符號整形的事情，而無符號整型卻會為這個值取反，變成一個很大的正數，這個時候就完蛋了，越界。
應該這樣：

int i = static_cast<int> (mSubsystems.size()-1);

for (; i>=0; --i)

{

Subsystem* subsystem = mSubsystems[i];

if (subsystem!=0) { delete subsystem; subsystem = 0;}

}

最近腦子真是糊涂到底了。寫了一大堆代碼，居然有這么弱智低級的錯誤，我先發現子系統沒有析構，等找出問題所在了，然后就是刪除子系統的順序錯了，開始用的是正序，顯然會出錯。再后面反序用無符號整型，數組越界。真是暈死掉了！！
真不知道最近在想什么去了。
代碼是越累越多，要還這么暈乎下去，那就意味著糊涂代碼也越來越多，那可就是一堆垃圾代碼呀，垃圾代碼有個P用呀！
嗨嗨。。。。

posted on 2008-07-01 01:14 楊粼波閱讀(269) 評論(0) 編輯收藏引用

只有注冊用戶登錄后才能發表評論。
【推薦】100%開源！大型工業跨平臺軟件C++源碼提供，建模，組態！



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

牽著老婆滿街逛

導航

統計

公告

常用鏈接

留言簿(11)

隨筆分類(466)

隨筆檔案(1513)

文章分類(46)

文章檔案(45)

相冊

收藏夾(39)

工具官網

技術網站

開源網站

其他窩點

收藏網站

銀行官網

友情鏈接

資源共享

搜索

積分與排名

最新評論

閱讀排行榜

記錄兩個錯誤