[问题] array pointer的问题

楼主: sighAll (sigh)   2018-07-29 17:54:53
大家好 朋友问了一个题目 说哪里有错
char * saveString(const char * s)
{
// allocate memory for a copy of the input string s
char *p = (char*) malloc(strlen(s));
while (*s)
{
*p++ = *s++;
}
*p = '\0'; // null-terminate saved copy
return p;
}
小弟研究半天 后来终于找到解答 多一个char *temp 去让p指过去 最后return temp,
但我不晓得为什么原本的写法有问题
以下是可以compiler和回传回来是正确的code
请大大开示 谢谢!!
char * saveString(char * s)
{
// allocate memory for a copy of the input string s
char *p = (char*) malloc(strlen(s));
char *temp;
temp = p;
while (*s)
{
printf("*s=%c\n", *s);
*p++ = *s++;
}
*p = '\0'; // null-terminate saved copy
return temp;
}
作者: leo850319 (不要说话)   2018-07-29 18:03:00
因为你的p一直++
作者: james732 (好人超)   2018-07-29 18:21:00
strlen应该不包含\0的长度吧?
作者: cutekid (可爱小孩子)   2018-07-29 18:30:00
malloc 少 1 byte
楼主: sighAll (sigh)   2018-07-29 18:44:00
malloc加一个byte也是一样结果
作者: jerryh001   2018-07-29 19:02:00
1楼的对
作者: cutekid (可爱小孩子)   2018-07-29 20:05:00
p 的位置已经不在字串开头,跑到结尾了
作者: Gway (我爱的你 也爱我 好吗?)   2018-07-29 21:42:00
malloc改成(strlen(s)+1) while loop 改成 while (*s !=‘\0’) 配合原先temp写法较佳你的temp跟p 建议交换使用 语意上比较清楚 temp 用来操作指标 p是new alloc要传回的指标
楼主: sighAll (sigh)   2018-07-29 21:47:00
谢谢大家 顿时豁然开朗!
作者: BelieveRich (Rich)   2018-07-30 01:51:00
所以最后是什么问题?
作者: steve1012 (steve)   2018-07-30 03:13:00
他的p已经移动了 并不是指到字串的头

Links booklink

Contact Us: admin [ a t ] ucptt.com