[問題] array pointer的問題

[sighAll (sigh)]

大家好  朋友問了一個題目 說哪裡有錯


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;
}

--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 1.161.253.187
※ 文章網址: https://www.ptt.cc/bbs/C_and_CPP/M.1532858095.A.F7D.html
→ leo850319: 因為你的p一直++
→ james732: strlen應該不包含\0的長度吧？
推 cutekid: malloc 少 1 byte
→ sighAll: malloc加一個byte也是一樣結果
推 jerryh001: 1樓的對
推 cutekid: p 的位置已經不在字串開頭，跑到結尾了
→ Gway: malloc改成（strlen(s)+1） while loop 改成 while (*s !=
→ Gway:  ‘\0’)  配合原先temp寫法較佳
推 Gway: 你的temp跟p 建議交換使用 語意上比較清楚 temp 用來操作指
→ Gway: 標 p是new alloc要傳回的指標
→ sighAll: 謝謝大家 頓時豁然開朗!
推 BelieveRich: 所以最後是什麼問題？
推 steve1012: 他的p已經移動了 並不是指到字串的頭