new delete 释放内存问题

[shiyunqiang]

代码如下：

	unsigned char* template;
	int template_length;

	CString DataStyle=_T("data(*.dat)|*.dat||");
	CFileDialog dlg(TRUE,NULL,FALSE,NULL,DataStyle,NULL);
	(dlg.m_ofn).lpstrTitle="open data";
	if (dlg.DoModal()==IDOK)
	{
		dlg.CenterWindow();
		CString m_strdataPath=dlg.GetPathName();
		FILE*fp;
		int i=0;
		fp=fopen(m_strdataPath,"r");
		if (fp==NULL)
		{
			AfxMessageBox(_T("Load data failed!"));
		}


		fscanf( fp, _T("%d\n"), &template_length);

		template=new unsigned char[template_length];

		while(!feof(fp))
		{
			fscanf( fp, _T("%d "), &template[i]);
			i++;
		}

		delete[] template;

	}

运行到delete[] template;就失败。

[kusey]

晕，我在三楼就提到这个问题了，怎么没给我分呀！哎......

[kusey]

估计是 template 申请的内存大小不够，你取得的 template_length 小于文件中数字个数

[shiyunqiang]

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

估计是 template 申请的内存大小不够，你取得的 template_length 小于文件中数字个数
[/Quote]

文件中200个数据，template_length获取的第一个数字是保存文件的数据量，我将template_length的数组扩大到10000也是释放不成功。

[kusey]

fscanf( fp, _T("%d "), &template[i]);

你定义的 unsigned char 类型的数组
存的是 int 型的数值

估计这块出错了

[sdpxpxp]

你debug跟踪到的fscanf( fp, _T("%d\n"), &template_length); 返回值是多少？

[rolandoo]

template=new unsigned char[template_length] 改成

template=new unsigned char[template_length+1];
再試看看

[dahaiI0]

是不是报 Heap corruption detected error，如果是可以按5楼的方法,个人习惯new了后memset下.

[kyotrue]

超界了，破坏了内存分配链。
scanf指定%d是读入整型，windows上整型是4字节的，不会管你传入的是int指针还是char指针，都会按照int来处理，用传入地址开始的4个字节来存储数据，因此最后一个循环时你就超界3字节了。写个例子试一下就知道了。

	unsigned char test[8] = { 0 };
	::_stscanf(_T("123456789"), _T("%d"), &test[3]);

可以先用一个int临时变量接收scanf结果，然后再将值赋给char数组。

[shiyunqiang]

[Quote=引用 7 楼 kyotrue 的回复:]

超界了，破坏了内存分配链。
scanf指定%d是读入整型，windows上整型是4字节的，不会管你传入的是int指针还是char指针，都会按照int来处理，用传入地址开始的4个字节来存储数据，因此最后一个循环时你就超界3字节了。写个例子试一下就知道了。
C/C++ code
unsigned char test[8] = { 0 };
::_stscanf(_T("1234……
[/Quote]

的确是这个问题，谢谢kyotrue！
同时对参与的朋友表示感谢！

[shiyunqiang]

[Quote=引用 9 楼 kusey 的回复:]

晕，我在三楼就提到这个问题了，怎么没给我分呀！哎......
[/Quote]

昨天结贴时忘记给你加分了，实在不好意思，今天重新开了一个加分贴，你回复一下，我给你加分。
连接如下：
http://topic.csdn.net/u/20120308/09/ebaac4eb-43cb-4b75-bf78-8f8edcc180be.html?23167