将指针传递给 strcat 不会更新字符串

Question

我正在按照 K&R 用 C 语言编写自己的 strcat 版本。这是我的代码：

#define MAXVALUE 1000

void concat(char *s, char *t)
{
    while (*s++)
        ;
    while (*s++ = *t++)
        ;
}

/* test */
int main()
{
    char s1[MAXVALUE];
    char s2[] = "Jim!";
    s1[0] = 'H', s1[1] = 'i', s1[2] = ' ';
    s1[3] = '\0';
    concat(s1, s2);
    printf("%s\n", s1);
    return 0;
}

这个想法是将s2 复制到s1 以获得“Hi Jim！”。我确保s1 足够大以包含两个字符串。但是，当我运行它时，它会输出“Hi”，所以基本上它不会更新字符串s1。我不知道为什么：s1 仍然指向s1[0] = 'H'，并且在运行concat 之后，s1[3] 中的'\0' 应该已经被替换，s1 应该在@ 处以'\0' 终止987654334@.

附：请注意，在 K&R 中，我的 strcat 版本与标准库之一不匹配。特别是返回值是void。

Answer 1

在您的代码中，问题是，在到达终止 NUL 之后，您正在推进指针 *s++，因此，它包含在目标字符串中，使得 printf() 被解释为结束细绳。根据字符串连接的设计规则，您需要删除[或替换，或覆盖]终止NUL并添加第二个字符串。

为避免终止 NUL 出现在输出字符串中，当指针到达 NUL 时不要增加指针，而是从该特定位置本身开始复制下一个字符串。

检查下面的代码。

#include <stdio.h>
#include <stdlib.h>

#define MAXVALUE 1000

void concat(char *s, char *t)
{
    while (*s)  s++;     //after NUL, do not increment, move on to copying
    while (*s++ = *t++)
        ;
}

/* test */
int main()
{
    char s1[MAXVALUE];
    char s2[] = "Jim!";
    s1[0] = 'H', s1[1] = 'i', s1[2] = ' ';
    s1[3] = '\0';
    concat(s1, s2);
    printf("%s\n", s1);
    return 0;
}

Answer 2

检查以下代码：

void concat(char *s, char *t) 
{
       while (*s != '\0')
          s++; 
           while (*s++ = *t++)
                      ;   
}

/* test */
int main()
{
   char s1[MAXVALUE];
   char s2[] = "Jim!";
   s1[0] = 'H', s1[1] = 'i', s1[2] = ' ';
   s1[3] = '\0';
   concat(s1, s2);
   printf("%s\n", s1);
   return 0;
}

Answer 3

感谢@MOehm，我通过简单地插入s-- 来修复代码，以补偿越过空终止符：

void concat(char *s, char *t)
{
    while (*s++)
        ;
    s--;
    while (*s++ = *t++)
        ;
}