请看一下该函数是否存在泄漏

永远爱好写程序 2009-08-13 05:12:08
我在CFormView中的OnPaint函数调用下面这个函数，控件移动触发OnPaint，刚开始没问题，运行1分钟左右时间就会出错，
好像是内存泄露了，去掉这个函数就没问题，请高手帮我看一下
BOOL CDragControlVC6View::DrawControlSelect(const CRect& rectControl,const CRect& rectControlOld,CClientDC& dcDrawSelect)
{
//CClientDC dcDrawSelect(this);
CPen pen(0,1,RGB(236,233,216));
CPen* poldpen;

CBrush* pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(236,233,216))));
poldpen=dcDrawSelect.SelectObject(&pen);
//SetROP2(dcDrawSelect,R2_XORPEN);
CRect rectSelect1(CPoint(rectControlOld.left-10,rectControlOld.top-11),CPoint(rectControlOld.left,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);
rectSelect1.SetRect(CPoint(rectControlOld.right,rectControlOld.top-11),CPoint(rectControlOld.right+10,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);
//dcDrawSelect.SelectObject(pOldBrush);


pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));
CRect rectSelect(CPoint(rectControl.left-10,rectControl.top-11),CPoint(rectControl.left,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
rectSelect.SetRect(CPoint(rectControl.right,rectControl.top-11),CPoint(rectControl.right+10,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
dcDrawSelect.SelectObject(pOldBrush);
dcDrawSelect.SelectObject(poldpen);//恢复画笔
pen.DeleteObject();
return TRUE;
}
...全文
155 17 打赏 收藏 转发到动态 举报
写回复
用AI写文章
17 条回复
切换为时间正序
请发表友善的回复…
发表回复
pady_pady 2009-08-13
  • 打赏
  • 举报
 回复
倒数第二行改为pOldBrush->DeleteObject(); //
快了点
pady_pady 2009-08-13
  • 打赏
  • 举报
 回复
[Quote=引用 4 楼 tttyd 的回复:]
pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));
你的两端代码创建的画笔都没有释放，调用下面代码释放
pOldBrush->DeleteObject();
另外
pen.DeleteObject(); 不必要，pen的析构函数会释放
[/Quote]




BOOL CDragControlVC6View::DrawControlSelect(const CRect& rectControl,const CRect& rectControlOld,CClientDC& dcDrawSelect) 

{ 

//CClientDC dcDrawSelect(this); 

CPen  pen(0,1,RGB(236,233,216)); 

CPen* poldpen;  



CBrush* pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(236,233,216)))); 

//pOldBrush->DeleteObject(); 

poldpen=dcDrawSelect.SelectObject(&pen); 

//poldpen->DeleteObject(); 

//SetROP2(dcDrawSelect,R2_XORPEN); 

CRect rectSelect1(CPoint(rectControlOld.left-10,rectControlOld.top-11),CPoint(rectControlOld.left,rectControlOld.top+1)); 

dcDrawSelect.Rectangle(rectSelect1); 

rectSelect1.SetRect(CPoint(rectControlOld.right,rectControlOld.top-11),CPoint(rectControlOld.right+10,rectControlOld.top+1)); 

dcDrawSelect.Rectangle(rectSelect1); 



pOldBrush = dcDrawSelect.SelectObject(pOldBrush); 

pOldBrush->DeleteObject(); 



pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0)))); 

//pOldBrush->DeleteObject(); 

CRect rectSelect(CPoint(rectControl.left-10,rectControl.top-11),CPoint(rectControl.left,rectControl.top+1)); 

dcDrawSelect.Rectangle(rectSelect); 

rectSelect.SetRect(CPoint(rectControl.right,rectControl.top-11),CPoint(rectControl.right+10,rectControl.top+1)); 

dcDrawSelect.Rectangle(rectSelect); 

pOldBrush = dcDrawSelect.SelectObject(pOldBrush); 

dcDrawSelect.SelectObject(poldpen);//恢复画笔 



pOldBrush.DeleteObject(); //







return TRUE; 

}
zgl7903 2009-08-13
  • 打赏
  • 举报
 回复
CBrush* pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(236,233,216))));
pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));
把语句拆散了写，不要偷懒
选入的要保留原始的句柄，使用完后恢复 创建的对象用完要删除
雪影 2009-08-13
  • 打赏
  • 举报
 回复
在可能出错的地方设置断点吧，单步调式
永远爱好写程序 2009-08-13
  • 打赏
  • 举报
 回复
出错时提示“必须的资源无法得到”
雪影 2009-08-13
  • 打赏
  • 举报
 回复
这里也没看到什么资源泄漏啊
你通过任务管理器查看GDI对象的变化，看看有无什么异常？
观察进程的内存，CPU、线程数、句柄数、GDI对象等等
永远爱好写程序 2009-08-13
  • 打赏
  • 举报
 回复
调用代码 onpaint

void CDragControlVC6View::OnPaint()
{
static BOOL bDraw=FALSE;
CPaintDC dc(this); // device context for painting
CClientDC dcClient(this);


if(bDraw)
return;

bDraw=TRUE;
POSITION pos;
pos=m_arrControls.GetHeadPosition();
CWnd* pWndControl;
while(pos!=NULL)
{
pWndControl=(CWnd*)m_arrControls.GetNext(pos);
CRect rectControl;
ASSERT(pWndControl);
pWndControl->GetWindowRect(&rectControl);
ScreenToClient(&rectControl);
BOOL bIsSelected;
bIsSelected=FALSE;
CRect rectControlOld;
if(pWndControl->IsKindOf(RUNTIME_CLASS(CButton)))
{
bIsSelected=((CButtonST*)pWndControl)->m_bMySelected;
rectControlOld=((CButtonST*)pWndControl)->m_rectControlOld;
}
else if(pWndControl->IsKindOf(RUNTIME_CLASS(CEdit)))
{
bIsSelected=((CMyEditBox*)pWndControl)->m_bMySelected;
rectControlOld=((CMyEditBox*)pWndControl)->m_rectControlOld;
}
if(bIsSelected)
{//这里调用了
DrawControlSelect(rectControl,rectControlOld,dcClient);
}
if(pWndControl->IsKindOf(RUNTIME_CLASS(CButton)))
{
((CButtonST*)pWndControl)->m_rectControlOld=rectControl;
}
else if(pWndControl->IsKindOf(RUNTIME_CLASS(CEdit)))
{
((CMyEditBox*)pWndControl)->m_rectControlOld=rectControl;
}
}

pos=m_arrControls.GetHeadPosition();
while(pos!=NULL)
{
pWndControl=(CWnd*)m_arrControls.GetNext(pos);
pWndControl->Invalidate();
}
//Invalidate();
//delete pWndControl;
pWndControl=NULL;

Sleep(1);

bDraw=FALSE;

// TODO: Add your message handler code here

// Do not call CFormView::OnPaint() for painting messages
}
永远爱好写程序 2009-08-13
  • 打赏
  • 举报
 回复
重新修改了一下，运行正常时间延长了一点，但还是出错
BOOL CDragControlVC6View::DrawControlSelect(const CRect& rectControl,const CRect& rectControlOld,CClientDC& dcDrawSelect)
{
//CClientDC dcDrawSelect(this);
CPen pen(0,1,RGB(236,233,216));
CPen* poldpen;

CBrush* pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(236,233,216))));
pOldBrush->DeleteObject();
poldpen=dcDrawSelect.SelectObject(&pen);
poldpen->DeleteObject();
//SetROP2(dcDrawSelect,R2_XORPEN);
CRect rectSelect1(CPoint(rectControlOld.left-10,rectControlOld.top-11),CPoint(rectControlOld.left,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);
rectSelect1.SetRect(CPoint(rectControlOld.right,rectControlOld.top-11),CPoint(rectControlOld.right+10,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);


pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));
pOldBrush->DeleteObject();
CRect rectSelect(CPoint(rectControl.left-10,rectControl.top-11),CPoint(rectControl.left,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
rectSelect.SetRect(CPoint(rectControl.right,rectControl.top-11),CPoint(rectControl.right+10,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
dcDrawSelect.SelectObject(pOldBrush);
dcDrawSelect.SelectObject(poldpen);//恢复画笔

return TRUE;
}
Fireway2008 2009-08-13
  • 打赏
  • 举报
 回复
pOldBrush->DeleteObject();
改为：
DeleteObject((HBRUSH)pOldBrush);

或者用CBrush* pOldBrush-》 HBRUSH hbr;

但是你已经选回了他，应该不是这个问题吧。
你怎么知道有泄漏的？
雪影 2009-08-13
  • 打赏
  • 举报
 回复
会不会是在其他的地方呢？
贴调用DrawControlSelect函数的代码
永远爱好写程序 2009-08-13
  • 打赏
  • 举报
 回复
修改后代码
BOOL CDragControlVC6View::DrawControlSelect(const CRect& rectControl,const CRect& rectControlOld,CClientDC& dcDrawSelect)
{
//CClientDC dcDrawSelect(this);
CPen pen(0,1,RGB(236,233,216));
CPen* poldpen;

CBrush* pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(236,233,216))));
poldpen=dcDrawSelect.SelectObject(&pen);
//SetROP2(dcDrawSelect,R2_XORPEN);
CRect rectSelect1(CPoint(rectControlOld.left-10,rectControlOld.top-11),CPoint(rectControlOld.left,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);
rectSelect1.SetRect(CPoint(rectControlOld.right,rectControlOld.top-11),CPoint(rectControlOld.right+10,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);
pOldBrush->DeleteObject();

pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));
CRect rectSelect(CPoint(rectControl.left-10,rectControl.top-11),CPoint(rectControl.left,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
rectSelect.SetRect(CPoint(rectControl.right,rectControl.top-11),CPoint(rectControl.right+10,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
dcDrawSelect.SelectObject(pOldBrush);
dcDrawSelect.SelectObject(poldpen);//恢复画笔
pen.DeleteObject();
pOldBrush->DeleteObject();
return TRUE;
}
MoXiaoRab 2009-08-13
  • 打赏
  • 举报
 回复
pOldBrush->DeleteObject();

太明显了。上面都说了，我就接分吧
永远爱好写程序 2009-08-13
  • 打赏
  • 举报
 回复
加了pOldBrush->DeleteObject(); 还是不行，修改后代码如下
BOOL CDragControlVC6View::DrawControlSelect(const CRect& rectControl,const CRect& rectControlOld,CClientDC& dcDrawSelect)
{
//CClientDC dcDrawSelect(this);
CPen pen(0,1,RGB(236,233,216));
CPen* poldpen;

CBrush* pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(236,233,216))));
poldpen=dcDrawSelect.SelectObject(&pen);
//SetROP2(dcDrawSelect,R2_XORPEN);
CRect rectSelect1(CPoint(rectControlOld.left-10,rectControlOld.top-11),CPoint(rectControlOld.left,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);
rectSelect1.SetRect(CPoint(rectControlOld.right,rectControlOld.top-11),CPoint(rectControlOld.right+10,rectControlOld.top+1));
dcDrawSelect.Rectangle(rectSelect1);
pOldBrush->DeleteObject();

pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));
CRect rectSelect(CPoint(rectControl.left-10,rectControl.top-11),CPoint(rectControl.left,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
rectSelect.SetRect(CPoint(rectControl.right,rectControl.top-11),CPoint(rectControl.right+10,rectControl.top+1));
dcDrawSelect.Rectangle(rectSelect);
dcDrawSelect.SelectObject(pOldBrush);
dcDrawSelect.SelectObject(poldpen);//恢复画笔
pen.DeleteObject();
return TRUE;
}
雪影 2009-08-13
  • 打赏
  • 举报
 回复
pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));
你的两端代码创建的画笔都没有释放，调用下面代码释放
pOldBrush->DeleteObject();
另外
pen.DeleteObject(); 不必要，pen的析构函数会释放
bohut 2009-08-13
  • 打赏
  • 举报
 回复
pOldBrush->DeleteObject();
雪影 2009-08-13
  • 打赏
  • 举报
 回复
pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(236,233,216))));
pOldBrush->DeleteObject();
pOldBrush=CBrush::FromHandle((HBRUSH)dcDrawSelect.SelectObject(CreateSolidBrush(RGB(255,0,0))));

写不动代码的人 2009-08-13
  • 打赏
  • 举报
 回复
pen.DeleteObject(); 这个不需要吧
Android应用程序隐私数据泄露检测.pdf
在检测过程中，源函数和锚函数的存在是检测隐私数据泄露的必要条件。当源函数和锚函数同时存在于应用程序时，检测系统开始分析数据流。 知识点5：组件间通讯（ICC） 组件间通讯（ICC）是指应用程序中的不同组件之间...
万恶的前端内存泄漏及万善的解决方案详解.docx
虽然别的地方删除了，但是对象中还存在对 dom 的引用，这也会导致内存泄露。 解决方法： * 手动删除，删除对 dom 的引用 四、被遗忘的定时器或者回调 定时器中有 dom 的引用，即使 dom 删除了，但是定时器还在，...
如何使用cmocka框架去测试某一个函数是否存在内存泄露问题
要使用cmocka框架来测试一个函数是否存在内存泄漏问题，你可以按照以下步骤进行： 确定需要测试的函数以及它所依赖的函数。 在测试用例中使用will_return函数来模拟函数的调用，以及它返回的结果。 在测试用例中...
频谱泄露与窗函数
频谱泄露与窗函数  今天，我想谈的是数字信号处理中的两个重要的问题，什么是频谱泄露，以及什么是窗函数，为什么使用窗函数。 文章目录频谱泄露与窗函数什么是频谱泄露一、频谱泄露是什么？二、窗函数的目的 ...
JS闭包 内存泄漏 匿名函数
1.闭包：有权访问另一个函数作用域中的变量的函数。 创建闭包的常见方式就是在一个函数内创建...1.闭包会使得函数中的变量都被保存在内存中，会增大内存消耗量，使用不当会导致内存泄露（解决方法就是在退出函数之...
VC/MFC

16,551

社区成员

421,606

社区内容

发帖
与我相关
我的任务
社区描述
VC/MFC相关问题讨论
社区管理员
  • 基础类社区
  • Creator Browser
  • encoderlee
加入社区
  • 近7日
  • 近30日
  • 至今

加载中

查看更多榜单
社区公告

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

试试用AI创作助手写篇文章吧

+ 用AI写文章