Question

用C++在编写程序时为什么当要释放内存是程序就报错

代码为
#include<iostream.h>
#include<stdlib.h>
#include<string.h>
#define N 10
void f(char *line[]);
int main()
{
int i,len;
char b[32],*c[N];
cout<<"请输入10串字符串"<<endl;
for(i=0;i<N;i++)
{
cout<<"第"<<i+1<<"串";
cin>>b;
cout<<endl;
len=strlen(b);
c[i]=new char[len];
if(c[i]==NULL)
{
cout<<"失败！"<<endl;
return 0;
}
strcpy(c[i],b);
}

f(c);

cout<<"排序后字符串为"<<endl;
for(i=0;i<N;i++)
cout<<c[i]<<" ";
cout<<endl;
for(i=0;i<N;i++)
delete [] c[N];
c[N]=NULL;
return 0;
}
void f(char *line[])
{
int i,j,min;
char *p;
for(i=0;i<N-1;i++)
{
min=i;
for(j=i+1;j<N;j++)
if(strcmp(line[j],line[min])<0)
{
min=j;
p=line[i];
line[i]=line[min];
line[min]=p;
}
}
}

Answer 1

c[i]=new char[len];
改成 c[i] = new char[len+1];
原因：c风格字符串都是以'\0'结尾的。函数strlen(char*p);仅是求的一个字符串的除结束符外的长度。

另外
for(i=0;i<N;i++)
delete [] c[N];
c[N]=NULL;
写错了
改成
for(i=0;i<N;i++)
{
delete [] c[i];
c[i]=NULL;
}

Answer 2

如果释放内存，以前的结果就没有了，当然出错