一段自绘按钮的代码，为什么会有内存泄露？

qq3568i

每运行一次，内存就会增长12到16字节，请问错在哪儿？
代码：
void CShadowCtrl::DrawItem(LPDRAWITEMSTRUCT lpDrawItemStruct)
{
        CDC * dc = GetDC() ;
        dcMemory.CreateCompatibleDC(dc);
        CRect rc;
        GetClientRect(&rc);

        CBitmap bmp;
        bmp.CreateCompatibleBitmap(dc,rc.Width(),rc.Height());
        dcMemory.SelectObject(&bmp);
       
        CBrush brush,*oldBrush;
        CPen pen,*oldPen;

        //draw BK
        brush.Detach();
        brush.CreateSolidBrush(m_bkColor);
        pen.Detach();
        pen.CreatePen(PS_SOLID,1,m_bkColor);
        oldBrush = dcMemory.SelectObject(&brush);
        oldPen = dcMemory.SelectObject(&pen);
        dcMemory.Rectangle(&rc);

        //Draw Text
        dcMemory.SelectObject(&m_textFont);
        CString strText;
        GetWindowText(strText);
        dcMemory.SetTextColor(m_textCurrColor);
        dcMemory.SetBkMode(TRANSPARENT);
        dcMemory.DrawText(strText,&rc,DT_CENTER | DT_VCENTER |DT_WORDBREAK);

        dc->BitBlt(0,0,rc.Width(),rc.Height(),&dcMemory,0,0,SRCCOPY);
        dcMemory.SelectObject(oldBrush);
        dcMemory.SelectObject(oldPen);
        brush.DeleteObject();
        pen.DeleteObject();
        bmp.DeleteTempMap();
        bmp.DeleteObject();
        dcMemory.DeleteDC();       
}

chris_zc

你用了
CDC * dc = GetDC() ;
你应该加上
ReleaseDC(dc);
你再试试吧，看有没有效果。

afanti_007

内存泄露怎么回事我不知道
不过我重载这个函数时rcItem ,hwnd....都是通过lpDrawItemStruct得到的

chris_zc

内存泄露就是你申请暂用了内存但没有释放，特别是在ontimer 事件中，假设你一秒申请一次这样没有多久你的程序就该死了。

chris_zc

例如：
CPen*apen=new CPen;//申请暂用内存；
你就应该加上
delete apen;
如果不加上，这就是内存泄露了。
不知道这样说合不合理，我也是个菜鸟。

qq3568i

确实是DC的问题，谢谢
用了lpDrawItemStruct的DC就没问题了

chris_zc

解决问题就好，慢慢的进步，时间长了我们应该就会变得强一些了。

深蓝旅者

you forget to restore old bmp and old font.
        CDC * dc = GetDC() ;
        dcMemory.CreateCompatibleDC(dc);
        CRect rc;
        GetClientRect(&rc);
       
        CBitmap bmp;
        bmp.CreateCompatibleBitmap(dc,rc.Width(),rc.Height());
        CBitmap* oldBmp = dcMemory.SelectObject(&bmp);
       
        CBrush brush,*oldBrush;
        CPen pen,*oldPen;
       
        //draw BK
//         brush.Detach();
        brush.CreateSolidBrush(m_bkColor);
//         pen.Detach();
        pen.CreatePen(PS_SOLID,1,m_bkColor);
        oldBrush = dcMemory.SelectObject(&brush);
        oldPen = dcMemory.SelectObject(&pen);
        dcMemory.Rectangle(&rc);
       
        //Draw Text
        CFont* oldFont = dcMemory.SelectObject(&m_textFont);
        CString strText;
        GetWindowText(strText);
        dcMemory.SetTextColor(m_textCurrColor);
        dcMemory.SetBkMode(TRANSPARENT);
        dcMemory.DrawText(strText,&rc,DT_CENTER | DT_VCENTER |DT_WORDBREAK);
       
        dc->BitBlt(0,0,rc.Width(),rc.Height(),&dcMemory,0,0,SRCCOPY);
        dcMemory.SelectObject(oldBrush);
        dcMemory.SelectObject(oldPen);
        dcMemory.SelectObject(oldFont);
        dcMemory.SelectObject(oldBmp);
        brush.DeleteObject();
        pen.DeleteObject();
//         bmp.DeleteTempMap();
        bmp.DeleteObject();
        dcMemory.DeleteDC();
        ReleaseDC(dc);