在C动态中复制字符串[关闭]

Question

我有两个版本的 C 代码。第一个有效，但第二个无效。
第二个的输出是分段错误，但我不知道为什么。
也许有人可以解释我的错误？我真的很感激。

这行得通：

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

void stringcpy(const char*, char*);

    int main() {

        const char * original = "C is fun.";
        int size = sizeof(original) / sizeof(char*);
        copy = (char*)malloc(sizeof(char) * 11 + 1);

        stringcpy(original, copy);
        printf("%s\n", copy);

        free(copy);

        return 0;
    }

    void stringcpy(const char* original, char* copy) {

        int i;
        for(i = 0; *(original + i) != '\0'; i++) {
            *(copy + i) = *(original + i);
        }
        *(copy + i) = '\0';
    }

这不起作用：

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

void stringcpy(const char*, char*);

int main() {

    const char * original = "C is fun.";

    char * copy;
    stringcpy(original, copy);
    printf("%s\n", copy);

    free(copy);

    return 0;
}

void stringcpy(const char* original, char* copy) {

    int size = sizeof(original) / sizeof(char*);
    copy = (char*)malloc(sizeof(char) * size + 1);

    int i;
    for(i = 0; *(original + i) != '\0'; i++) {
        *(copy + i) = *(original + i);
    }
    *(copy + i) = '\0';
}

Answer 1

int size = sizeof(original) / sizeof(char*);

将sizeof 应用于此处的指针而不是数组。这不会返回它指向的字符串的大小。 strlen 是您应该在这里寻找的那个。用这个替换上面的行（size确定的解决方案）

int size = strlen(original); // better to use `size_t`

现在发生了什么 - 字符串文字是一个 char 数组，它衰减为指向第一个元素的指针，并且正在分配该指针值 - 您在其上应用了 sizeof。这里没有阵列的痕迹。要获取数组可以容纳的元素数量，您必须传递数组本身而不是指针。

另一件事是 C 是按值传递 - 所以这里要保留更改，要么将指针传递给指针，要么从函数传递分配的内存地址。

sizeof(char) 是 1 所以你不需要显式地写 - 你可以简单地写

copy = malloc(size + 1);

代码：

stringcpy(original, &copy);

和

void stringcpy(const char* original, char** copy) {

    int size = strlen(original);
    *copy = malloc(size + 1);

    int i;
    for(i = 0; original[i] != '\0'; i++) {
        (*copy)[i] = original[i];
    }
    (*copy)[i] = '\0';
}

我通过将指针变量的地址传递给另一个函数，向您展示了问题的解决方案。或者（自己尝试）您可以从函数返回分配的内存块的地址并将其分配给copy。这也可以。

编辑：如果您不允许更改函数签名，那么是的，除非您像之前在 case-1 中所做的那样将分配的内存提供给 copy，否则这是不可能的。

实现正确行为的另一种方法是这样做

char* stringcpy(const char* original) {

    int size = strlen(original) ;
    char* copy = malloc(size + 1);

    int i;
    for(i = 0; original[i] != '\0'; i++) {
        copy[i] = original[i];
    }
    copy[i]=0;
    return copy;
}

并像这样使用它

copy = stringcpy(original);

这个答案中省略了几件事：

• 始终检查 malloc 的返回值 - 如果它返回 NULL，您不会取消对它的引用。

• 不要将malloc - char* 的返回类型强制转换为void* 隐式转换。

• 使用完动态分配的内存后释放它。

Answer 2

您在函数内部分配的内存是从堆中分配的。当您退出函数时，该内存会返还给系统，因此在您返回时不再分配。