利用(unsigned long)（new CString）将字符串转换为unsigned long型会不会造成内存泄露？

[matrix2009]

如题

我找到一段代码，是这样转的

const char* szInstallID = "TAPI\ACPI0003\1";

unsigned long tmp = (unsigned long)(new CString(szInstallID ));

结果 tmp 为3690472.

我想知道 用new CString转换会不会在堆上分配内存？如果不显示释放会不会造成内存泄露？
还有，用new CString转换是如何实现的？

另外还有没有其它的办法转换字符串？

[The_facE]

unsigned long tmp = (unsigned long)(new CString(szInstallID));
// 这一行相当于：
unsigned long tmp = (unsigned long)0;  // 声明一个unsigned long型数据tmp
CString * pointer = new CString(szInstallID);  // 在堆上构造一个CString对象，并把其地址保存在pointer指针中
tmp = (unsigned long)pointer;  // 将指针pointer的地址值转换为unsigned long类型，并赋值给tmp

[skywoodsky]

new CString会在堆上分配内存
不用delete释放会造成内存泄露
CString str(szInstallID);直接通过构建函数就可以转换了

[似水流年__]

[Quote=引用 2 楼 skywoodsky 的回复:]
new CString会在堆上分配内存
不用delete释放会造成内存泄露
CString str(szInstallID);直接通过构建函数就可以转换了
[/Quote]

CDialog *pDlg = new CDialog();呢?用不用 delete pDlg;

[linan161051]

new CString会在堆上分配内存
不用delete释放会造成内存泄露
CString str(szInstallID);直接通过构建函数就可以转换

[哈利路亚1874]

[Quote=引用 3 楼 xhk456 的回复:]
引用 2 楼 skywoodsky 的回复:
new CString会在堆上分配内存
不用delete释放会造成内存泄露
CString str(szInstallID);直接通过构建函数就可以转换了


CDialog *pDlg = new CDialog();呢?用不用 delete pDlg;
[/Quote]
当然要，在最后退出的时候清理

[写不动代码的人]

new 和delete 是配对使用的

[matrix2009]

源代码是在堆上分配内存，然后设置一个树形控件的值，代码如下，
m_ctrlTree.SetItemData(hItem,(unsigned long)(new CString(szID)));

我该如何释放内存？

我想写一个遍历树的函数，然后释放分配的内存

void CDeviceCtrlDlg::TraverTree(HTREEITEM hItem)
{
hItem = m_ctrlTree.GetChildItem(hItem);
if(NULL != hItem)
{
TraverTree(hItem);
}
else
{
HeapFree(GetProcessHeap(), 0, (LPVOID)m_ctrlTree.GetItemData(hItem));
}

hItem = m_ctrlTree.GetNextSiblingItem(hItem);
if(NULL != hItem)
{
TraverTree(hItem);
}
}

这样可以吗？

还是必须用delete?

[Ryanwen]

还是用delete吧 用CString的析构函数来释放空间

[webipstin]

把CString看成是普通的对象来对待就行了。

new--delete

new [] --delete []pPointer

[zhouzhipen]

unsigned long tmp = (unsigned long)(new CString(szInstallID ));
这样不用delete清除肯定会内存泄露
CString对象实际上也可以看作是指向数据的指针,但CString对象在析构时对内存做了清理释放,所以不是动态分配的对象不会内存泄露.
但是动态分配的CString对象,在程序运行期间,一直存在,不会调用析构函数,而且CString对象本身所占用的指针空间也得不到释放.

[matrix2009]

刚才写的代码有问题
这样释放就可以了吧
void CDeviceCtrlDlg::TraverTree(HTREEITEM hItem)
{
hItem = m_ctrlTree.GetChildItem(hItem);
if(NULL != hItem)
{
HTREEITEM hParent = m_ctrlTree.GetParentItem(hItem);
if ( hParent != hRoot )
{
CString* tmp = (CString*)m_ctrlTree.GetItemData(hItem);
delete tmp;
}
TraverTree(hItem);
}

hItem = m_ctrlTree.GetNextSiblingItem(hItem);
if(NULL != hItem)
{
TraverTree(hItem);
}
}

[oldn123]

CString* tmp = (CString*)m_ctrlTree.GetItemData(hItem);
delete tmp;
_______________
看着真别扭。。
ps:楼主的头像可以吸引我的目光..

[oldn123]

这样写是对的。
只是看着别扭，至于为什么？我还不太好说了，有两点吧。
第一感觉CString *这个形式似乎并不多见..（个人感觉）.不过看你的上下文代码这样用倒也说得过去。
第二delete 指针时最好先判断下指针的有效性.
if(tmp) delete tmp; tmp = NULL;

[matrix2009]

to oldn_cc_bird:

CString* tmp = (CString*)m_ctrlTree.GetItemData(hItem);
delete tmp;

我写的不对吗？为什么看着别扭？你觉得应该怎么写？

[matrix2009]

谢谢