我的 strcat 代码中的 printf 有什么问题？

Question

我已经制作了这个程序来模拟 strcat 功能，但是 printf 有一个我不明白的错误...

代码如下：

#include <stdio.h>

char *mystrcat(char *s1, char *s2);

int main(void)
{
    char *s1,*s2;
    s1="asdad";
    s2="asdad";
    s1=mystrcat(s1,s2);
    printf(s1);

    return 0;
}
char *mystrcat(char *s1,char *s2)
{
    int i,j;
    for(i=0;s1[i]<'\0';i++) ;
    for(j=0;s2[j]!='\0';j++) s1[i+j]=s2[j];
    s1[i+j]='\0';
    return s1;
}

Answer 1

第一个问题是s1 没有足够的空间来追加s2。您需要s1 指向的缓冲区大小至少为strlen(s1) + strlen(s2) + 1（+ 1 是NUL 终止符）。

第二个问题是字符串文字是只读的。您从"asdad" 分配s1，这会创建一个指向（可能）只读内存的指针。当然第一个问题意味着即使它是可写的，你也没有足够的空间来追加到末尾，但这是 C 语言中的常见缺陷之一，值得一提。

第三个问题（在another answer 中已经提到）是比较s1[i] < '\0' 是错误的，您将无法正确找到s1 的长度，因为循环不会运行一次迭代。正确的条件与第二个循环 != '\0' 中的条件相同。 （从那时起，这掩盖了问题 1，您无意中从一开始就覆盖了 s1。）

Answer 2

至少，s1[i] < '\0' 和s1[i] < 0 一样，总是假的。