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

[dianhui]

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上看到有段演示代码，看见它没有最后一句。难道最后一句是多余的或者多了这一句代码又是有误的吗？

[blonea]

应加上pOldBitmap->DeleteObject();

MemDC是中间变量，会自动释放，最后一句可无

[lazy_2010]

需要看看 pOldBitmap 是否为 NULL；如果是 NULL，说明画图不成功；没有明显的泄漏；

#1 说得不对，不需要调用 pOldBitmap->DeleteObject；

MemDC 的析构函数会自动调用 DeleteDC。

[varding]

看看CDC的源代码就知道了，他会自己DeleteDC
CDC::~CDC()
{
if (m_hDC != NULL)
::DeleteDC(Detach());
}

[ctwoz]

最后一句按理说不需要…

[zzw820626]

[Quote=引用 3 楼 varding 的回复:]

看看CDC的源代码就知道了，他会自己DeleteDC
CDC::~CDC()
{
if (m_hDC != NULL)
::DeleteDC(Detach());
}
[/Quote]
+1

[dianhui]

楼上的朋友们意见不一啊。
到里有没有泄露啊？
最后一句话真的不需要么？我看MFC上说，对GDI与CDC资源的使用都要记得释放啊？如果CDC对象不需要手动释放，那么CDC的一些子类如：CClientDC,CMetaFileDC,CPaintDC,CWindowDC使用完后要手动释放吗？

[向立天]

你这代码应该没问题
原则上dc这些都是是要释放的
不过MFC相关的类比如CDC在析构函数都做了释放处理
手动释放是好习惯
不过不释放也会有人收拾残局

[sunlin7]

楼主的代码很好，没有问题。释放是一个好习惯，在CDC确实会自行释放，但不要因此就降低对自己的要求。

[信阳毛尖]

[Quote=引用 7 楼 xianglitian 的回复:]
你这代码应该没问题
原则上dc这些都是是要释放的
不过MFC相关的类比如CDC在析构函数都做了释放处理
手动释放是好习惯
不过不释放也会有人收拾残局
[/Quote]

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

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

另外，pOldBitmap->DeleteObject是一定要加上的