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

dianhui 2010-12-07 02:02:13
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上看到有段演示代码,看见它没有最后一句。难道最后一句是多余的或者多了这一句代码又是有误的吗?
...全文
69 点赞 收藏 10
写回复
10 条回复
切换为时间正序
当前发帖距今超过3年,不再开放新的回复
发表回复
信阳毛尖 2010-12-07
[Quote=引用 7 楼 xianglitian 的回复:]
你这代码应该没问题
原则上dc这些都是是要释放的
不过MFC相关的类比如CDC在析构函数都做了释放处理
手动释放是好习惯
不过不释放也会有人收拾残局
[/Quote]

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

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

另外,pOldBitmap->DeleteObject是一定要加上的
回复
lisunlin0 2010-12-07
楼主的代码很好,没有问题。释放是一个好习惯,在CDC确实会自行释放,但不要因此就降低对自己的要求。
回复
向立天 2010-12-07
你这代码应该没问题
原则上dc这些都是是要释放的
不过MFC相关的类比如CDC在析构函数都做了释放处理
手动释放是好习惯
不过不释放也会有人收拾残局
回复
dianhui 2010-12-07
楼上的朋友们意见不一啊。
到里有没有泄露啊?
最后一句话真的不需要么?我看MFC上说,对GDI与CDC资源的使用都要记得释放啊?如果CDC对象不需要手动释放,那么CDC的一些子类如:CClientDC,CMetaFileDC,CPaintDC,CWindowDC使用完后要手动释放吗?
回复
zzw820626 2010-12-07
[Quote=引用 3 楼 varding 的回复:]

看看CDC的源代码就知道了,他会自己DeleteDC
CDC::~CDC()
{
if (m_hDC != NULL)
::DeleteDC(Detach());
}
[/Quote]
+1
回复
ctwoz 2010-12-07
最后一句按理说不需要…
回复
varding 2010-12-07
看看CDC的源代码就知道了,他会自己DeleteDC
CDC::~CDC()
{
if (m_hDC != NULL)
::DeleteDC(Detach());
}
回复
lazy_2010 2010-12-07
需要看看 pOldBitmap 是否为 NULL;如果是 NULL,说明画图不成功;没有明显的泄漏;

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

MemDC 的析构函数会自动调用 DeleteDC。
回复
blonea 2010-12-07
应加上pOldBitmap->DeleteObject();

MemDC是中间变量,会自动释放,最后一句可无
回复
相关推荐
发帖
VC/MFC
创建于2007-09-28

1.5w+

社区成员

VC/MFC相关问题讨论
申请成为版主
帖子事件
创建了帖子
2010-12-07 02:02
社区公告

        VC/MFC社区版块或许是CSDN最“古老”的版块了,记忆之中,与CSDN的年龄几乎差不多。随着时间的推移,MFC技术渐渐的偏离了开发主流,若干年之后的今天,当我们面对着微软的这个经典之笔,内心充满着敬意,那些曾经的记忆,可以说代表着二十年前曾经的辉煌……
        向经典致敬,或许是老一代程序员内心里面难以释怀的感受。互联网大行其道的今天,我们期待着MFC技术能够恢复其曾经的辉煌,或许这个期待会永远成为一种“梦想”,或许一切皆有可能……
        我们希望这个版块可以很好的适配Web时代,期待更好的互联网技术能够使得MFC技术框架得以重现活力,……