C语言中strdup()的实现

Question

我正在努力在 C 中实现 strdup()，我是 C 新手，我正在尝试构建自己的库。我正在使用来自string.h 的实际strdup() 来测试我的，当我针对主程序测试我的源代码时遇到了总线错误，但当我使用实际的strdup() 时却没有。我也不能使用除malloc()之外的任何现有功能。

源代码：

#include <stdlib.h>

char *ft_strdup(char *src)
{

    char *str;
    int len;

    while (src[len])
        len++;
    str = (char*)malloc(sizeof(*str) * (len+1));
    while (*src)
        *str++ = *src++;
    *str = '\0';
    return (str);
}

为什么我总是收到总线错误？

Answer 1

尝试以下修复：

1. 在递增之前初始化len。
2. Don't cast malloc's return value，不要使用sizeof(char)，它在标准中被定义为1，每个cstd 6.5.3.4p4：

   当 sizeof 应用于类型为 char、unsigned char 或 带符号的字符，（或其合格版本）结果为 1。

3. 使用指针保存原来的str指针

---

#include <stdlib.h>

char *ft_strdup(char *src)
{
    char *str;
    char *p;
    int len = 0;

    while (src[len])
        len++;
    str = malloc(len + 1);
    p = str;
    while (*src)
        *p++ = *src++;
    *p = '\0';
    return str;
}

Answer 2

如果您必须实现自己的 strdup 函数，至少尽可能多地依赖 string.h 的其余部分。其中许多函数都经过优化，速度比您自己编写的要快。例如，strlen 可能正在使用 SSE 指令，而您手动搜索空终止符字节却不是。它的代码也更少，这总是更好。简而言之，我建议这样做：

#include <stdlib.h>
#include <string.h>

char *ft_strdup(const char *s1)
{
  char *str;
  size_t size = strlen(s1) + 1;

  str = malloc(size);
  if (str) {
    memcpy(str, s1, size);
  }
  return str;
}