二分合并排序

算法课程实验一。对其中出现的问题做一下记录。
VS2010 

原始代码（错误）

#include<iostream>
using namespace std;
class SortableList{
private:
	int *l;
	int maxSize;
	int n;
	void MergeSort(int left,int right);
	void Merge(int left,int mid,int right);
public:
	SortableList(int mSize){
		maxSize=mSize;
		l=new int[maxSize];
		//n=0;  //数组中已有元素个数
	}
	~SortableList(){
		delete[]l;
	}
	void Input();
	void Output();
	void MergeSort();

};
void SortableList::Input(){
		for(int i=0;i<maxSize;i++)
		cin>>l[i];
}
void SortableList::Output(){
	for(int i=0;i<maxSize;i++)
		cout<<l[i]<<" ";
}
void SortableList::MergeSort(){
	MergeSort(0,maxSize-1);
}
void SortableList::MergeSort(int left,int right){ //将数组通过二分法分解
	if(left<right){
		int mid=(left+right)/2;
		MergeSort(left,mid);
		MergeSort(mid+1,right);
		Merge(left,mid,right);
	}
}
void SortableList::Merge(int left,int mid,int right){  //两两合并
	int *temp=new int(right-left+1);
	int i=left,j=mid+1,k=0;
	while(i<=mid&&j<=right)
		if(l[i]<=l[j]) temp[k++]=l[i++];
		else temp[k++]=l[j++];
	while(j<=right) temp[k++]=l[j++];
	while(i<=mid) temp[k++]=l[i++];
	for(k=left,i=0;k<=right;) l[k++]=temp[i++];
}

int main(){
	int n;
	cout<<"待排序数列的长度：";
	cin>>n;
	SortableList list(n);
	cout<<"待排序序列：";
	list.Input();
	list.MergeSort();
	list.Output();
	return 0;
}

第一次报错

提示
Expression:_Crtls ValidHeapPointer(pUserData)
For information on how your program can cause an assertion failure, see the Visual C++ documentation on asserts.
(Press Retry to debug the application)

查找百度
说法一：
ValidHeapPointer意思是合法的堆的指针，所以这个错误是由于堆引起的。比如delete[]p；有问题。
说法二：
假设有int *p=new int[234]; delete[]p;而报错是在delete处，检查是不是由于程序半途中更改了p的指向，导致释放时为空，而报错。
说法三：
内存不能跨模块地获取和释放。比如在DLL申请一个内存，但是最后不是由DLLfree或者delete，而是由主程序exe来执行，那么就会报错。属于堆栈出错。一般都是指针，Vactor集合没有释放出错。
尝试一
首先排除说法二，因为我是在return 0；时候报错的。由于我对内存动态分配问题不是很清楚，而且将近一年没有用过c++，构造析构函数相关知识也遗忘了。所以我以为是因为问题出在析构函数。

	~SortableList(){
		delete[]l;
	}

尝试1.1
在主函数return前加上list.~SortableList();
报错变为EXPRESSION:_BLOCK_TYPE_IS_VALID(pHead->nBlockUse )（见下面报错二）

尝试1.2
在Output()内部最后一行加上delete[]l;
报错变为EXPRESSION:_BLOCK_TYPE_IS_VALID(pHead->nBlockUse ) （见下面报错二）

分析
以上两个修改本意都是类似的，感觉是不是“ l=new int[maxSize];”没有释放。其实是释放了的。
析构函数作用是在最后一次使用了对象之后释放对象的内存资源。如果你自己在类中定义了析构函数，结束时会调用这个函数，并在释放对象前执行你定义的析构函数中的任务。所以上面的代码，在执行到return 0；时调用析构函数，先释放掉l的内存资源，然后再释放掉SortableList的内存资源。

第二次报错

提示
EXPRESSION:_BLOCK_TYPE_IS_VALID(pHead->nBlockUse )

查找百度
说法一：
出现这种问题一般都是同一个内存释放了两次。
说法二：
内存块的类型不对。比如说用delete来释放了栈上的内存vector。然而delete只能释放堆上的内存，栈上内存是不需要手动释放的。

尝试二
研究后发现，这必须是因为int *temp=new int(right-left+1);这句话里面的temp没有释放啊！

尝试2.1
在这个void SortableList::Merge(int left,int mid,int right)函数里面最下面加了一句delete[]temp;
报错变为HEAP CORRUPTION DETECTED

第三次报错

提示
HEAP CORRUPTION DETECTED

查询百度
说法一：
HEAP CORRUPTION DETECTED，检查到堆被破坏了。如果输入的数据大于了你规定的大小，比如申请了长度为10的字符串，但是输入了11个（注意！字符串长度末尾有‘\0’），那么超出长度的数据会被覆盖。这就是HEAP CORRUPTION
例如：

char lnstr[11]="0123456789";
char* str=(char*)malloc(sizoef(char)*strln(lnstr)); 
strcpy(str,lnstr);
free(str);

free（str）这一步会报错。
因为lnstr实际上还有一个‘\0’，有11位，但是str只申请了10位。(strln（）不会把‘\0’计算在内)拷贝时超出了str的范围，让str指针损害，free时就会报错。

说法二：
new char()写成了new char[],没有分配合适的内存空间。
醍醐灌顶啊！然后我把int *temp=new int(right-left+1);中的()改成了[],就成功了。

• 补充：char *p=new char(5)开辟一个内存单元，并用括号里的数初始化，而char *p=new char[5]是申请一个内存单元为5的数组。

正确代码

#include<iostream>
using namespace std;
class SortableList{
private:
	int *l;
	int maxSize;
	int n;
	void MergeSort(int left,int right);
	void Merge(int left,int mid,int right);
public:
	SortableList(int mSize){
		maxSize=mSize;
		l=new int[maxSize];
		//n=0;  //数组中已有元素个数
	}
	~SortableList(){
		delete[]l;
	}
	void Input();
	void Output();
	void MergeSort();

};
void SortableList::Input(){
		for(int i=0;i<maxSize;i++)
		cin>>l[i];
}
void SortableList::Output(){
	for(int i=0;i<maxSize;i++)
		cout<<l[i]<<" ";
}
void SortableList::MergeSort(){
	MergeSort(0,maxSize-1);
}
void SortableList::MergeSort(int left,int right){ //将数组通过二分法分解
	if(left<right){
		int mid=(left+right)/2;
		MergeSort(left,mid);
		MergeSort(mid+1,right);
		Merge(left,mid,right);
	}
}
void SortableList::Merge(int left,int mid,int right){  //两两合并
	int *temp=new int[right-left+1];
	int i=left,j=mid+1,k=0;
	while(i<=mid&&j<=right)
		if(l[i]<=l[j]) temp[k++]=l[i++];
		else temp[k++]=l[j++];
	while(j<=right) temp[k++]=l[j++];
	while(i<=mid) temp[k++]=l[i++];
	for(k=left,i=0;k<=right;) l[k++]=temp[i++];
	delete[]temp;
}

int main(){
	int n;
	cout<<"待排序数列的长度：";
	cin>>n;
	SortableList list(n);
	cout<<"待排序序列：";
	list.Input();
	list.MergeSort();
	list.Output();
	return 0;
}