MyMSDN

關于內存泄露的問題（解決+剖析）

首先先闡明這篇隨筆的意圖，只在告訴讀者，內存泄露的神不知鬼不覺，希望能引起大家的注意。
一段代碼的意思如何正確表達，才能不造成內存泄露呢？很多朋友經常泄露了內存但卻查找不到原因。當然在CLI/C++中利用托管對象堆上的垃圾收集器是可以更好地避免這一點。但是在更早的版本中，程序員有必要去手動刪除這些相關資源。否則將在程序關閉的時候出現一些錯誤。
MFC
現在我們去重載一個虛函數virtualvoidDeleteContents();用來在銷毀文檔數據前調用框架刪除一些文檔類的數據，（MSDN：Called by the framework to delete the document's data without destroying the CDocument object itself.）
先批評一段代碼：

評價：這段代碼看似簡練，但是卻很浪費資源，在第3行的for循環中，i<m_obArray.GetSize();當每次進行判斷的時候將再次調用GetSize()，如果這個數據的量是個天文數字，那么這樣的調用無疑是一種災難。

優化：

?1void?CGraphicDoc::DeleteContents()?
?2{
?3????int?nCount;
?4????nCount=m_obArray.GetSize();
?5????for(int?i=0;i<nCount;i++)
?6????{
?7????????……
?8????}
?9????CDocument::DeleteContents();
10}

填寫for循環內的語句。這里的任務：刪除之前利用CObArray : m_obArray對象保存的一個指針所指向的對象，以及指針本身。因此（以下提供幾種常見的錯誤代碼）
代碼A：
代碼B：
代碼A看起來似乎很符合常規思維，因此也很容易迷惑人。但是在程序運行的時候出現了錯誤。但是在MSDN中查找CObArray Class Members，查看RemoveAt，其中remarks中有這樣一句：In the process, it shifts down all the elements above the removed element(s). It decrements the upper bound of the array but does not free memory. 它的意思就是假設你一共有3個數據 p[0]=a、p[1]=b、p[2]=c，當你刪除第0個數據之后，數據將整體向前移動，變為p[0]=b、p[1]=c。這樣當你i=nCount-1的時候就根本沒有這樣的數讓你刪除，因為假設有那個時刻的話，數據的元素只有1個，而他的編號是0而不是nCount.因此出現了無法刪除的現象。因此也就隱含了問題了。
因此有人突發奇想：如果我每次只刪除第0個數據的話，那么是否就可以了呢？于是代碼B誕生了。可是問題終究沒能得到解決。因為假設有一組數據一共3個。刪除了編號0的元素（delete語句），移除了該元素的指針，此時i=1，進入刪除，又到了delete語句，這時候刪除元素i=1這樣的語句，這時實際上是刪除了先前元素中的第二個元素，而不是第一個。而0與2中間的第1個元素則未被刪除。又出現了隱含問題。其實只要將兩個都改為0，每次都刪除第一個就可以了。
另外可以將程序倒寫過來，避免RemoveAt對其進行重新整合隊列做產生的不可預料的麻煩。
或者仔細察看MSDN中還有一個函數叫RemoveAll()它是用來刪除整個CObArray集合對象的。

posted on 2006-08-16 03:44 volnet 閱讀(4195) 評論(5)  編輯 收藏 引用