局部变量问题

[asdfg014]

char *Test()
{
char p[20];
memset(p,0,20);
strcpy(p,"123456789" );
return p;
}
void main()
{
char *p=Test();
printf("\n-----%s---------\n",p);
}
这样的程序有问题吗？有什么问题

[aminconi]

当然有问题了
char *Test(){}
里的p是临时变量
函数体里分配的空间在执行完函数就被释放了

[sumjohn]

有问题
char p[20]是个局部变量
是建立在栈空间里的
在Test运行结束后就消亡了
所以你在main函数里调用时值已经是错误的了

[sankt]

条款31: 千万不要返回局部对象的引用，也不要返回函数内部用new初始化的指针的引用

[sankt]

//另外一种解决方案：

char *Test()
{
char *p = new char[20];
memset(p,0,20);
strcpy(p,"123456789" );
return p;
}
void main()
{
char *p=Test();
printf("\n-----%s---------\n",p);
delete []p; //注意释放空间

}

[飞哥]

看看输出就知道行不行了

指针可以解决

[twinsen0398]

用个局部字符串指针！~用strdup函数！~不要忘记释放最后的指针！~

[ammana_babi]

同意楼上的。

不过如果采用strdup()函数的话，容易造成内存泄漏的问题，这是C/C++程序员比较痛苦的一个

问题。

下面是MS给出的strdup()函数的源代码：

char * __cdecl strdup ( const char * string )
{
char *memory;
if (!string)
return(NULL);
if (memory = malloc(strlen(string) + 1))
return(strcpy(memory,string));
return(NULL);
}

这个函数通过malloc函数来申请内存，所申请的内存必须由函数调用者释放。

修改后的代码：

char *Test()
{
char *p;
p = strdup("123456789");
return p;
}
main()
{
char *p=Test();
printf("\n-----%s---------\n",p);
free(p);//千万别忘记释放所申请的内存吆
}

[alphajin]

char *Test(void)
{
char *p = (char *)malloc(sizeof(char)*20);
memset(p, 0, 20);
strcpy(p,"12345667890");
return p;
}

main()
{
char *p=Test();
printf("\n-----%s---------\n",p);
free(p);//千万别忘记释放所申请的内存吆
}
这样有错嘛?
为什么?

[lilome]

free(p) free应该释放多大的内存?
Test函数里面根本就不应该分配20个char的内存
应该分配strlen("12324567891") + 1个char

[alphajin]

我只是把楼主的程序改写一下而已,没有free这个函数,贴错了

[ammana_babi]

char *p = (char *)malloc(sizeof(char)*20);
memset(p, 0, 20);
strcpy(p,"12345667890");

这三句话不就是strdup()函数等同于strdup()函数的调用吗。

唯一的区别是把申请的内存先置0而已。