hur.cn - 华软网

 热门搜索

请高人看看,这断代码有没有内存泄露

  作者:未知    来源:网络    更新时间:2010/12/7
CDialog::DrawBmp(CDC *pDC, CBitmap *pBitmap, CPoint pt, int MemBit_x, CSize bmpSize)
{
CDC MemDC;
MemDC.CreateCompatibleDC(pDC);
CBitmap *pOldBitmap = (CBitmap*)MemDC.SelectObject(pBitmap);

pDC->BitBlt(pt.x, pt.y, bmpSize.cx, bmpSize.cy, &MemDC, MemBit_x, 0, SRCCOPY);

MemDC.SelectObject(pOldBitmap);
MemDC.DeleteDC();
}

1)请问一下这段代码有没有内存-资源泄露?
2)我在BoundsChecker上看到有段演示代码,看见它没有最后一句。难道最后一句是多了的或多了这一句代码又是有误的吗?
---华软 网友回答---
应加上pOldBitmap->DeleteObject();

MemDC是中间变量,会自动释放,最后一句可无
---华软网友回复---
需要看看 pOldBitmap 是否为 NULL;如果是 NULL,说明绘图不可功;没有明显的泄漏;

#1 说得不对,不需要调用 pOldBitmap->DeleteObject;

MemDC 的析构函数会自动调用 DeleteDC。
---华软网友回复---
看看CDC的源代码就知道了,他会自己DeleteDC
CDC::~CDC()
{
if (m_hDC != NULL)
::DeleteDC(Detach());
}
---华软网友回复---
最后一句按理说不需要…
---华软网友回复---
引用 3 楼 varding 的回复:
看看CDC的源代码就知道了,他会自己DeleteDC
CDC::~CDC()
{
if (m_hDC != NULL)
::DeleteDC(Detach());
}

+1
---华软网友回复---
楼上的朋友们意见不一啊。
到里有没有泄露啊?
最后一句话真的不需要么?我看MFC上说,对GDI与CDC资源的运用都要记得释放啊?如果CDC对象不需要手动释放,那么CDC的一些子类如:CClientDC,CMetaFileDC,CPaintDC,CWindowDC运用完后要手动释放吗?
---华软网友回复---
你这代码应该没问题
原则上dc这些都是是要释放的
不过MFC相关的类比如CDC在析构函数都做了释放措置惩罚
手动释放是好习气
不过不释放也会有人收拾残局
---华软网友回复---
  class="deleted_message"> 该回复于2010-12-07 10:43:15被版主删除
---华软网友回复---
楼主的代码很好,没有问题。释放是一个好习气,在CDC确实会自行释放,但不要因此就降低对自己的要求。
---华软网友回复---
引用 7 楼 xianglitian 的回复:
你这代码应该没问题
原则上dc这些都是是要释放的
不过MFC相关的类比如CDC在析构函数都做了释放措置惩罚
手动释放是好习气
不过不释放也会有人收拾残局


是的,手动申请的上下文资源CDC和位图资源,无论在什么情况下,原则上都要手动释放,正所谓有始有终,虽然CDC在析构函数都做了释放措置惩罚,你不手动释放固然没什么问题,然则,从代码的可读性来说,就显得不那么完整了

所以,愚见以为用完这些资源之后手动释放较为合理

另外,pOldBitmap->DeleteObject是一定要加上的
     
华软声明:本内容来自网络,如有侵犯您版权请来信指出,本站立即删除。