请问这个函数为什么会泄露内存,应该怎么改?

[kenshu]

void MyFrame::MyFillRect(){
CBrush brushb(RGB(228,228,228));
int m_breakpoint;
CDC* hdc;
hdc = GetWindowDC();
CRect rect;
GetClientRect(&rect);
hdc->FillRect (CRect(rect.left+2 ,rect.top+2 ,rect.left + 30,rect.Height ()+2),&brushb);//画底色
brushb.DeleteObject ();
}
每调一次内存就长一点,降不下来了.

谢谢!

[skyxie]

CWnd::GetWindowDC

After painting is complete, the ReleaseDC member function must be called to release the display context.

[fishion]

你都没用CreateXXXX怎么要用DeleteObject 了

[晨星]

MSDN中对CDC::FillRect的第二个参数有如下说明：
The brush needs to either be created using the CBrush member functions CreateHatchBrush, CreatePatternBrush, and CreateSolidBrush, or retrieved by the GetStockObject Windows function.

也就是说，不能直接使用栈上定义的CBrush对象，问题会不会出在这里？

[paerxiushi]

void MyFrame::MyFillRect(){
CBrush brushb(RGB(228,228,228));
int m_breakpoint;
CDC* hdc;
hdc = GetWindowDC();
CRect rect;
GetClientRect(&rect);
hdc->FillRect (CRect(rect.left+2 ,rect.top+2 ,rect.left + 30,rect.Height ()+2),&brushb);//画底色
brushb.DeleteObject ();
ReleaseDC(hdc); //GetWindowDC之后的句柄不需要时，应释放它
}

[zoulie]

============================
[Quote=引用楼主 kenshu 的帖子:]
void MyFrame::MyFillRect(){
CBrush brushb(RGB(228,228,228));
int m_breakpoint;
CDC* hdc;
hdc = GetWindowDC();
CRect rect;
GetClientRect(&rect);
hdc->FillRect (CRect(rect.left+2 ,rect.top+2 ,rect.left + 30,rect.Height ()+2),&brushb);//画底色
brushb.DeleteObject ();
}
每调一次内存就长一点,降不下来了.

谢谢!
[/Quote]

[zoulie]

调用ReleaseDC释放 GetWindowDC获得的DC

[fengrx]

brushb可以不用释放。CBrush类的封装中已经有了吧。

Get得到的DC要Release掉。
Create得到的DC要Delete掉。

[fengrx]

楼主可以看看下面的文章，以后就知道怎么操作GDI了：

gdi使用经验分享

[kenshu]

收到,谢谢楼上这么多位,先结贴,我回头试试.

谢谢!

[oyljerry]

GDI操作还是要注意对应的分配，释放的API的配对调用等

[kenshu]

试过了,确实是这样子.
我平时只用VC写算法.

再次感谢楼上各位.

谨受教.

谢谢!