这个小程序有问题吗？

[HKer_007]

#include ......

char * str;

char * getstr()
{
str = new char [100];
strcpy(str, "This is a test.");
return str;
}

void main()
{
char * p;
p = new char [100];

p = getstr;

delete [] p;
}

[puppet]

有问题

p = new char [100];//内存没有释放

p = getstr;

[liushmh]

char * getstr()
{
str = new char [100];
strcpy(str, "This is a test.");
return str;//返回了局部变量，内存已经释放了
}
p = getstr（）;//导致此处有问题，p指向了一块释放的内存。

[zongta]

不需要给p 分配100个空间

[liushmh]

sorry,我看错了，好像没有错误。

[liushmh]

哦，请原谅我拙劣的表现
p = getstr;//改为strcpy(p,str);
由于全局变量在程序结束后会自动释放内存
又由于delete []p;
这样会导致str指向的内存被释放两遍。

[vixkywx]

#include ......

char * str;

char * getstr()
{
str = new char [100]; //申请了str
strcpy(str, "This is a test.");
return str;
}

void main()
{
char * p;
p = new char [100]; //申请了p

p = getstr; //p指向的空间已经无法访问了

delete [] p; //p和str是同一地址，所以p和str都不能用了
//原来指向p的内存没释放
}

[liushmh]

看来，我还是没有考虑全面。；（

[ningzhiyu]

p = getstr;应该是p = getstr（）；吧

vixkywx(vicky) 说的对
程序共申请了200的char大小的空间，只是释放了其中的100个
造成100的leak。

char *q;
q=p;
p = getstr（）;
printf("%ld,%ld,%ld\n",p,q,p-q);
会发现p-q的值是104。他们的地址是不一样的。

[glassshark]

//这样可以，不过在不同的可见域里申请和释放堆空间不是一个好习惯，建议修改getstr函数，将其所需的内存由调用者来申请和释放。
#include ......

char * str;

char * getstr()
{
str = new char [100];
strcpy(str, "This is a test.");
return str;
}

void main()
{
char * p;
p = getstr;
delete [] p;
}