为什么运行到释放空间就被中断，释放错误？

[wensheng_zh2007]

在mfc的一个dialog中

在类的声明中
CButton* p_MyBut[3];


在实现中
p_MyBut[0] = new CButton();
p_MyBut[1] = new CButton();
p_MyBut[2] = new CButton();

在这个dialog的重载OnClose()中


if(p_MyBut[0] != NULL)
{
delete p_MyBut[0];//运行到这个地方就被中断了，难道内存释放错误，请高手指点！
p_MyBut[0] = NULL;
}

if(p_MyBut[1] != NULL)
{
delete p_MyBut[1];
p_MyBut[1] = NULL;
}

if(p_MyBut[2] != NULL)
{
delete p_MyBut[2];
p_MyBut[2] = NULL;
}

[supconsupcon]

p_MyBut是不是被delete多次了？？？

[zgl7903]

有可能是CButton的 PostNcDestroy()中已经使用了 delete this操作
if(p_MyBut[0] != NULL)
{
//delete p_MyBut[0];//直接屏蔽试试看如果没有报内存泄漏 就说明已经释放过了
p_MyBut[0] = NULL;
}

[zhouzhipen]

在delete之前可能有正在使用p_MyBut的情况,先DestroyWindow()再删除,试试

[oyljerry]

p_MyBut[0]->DestroyWindow();

[pady_pady]

应该在对话框析构函数中delete

CxxDlg::~Cxxdlg()
{
    for(int i = 0;i<3;++i)
      if(p_MyBut[i] != NULL)
          delete p_MyBut[i];
}

[biweilun]

[Quote=引用 2 楼 oyljerry 的回复:]
p_MyBut[0]->DestroyWindow();
[/Quote]
可行。

我觉得你是有些代码没贴出来，这个一般不会有问题的

[写不动代码的人]

DestroyWindow();

[dengxuxing]

[Quote=引用 3 楼 pady_pady 的回复:]
应该在对话框析构函数中delete

C/C++ code
CxxDlg::~Cxxdlg()
{
for(int i = 0;i<3;++i)
if(p_MyBut[i] != NULL)
delete p_MyBut[i];
}
[/Quote]

同意!

[supconsupcon]

在析构函数调用delete最保险

[pady_pady]

CButton::~CButton()
{
DestroyWindow();
}
你的程序是不是在别的地方自作聪明用了p_MyBut[]->DestroyWindow();?
这样当你delete p_MyBut[0];会再运行DestroyWindow,断言出错的,

[pady_pady]

[Quote=引用 11 楼 pady_pady 的回复:]
说错不好意思;
CButton::~CButton()
{
DestroyWindow();
}
你的程序是不是在别的地方自作聪明用了p_MyBut[]->DestroyWindow();?
这样当你delete p_MyBut[0];会再运行DestroyWindow,断言出错的,
[/Quote]

BOOL CWnd::DestroyWindow()
{
CWnd* pWnd;
CHandleMap* pMap;
HWND hWndOrig;
BOOL bResult;

if ((m_hWnd == NULL) && (m_pCtrlSite == NULL))
return FALSE;...
}
它会自动判断窗口的呵呵,可能是你的BUTTON还没有create或者是多次delete了吧

[pady_pady]

试试一个简单的按钮:

	RECT rect =  { 10,10,50,50};
        p_MyBut[0]->Create(L"dd",WS_VISIBLE,rect,this,IDYES);
	p_MyBut[0]->ShowWindow(SW_SHOW);

[wensheng_zh2007]

我定义的类只是继承了CButton，再就是添加了一个响应事件。应该没有问题。

我不用我定义的类，直接用CButton也是不可以

[gg606]

如你所说，我做了一个实验，没问题呀！

[wensheng_zh2007]

[Quote=引用 10 楼 zgl7903 的回复:]
有可能是CButton的 PostNcDestroy()中已经使用了 delete this操作
if(p_MyBut[0] != NULL)
{
//delete p_MyBut[0];//直接屏蔽试试看如果没有报内存泄漏 就说明已经释放过了
p_MyBut[0] = NULL;
}

[/Quote]
这样在输出中显示如下：
d:\my documents\visual studio 2005\projects\dynamiccreatecontrol\dynamiccreatecontroldlg.cpp(169) : {166} client block at 0x003BFAD0, subtype c0, 84 bytes long.
a CMyButton object at $003BFAD0, 84 bytes long
d:\my documents\visual studio 2005\projects\dynamiccreatecontrol\dynamiccreatecontroldlg.cpp(169) : {165} client block at 0x003BFA40, subtype c0, 84 bytes long.
a CMyButton object at $003BFA40, 84 bytes long
d:\my documents\visual studio 2005\projects\dynamiccreatecontrol\dynamiccreatecontroldlg.cpp(169) : {163} client block at 0x003BF9B0, subtype c0, 84 bytes long.
a CMyButton object at $003BF9B0, 84 bytes long


这不就表示内存泄露了？

[pady_pady]

你先把你的BUTTON不用,用我的IDYES/IDNO/IDABORT button试试
RECT rect = { 10,10,50,50};
p_MyBut[0]->Create(L"dd",WS_VISIBLE,rect,this,IDYES);
p_MyBut[0]->ShowWindow(SW_SHOW);
如果用这三个按钮正常,可能问题出在你的BUTTON上了

[wensheng_zh2007]

[Quote=引用 15 楼 pady_pady 的回复:]
你先把你的BUTTON不用,用我的IDYES/IDNO/IDABORT button试试
RECT rect =  { 10,10,50,50};
p_MyBut[0]->Create(L"dd",WS_VISIBLE,rect,this,IDYES);
p_MyBut[0]->ShowWindow(SW_SHOW);
如果用这三个按钮正常,可能问题出在你的BUTTON上了

[/Quote]

[pady_pady]

a CMyButton object at $003BF9B0, 84 bytes long
CMyButton类有问题,楼主是不是自定义了一个BUTTON类,中间处理有问题吧

[wensheng_zh2007]

[Quote=引用 19 楼 gg606 的回复:]
如你所说，我做了一个实验，没问题呀！
[/Quote]
我在另一台机器也试过，也是没有问题！

[Ghost90]

全部从新编译程序
Rebuild All