c - 我自己的 strcat 函数使用 C 错误

Question

我基本上是在用 C 编写我自己的字符串函数。

我一直在尝试strcat使用指针来执行该函数，但似乎无法理解我是否应该使用malloc或将其留给堆来分配内存。

char *my_strcat(const char *s1, const char *s2)
{


    const unsigned char *p1 = (const unsigned char *)s1;
    const unsigned char *p2 = (const unsigned char *)s2;

    unsigned char *string;
        //string = malloc(strlen(s1) + strlen(s2) + 1);
    while (*p1 != '\0')
{
        *string = *p1;
        string++;
        p1++;

        if(*p1 == '\0')
        {
            while(*p2 != '\0')
            {
                *string = *p2;
                string++;
                p2++;
            }
        }
    }
    return (char *)string;  
}

有关更有效地执行此任务或我做错的事情的任何提示都会很棒！

干杯

编辑

好的，所以我得到了一个可行的解决方案，但只是想知道在我使用 malloc 之后我应该在哪里释放（）它？

char *my_strcat(const char *s1, const char *s2)
{


    const unsigned char *p1 = (const unsigned char *)s1;
    const unsigned char *p2 = (const unsigned char *)s2;

    char *string = malloc(sizeof(char *));
    char *res = string;

    while (*p1 != '\0')
{
        *string = *p1;
        string++;
        p1++;
    }
    while (*p2 != '\0')
    {
        *string = *p2;
        string++;
        p2++;
}
    *string = '\0'; 

    return (char *)res; 
}

Answer 1

首先，我假设分配被错误地注释掉了。

• 您需要保存分配的指针并返回它。否则，您将返回一个指针string，它指向连接结果的末尾
• 您没有终止结果字符串；你需要添加*string = '\0'
• 您应该将第二个循环移到第一个循环的外部，并删除if它周围的条件：如果第一个循环已经终止，您知道*p1指向\0

---

char *string = malloc(strlen(s1) + strlen(s2) + 1);
char *res = string;
for (; *p1 ; *string++ = *p1++);
for (; *p2 ; *string++ = *p2++);
*string = '\0';
return res;  

Answer 2

strcat 不分配任何内存，所以如果你想完成同样的事情，那么你不需要使用 malloc。

char* strcat(char* destination,char* source) {
 int c = 0;
 int sc;

 while(destination[c] != 0) {  c++; }

 for(sc = 0;sc < strlen(source);sc++) {
  destination[sc+c] = source[sc];
 }

 destination[sc+c] = 0;

 return destination;

}

Answer 3

这对我有用。

char* my_strcat(char* a,char* b)
{
        int i,j;

        for(i=0;a[i];i++);
        for(j=0;b[j];j++,i++)
         a[i]=b[j];
        a[i]='\0';

}