C：缺少一些关于指针的逻辑

Question

我正在编写自己的字符串复制函数。以下有效：

char *src, *dest;
src = (char *) malloc(BUFFSIZE);
//Do something to fill the src
dest = (char *) malloc(strlen(src) + 1);
mystringcpy(src, dest);

void mystringcopy(char *src, char *dest) {
   for(; (*dest = *src) != '\0'; ++src, +dest);
}

但这不起作用：

char *src, *dest;
src = (char *) malloc(BUFFSIZE);
//Do something to fill the src
mystringcpy(src, strlen(src), dest);

void mystringcopy(char *src, size_t length, char *dest) {
   dest = (char *)malloc(length + 1);
   for(; (*dest = *src) != '\0'; ++src, +dest);
}

我不明白为什么......在被调用函数内分配内存是一个错误吗？

Answer 1

您还没有真正说明“有效”的含义，但我假设您很困惑为什么 dest 没有在调用函数中更改为新内存。

原因是在您的 mystringcopy 函数中，参数 dest 是调用中指针 dest 的副本功能。

然后，您将该副本分配给一个新的缓冲区，进行复制，然后该副本就会消失。原件不变。您需要将 dest 作为指针（指向指针）传递。

另外，我假设您从内存中编写了您所做的事情，因为它不应该按原样编译（调用函数中的错误取消引用）。下面是固定代码：

char *src, *dest;
src = (char *)malloc(BUFFSIZE); // no dereference on src, it's a pointer

//Do something to fill the src
mystringcpy(src, strlen(src), &dest); // pass the address of dest

// take a pointer to a char*
void mystringcopy(char *src, size_t length, char **dest) {
    // now you should dereference dest, to assign to
    // the char* that was passed in
    *dest = (char *)malloc(length + 1);

    // for simplicity, make an auxiliary dest
    char* destAux = *dest;

    // and now the code is the same
    for(; (*destAux = *src) != '\0'; ++src, ++destAux);
}

另一种方法是返回 dest 指针：

char *src, *dest;
src = (char *)malloc(BUFFSIZE);

//Do something to fill the src
dest = mystringcpy(src, strlen(src)); // assign dest

char* mystringcopy(char *src, size_t length) {
    char* dest = (char *)malloc(length + 1);

    // for simplicity, make an auxiliary dest
    char* destAux = dest;

    for(; (*destAux = *src) != '\0'; ++src, ++destAux);

    return dest; // give it back
}

请记住，如果长度小于源缓冲区的实际长度，则会超出目标缓冲区。请参阅解决方案的评论，尽管这由您决定。

Answer 2

在函数内部执行 malloc 是可以的，但您不会将指针从函数中传回。返回指针：

char * mystringcopy(char *src)

或将指针传递给指针：

void mystringcopy(char *src, char **dest)

Answer 3

C 中的参数按值传递，因此您的函数会获取 dest 指针的副本，用 malloc 覆盖它，然后丢弃它。试试这个：

void mystringcopy(char *src, size_t length, char **dest) {
   *dest = (char *)malloc(length + 1);
   char *p=*dest;
   for(; (*p = *src) != '\0'; ++src, ++p);
}

现在你将一个指针传递给你的字符串，这样你就可以在主过程中覆盖它。你会这样使用它：

char *src, *dest;
*src = (char *) malloc(BUFFSIZE);
//Do something to fill the src
mystringcpy(src, strlen(src), &dest);
// now in dest you have your copy

Answer 4

在函数内部分配没有问题。

问题是在 C 中参数是按值传递的。因此，当您为 dest 赋值时，这只是修改函数本地的 dest。

你有两个选择。您可以返回 dest 指针：

char *alloc_and_copy(const char *src, size_t length)
{
    char *dest = malloc(length + 1);
    ... do your copying
    return dest;
}

或者您可以将指针传递给参数并修改所指向的内容：

void alloc_and_copy(const char *src, size_t length, char **dest)
{
    char *local_dest = malloc(length + 1);
    ... do your copying using local_dest

    *dest = local_dest;
}

不需要使用局部变量的技术，但我认为它可以使代码更具可读性。

Answer 5

一般来说，在分配内存时，对于哪些代码负责在完成后释放内存有一定的假设。我赞同这样一种观念：函数应该负责一项主要操作，就像一个黑匣子一样。出于这两个原因，最好分配自己的内存并将指针传递给函数以填充其缓冲区。

除此之外，您可以返回 char * 指针作为返回值。

或者，将 char *dest 参数更改为 char **dest。然后，调用如下函数：mystringcopy(src, strlen(src), *dest)。在函数中，它通过以下方式返回指针：*dest = (char *)malloc(length + 1);。不漂亮。