OnPaint里的new后内存怎么办，在线等！！！

[ningto.com]

在OnPaint里画图需要new一个写好了的画图的类，现在有两个问题：
部分代码如下：

LRESULT OnPaint(UINT /*uMsg*/, WPARAM /*wParam*/, LPARAM /*lParam*/, BOOL& /*bHandled*/)
	{
		PAINTSTRUCT ps;

		HDC hdc = BeginPaint(&ps);

		m_spObj = new CSpScheme(hdc);
		
		m_spObj->DrawBk(...);
                    ...
                    ...             
                EndPaint(&ps);
                return 0;
         }

1>我上面这样写的话运行是好好的，但是会不会造成内存泄露啊？为什么？
2>还有如果在返回之前delete的话，运行会出问题的，希望高手能详细解释一下，感觉OnPaint里new有些特殊，毕竟这个函数窗口一重绘就要执行的

[tompaz]

当然会出问题，直接用栈变量试试
new是统一的，不可能在哪个函数里不一样

[ningto.com]

[Quote=引用 1 楼 tompaz 的回复:]

当然会出问题，直接用栈变量试试
new是统一的，不可能在哪个函数里不一样
[/Quote]
m_spObj这个是指针类型的成员变量，用栈会造成其他问题，最好不用栈，在我那个基础上最好是添代码，不要改代码。

[whuhhz]

那肯定内存会泄露了，每次刷新都会new CSpScheme(hdc)，如果m_spObj是类的成员变量，在构造函数中
m_spObj = NULL,析构函数中if (m_spObj) delete m_spObj; m_spObj = NULL;

你的OnPaint中，改为if (!m_spObj) m_spObj = new CSpScheme(hdc);就不会有内存泄露了。

[Eleven]

[Quote=引用 2 楼 tujiaw 的回复:]
引用 1 楼 tompaz 的回复:

当然会出问题，直接用栈变量试试
new是统一的，不可能在哪个函数里不一样

m_spObj这个是指针类型的成员变量，用栈会造成其他问题，最好不用栈，在我那个基础上最好是添代码，不要改代码。
[/Quote]
那你的new放在外面，不要放到OnPaint中，不需要的地方在delete

[向立天]

delete会有什么问题

[一桶姜山]

在构造函数中：m_spObj = NULL;

在OnPaint中：
if(m_spObj==0)m_spObj = new CSpScheme(hdc);

[ningto.com]

[Quote=引用 4 楼 visualeleven 的回复:]

引用 2 楼 tujiaw 的回复:
引用 1 楼 tompaz 的回复:

当然会出问题，直接用栈变量试试
new是统一的，不可能在哪个函数里不一样

m_spObj这个是指针类型的成员变量，用栈会造成其他问题，最好不用栈，在我那个基础上最好是添代码，不要改代码。

那你的new放在外面，不要放到OnPaint中，不需要的地方在delete
[/Quote]
放在外面我传不进去DC

[ningto.com]

[Quote=引用 5 楼 xianglitian 的回复:]

delete会有什么问题
[/Quote]
我画图的这个对话框是点击按钮后弹出来的模态对话框，如果加delete的话，我一点按钮就报：
Unhandled exception at 0xxxxxxx...: User breakpoint.

[ningto.com]

[Quote=引用 3 楼 whuhhz 的回复:]

那肯定内存会泄露了，每次刷新都会new CSpScheme(hdc)，如果m_spObj是类的成员变量，在构造函数中
m_spObj = NULL,析构函数中if (m_spObj) delete m_spObj; m_spObj = NULL;

你的OnPaint中，改为if (!m_spObj) m_spObj = new CSpScheme(hdc);就不会有内存泄露了。
[/Quote]
我也试过这样的，可是图被其他窗口覆盖后就不重绘了，变成了空白

[alexander_david]

不妨检查一下CSpScheme的析构函数。

[向立天]

[Quote=引用 8 楼 tujiaw 的回复:]

引用 5 楼 xianglitian 的回复:

delete会有什么问题

我画图的这个对话框是点击按钮后弹出来的模态对话框，如果加delete的话，我一点按钮就报：
Unhandled exception at 0xxxxxxx...: User breakpoint.
[/Quote] CSpScheme是什么？

[shiyunqiang]

界面没更新一次就调用一次OnPaint函数，就需要创建一次类，有没有及时删除，发生泄漏是必然的。

[ningto.com]

[Quote=引用 10 楼 alexander_david 的回复:]

不妨检查一下CSpScheme的析构函数。
[/Quote]
确是这个析构函数有问题，问一下这个怎么析构：

成员变量：
	wchar_t		*m_szInitials[KEY_COUNT];
	wchar_t		*m_szFinals1[KEY_COUNT];
	wchar_t		*m_szFinals2[KEY_COUNT];

初始化：
for (int i = 0; i < KEY_COUNT; i++)
		{
			m_szInitials[i] = (wchar_t *)malloc(sizeof(wchar_t) * KEY_LEN);
			memset(m_szInitials[i], 0, sizeof(m_szInitials[i]));
			m_szFinals1[i] = (wchar_t *)malloc(sizeof(wchar_t) * KEY_LEN);
			memset(m_szFinals1[i], 0, sizeof(m_szFinals1[i]));
			m_szFinals2[i] = (wchar_t *)malloc(sizeof(wchar_t) * KEY_LEN);
			memset(m_szFinals2[i], 0, sizeof(m_szFinals2[i]));
		}

请问那三个成员变量怎么析构掉？
我下面两种都试了不行：

1>
		free(m_szInitials);
		free(m_szFinals1);
		free(m_szFinals2);
2>
for (int i = 0; i < KEY_COUNT; i++)
		{
			if (m_szInitials[i])
			{
				free(m_szInitials[i]);
				m_szInitials[i] = NULL;
			}
			if (m_szFinals1[i])
			{
				free(m_szFinals1[i]);
				m_szFinals1[i] = NULL;
			}
			if (m_szFinals2[i])
			{
				free(m_szFinals2[i]);
				m_szFinals2[i] = NULL;
			}
		}

[CSharp_XinBing]

OnDraw里最好只是使用数据，而不生成数据最好。
生成改变数据在其它函数中。

[ningto.com]

13楼那个问题怎么释放掉啊

[alexander_david]

你的那个初始化函数是在构造函数内调用的吗？还有那个memset的第三个参数不对的，
sizeof(m_szInitials[i]));返回的值是4，这样只将前4个字节置零。

memset(m_szInitials[i], 0, sizeof(wchar_t) * KEY_LEN);

你的第二个析构函数我觉得是没有问题的。

[ningto.com]

[Quote=引用 16 楼 alexander_david 的回复:]

你的那个初始化函数是在构造函数内调用的吗？还有那个memset的第三个参数不对的，
sizeof(m_szInitials[i]));返回的值是4，这样只将前4个字节置零。

memset(m_szInitials[i], 0, sizeof(wchar_t) * KEY_LEN);

你的第二个析构函数我觉得是没有问题的。
[/Quote]
你说的是对的，不过析构函数还是有问题

[alexander_david]

第二个析构函数还有问题？不明白了。

[ningto.com]

测试了第二个析构函数没问题，是其他问题，结贴了