内存泄露问题

[FireBirdCN]

有谁用过 IconPicker这个类不 是网上有人写的
Debug版本没问题
Release版本 有内存泄露 但我总是改不好 郁闷啊
高手们能不能帮忙看一下
附件传不上去啊~~- -！
给个链接 http://www.programsalon.com/downloads25/sourcecode/windows/control/combobox/detail81595.html
达人也可以留个Email我发给你
在此先谢谢了~~

[FireBirdCN]

楼上第一段代码太少了 补贴

void CIconPicker::OnLButtonDown(UINT nFlags, CPoint point) 
{
	if(m_bState) return ;
	m_bState=TRUE;
	this->SetState(TRUE);
	

	RECT rect;
	this->GetWindowRect(&rect);
	
	POINT pt;
	pt.x=rect.left;pt.y=rect.bottom;

	m_pIconContainer=new CIconContainer;//这句有泄漏 但不知道在哪里释放 CIconContainer是一个容器类
	if(m_pIconContainer->Create(pt,this,&m_BitmapArray))
	{
		m_pIconContainer->ShowWindow(SW_SHOW);
		m_pIconContainer->UpdateWindow();
		m_pIconContainer->SetFocus();
	}
}

[FireBirdCN]

[Quote=引用 9 楼 happyparrot 的回复:]
m_pIconContainer=new CIconContainer;//这句有泄漏 但不知道在哪里释放 CIconContainer是一个容器类
===这要看你这个对象的作用域了。如果是全局的，那么在系统退出时释放。如果在类中，那么在类析构时释放。
[/Quote]
m_pIconContainer是在IconPicker.h中声明的

CIconContainer *m_pIconContainer;
是不是在IconPicker类的 析构函数中就可以了？

[快乐鹦鹉]

m_InnerButtonArray.Add(pInnerButton);
delete pInnerButton; //在这里释放后 CArray里好像就没有了~~ 不知道怎么做
==========
在这里释放肯定是不行的。因为你加入数组的目的，是被其它地方调用。所以，你应该在m_InnerButtonArray变量所在类的析构函数中使用循环释放Array中的每个指针对象

[FireBirdCN]

[Quote=引用 7 楼 scq2099yt 的回复:]
昨晚我也遇到一个Debug没问题而Release有问题的情况
最后我将程序中的ASSERT改为VERIFY就没问题了。
[/Quote]
全部替换了 问题依旧
T_T

[FireBirdCN]

[Quote=引用 12 楼 happyparrot 的回复:]
m_InnerButtonArray.Add(pInnerButton);
delete pInnerButton; //在这里释放后 CArray里好像就没有了~~ 不知道怎么做
==========
在这里释放肯定是不行的。因为你加入数组的目的，是被其它地方调用。所以，你应该在m_InnerButtonArray变量所在类的析构函数中使用循环释放Array中的每个指针对象
[/Quote]
是这样释放的
CIconContainer::~CIconContainer()
{
for(int i=0;i<m_InnerButtonArray.GetSize();i++)
{
CInnerButton* pInnerButton;
pInnerButton=m_InnerButtonArray.GetAt(i);
pInnerButton->SendMessage(WM_CLOSE,0,0);
delete pInnerButton;
}

}

[rageliu]

CIconPicker这个类的析构函数中释放

[FireBirdCN]

happyparrot
加下QQ吧 请教下
或者我把这个类发给您 帮忙看一下
我QQ 23795505

[快乐鹦鹉]

发过来的程序能运行么？

[FireBirdCN]

[Quote=引用 15 楼 rageliu 的回复:]
CIconPicker这个类的析构函数中释放
[/Quote]
是这样释放的 ，不知对不对
CIconPicker::~CIconPicker()
{
CBitmap* pBitmap;
for(int i=0;i<m_BitmapArray.GetSize();i++)
{
pBitmap=m_BitmapArray.GetAt(i);
pBitmap->DeleteObject();
}
delete m_pIconContainer;
}

[快乐鹦鹉]

发到信箱[e-mail]

[FireBirdCN]

[Quote=引用 17 楼 happyparrot 的回复:]
发过来的程序能运行么？
[/Quote]
能运行的
可以把整个代码发给您
环境是VC6

[rageliu]

老大出手了，坐等分享经验

[FireBirdCN]

[Quote=引用 19 楼 happyparrot 的回复:]
发到信箱[e-mail]
[/Quote]
已经发送

[cnzdgs]

代码写的太……，主要问题出在CIconPicker::OnSelectICON的参数定义上，把定义改为：
afx_msg void OnSelectICON(WPARAM wParam, LPARAM lParam);

void CIconPicker::OnSelectICON(WPARAM wParam, LPARAM lParam)
{
int index = (int)wParam;
……

[rageliu]

哈哈！！楼上的意思是说楼主消息处理函数的申明未带2个参数？？导致了release下的堆栈不平，挂了

[快乐鹦鹉]

已经改完了，泄漏全部去除
请接收邮件

[FireBirdCN]

[Quote=引用 23 楼 cnzdgs 的回复:]
代码写的太……，主要问题出在CIconPicker::OnSelectICON的参数定义上，把定义改为：
afx_msg void OnSelectICON(WPARAM wParam, LPARAM lParam);

void CIconPicker::OnSelectICON(WPARAM wParam, LPARAM lParam)
{
int index = (int)wParam;
……
[/Quote]
果然解决了~~~
检查了好久的泄露，原来是这里的问题。
尴尬~
但为什么现在用BoundChecker 看到了还是那些Memory leak程序却没问题了
是不是有一些小的内存泄露，可以忽略不计的？
还是说 这些也是程序的隐患？

[快乐鹦鹉]

都是新手常见的问题。对指针还不能很好的驾驭。
1。局部变量进行new，函数退出后成无头之鬼。
2。CIconPicker类的析构函数并没有被执行。需要显示的增加释放对象的函数，由主界面的OnClose调用。
3。如下代码：
CIconPicker::~CIconPicker()
{
CBitmap* pBitmap;
for(int i=0;i <m_BitmapArray.GetSize();i++)
{
pBitmap=m_BitmapArray.GetAt(i);
pBitmap->DeleteObject();
}
delete m_pIconContainer;
}
pBitmap对象并没有被delete，调用DeleteObject只是删除该类中绑定的图像句柄，但这个类对象本身没有释放。

[FireBirdCN]

[Quote=引用 25 楼 happyparrot 的回复:]
已经改完了，泄漏全部去除
请接收邮件
[/Quote]
恩 用BoundChecker 看了下 没有泄露了 ^_^
但Release版本关闭还是有问题，用了cnzdgs 的方法，程序关闭也没有错了
搞了半天 原来不是内存泄露引起的那个关闭错误
不过正好我也学习了内存泄露的相关知识
谢谢啦~~
结贴去

[老窝瓜]

努力学习中,感谢大家!