strcat 实现

Question

我尝试自己实现 strcat，并从 Wiki 找到了 strcat 实现，如下所示。 ....但是当我使用它时，出现分段错误。

下面的代码有什么问题？

char *
strcat(char *dest, const char *src)
{
    size_t i,j;
    for (i = 0; dest[i] != '\0'; i++)
        ;
    for (j = 0; src[j] != '\0'; j++)
        dest[i+j] = src[j];
    dest[i+j] = '\0';
    return dest;
}

Answer 1

代码没问题。

看来您的调用代码有问题。

您是否记得为目标字符串分配足够的内存？

Answer 2

我强烈建议使用指针而不是整数索引，因为担心整数溢出。即使 size_t 与 char * 的位数相同，您也会在不添加指针的地方添加索引。

我想这或多或少是学术性的；如果您在数 GB 字符串上调用 strcat()，您可能会遇到各种麻烦。

为了完整起见，这是一个基于指针的版本：

char *
my_strcat(char *dest, const char *src)
{
    char *rdest = dest;

    while (*dest)
      dest++;
    while (*dest++ = *src++)
      ;
    return rdest;
}

当然，这确实需要另一个指针的空间来存储 rdest 返回值，但我认为这是一个很好的权衡。

另请注意，您无法在普通应用程序代码中合法地定义名为 strcat() 的函数；整个命名空间（名称以 str 开头的公共函数）被保留用于实现。

Answer 3

dest 需要有足够的内存来容纳此实现中的串联。在此实现中，它必须由调用者分配。您还应该确保 dest 和 src 都以 null 结尾。如果 dest 没有足够的内存，这将覆盖可以被其他东西使用的内存。

Answer 4

它对我来说工作正常，我已经检查过。

    #include "stdio.h"


    char *strcat(char *dest, const char *src)

    {

    size_t i,j;

    for (i = 0; dest[i] != '\0'; i++)

        ;

    for (j = 0; src[j] != '\0'; j++)

        dest[i+j] = src[j];

    dest[i+j] = '\0';

    return dest;

}


void main(void)

{

    char a[10]={"abc"}, b[10]={"def"};

    strcat(a,b);

    printf("%s",a);

    getchar();

}

Answer 5

为目标字符串分配足够的内存..即至少（源字符串的长度+ 1）。