大家看看我释放内存的方法对不对？？？

[wangsiyuanoo]

程序一共就NEW了两个地方
不知道是释放的方法错了还是什么
依然有内存泄露！

//第一个NEW
ExeInfo * pExeInfo = new ExeInfo; //！
pExeInfo->strName = SExeInfoIni.ReadText("info", "Name");
m_ExeInfoList.push_back(pExeInfo);

//释放
if (m_ExeInfoList.size() > 0)
{
	for (vector<ExeInfo*>::iterator iter = m_ExeInfoList.begin(); iter != m_ExeInfoList.end(); iter++)
	{
		delete * iter;
	}
}
m_ExeInfoList.clear();

//第二个NEW
g_strGamePathRes = new char[i+1];  //！

//释放
delete []g_strGamePathRes;

Detected memory leaks!
Dumping objects ->
{178} normal block at 0x005091C8, 1 bytes long.
Data: < > 00
{177} normal block at 0x00509188, 1 bytes long.
Data: < > 00
{174} normal block at 0x00508EE8, 1 bytes long.
Data: < > 00
{169} normal block at 0x00509018, 1 bytes long.
Data: < > 00
{168} normal block at 0x00508FD8, 1 bytes long.
Data: < > 00
{165} normal block at 0x00508EA8, 1 bytes long.
Data: < > 00
Object dump complete.

[lockhall]

pExeInfo 这个指针没释放啊

delete pExeInfo;

[lbh2001]

pExeInfo->strName = SExeInfoIni.ReadText("info", "Name");

pExeInfo中strName是指针吧，所指的内存释放了吗

[deerwin1986]

一个NEW对一个DELETE
循环里的NEW对循环里的DELETE
NEW数组对DELETE数组

[hhyttppd]

只是一个byte,应该不会是这两个地方，楼主再查下其它地方吧。

[syd_programming]

同意2楼的，是strName指向内存没释放吧？

[wangsiyuanoo]

[Quote=引用 2 楼 lbh2001 的回复:]
pExeInfo->strName = SExeInfoIni.ReadText("info", "Name");

pExeInfo中strName是指针吧，所指的内存释放了吗
[/Quote]
strName不是指针啊，他只是我定义的一个结构里的成员变量。
pExeInfo->strName = SExeInfoIni.ReadText("info", "Name"); //意思是从配置文件中读一个字符串然后赋给strName

[wangsiyuanoo]

[Quote=引用 1 楼 lockhall 的回复:]
pExeInfo 这个指针没释放啊

delete pExeInfo;
[/Quote]

难道
for (vector<ExeInfo*>::iterator iter = m_ExeInfoList.begin(); iter != m_ExeInfoList.end(); iter++)
{
delete * iter;
}
这个不是释放pExeInfo 的方法么？

[lbh2001]

从配置文件中读一个字符串然后赋给strName--->字符串怎么赋给strName，字符串是要拷贝才能赋给的，除非是指针

[Allen_zhang]

只有一个字节的泄漏，应该不是这里的问题
看看vector这个会不会有问题

或者用BoundsChecker查一下看看

[wangsiyuanoo]

[Quote=引用 8 楼 lbh2001 的回复:]
从配置文件中读一个字符串然后赋给strName--->字符串怎么赋给strName，字符串是要拷贝才能赋给的，除非是指针
[/Quote]
额.....比如配置文件里是Name="adadadada"
我用ReadText去读他，读出来后再赋给string strName
其实就是

string name = SExeInfoIni.ReadText("info", "Name"); //不过ReadText返回的是char *,但可以直接用string去赋值的。不知道这里是不是你说的问题所在！
string strName = name;

[gongnai]

楼主, 能否是你的 class ExeInfo 里面的析构函数

写的时候有内存泄露呢?

你的在vector 里面 delete * itor 的时候 应该是

调用你的 析构函数吧? 如果你的析构函数写的不好的好.

就容易内存泄露了. 是不是?

望大家批评指正!

[ypb362148418]

[Quote=引用 1 楼 lockhall 的回复:]
pExeInfo 这个指针没释放啊

delete pExeInfo;
[/Quote]

//支持，正确

[jackzhhuang]

pExeInfo 内存释放了

但可能是ExeInfo这个类有内存泄露，仔细检查成员数据有没有可能会造成内存泄露。

ps：
if (m_ExeInfoList.size() > 0)
多余

[shyli]

是不是SExeInfoIni执行ReadText时有new char[]但没有delete[]？

[yueyucanyang]

[Quote=引用 6 楼 wangsiyuanoo 的回复:]
引用 2 楼 lbh2001 的回复:
pExeInfo->strName = SExeInfoIni.ReadText("info", "Name");

pExeInfo中strName是指针吧，所指的内存释放了吗

strName不是指针啊，他只是我定义的一个结构里的成员变量。
pExeInfo->strName = SExeInfoIni.ReadText("info", "Name"); //意思是从配置文件中读一个字符串然后赋给strName
[/Quote]

正常情况下，这个应该是没有问题的，string一般没有内存泄露的问题！有可能是程序别的地方，或者是上释放了一个指针。

[waizqfor]

UP下吧

[yytlan]

我觉得楼主发贴的时候,应该发全了,要不然就不有这样多的观点了.

[wangsiyuanoo]

[Quote=引用 13 楼 jackzhhuang 的回复:]
pExeInfo 内存释放了

但可能是ExeInfo这个类有内存泄露，仔细检查成员数据有没有可能会造成内存泄露。

ps：
if (m_ExeInfoList.size() > 0)
多余
[/Quote]

ExeInfo是一个我自定义的结构的，没有析构，成员变量也就是string、int类型的