c - 动态和静态字符数组

Question

程序代码如下:

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

char * cloning(char * q){
    char s[strlen(q)];
    int i;
    for(i = 0; i < strlen(q); i++)
        s[i] = q[i];
    return s;
}

int main(){
    char q[] = "hello";
    char *s = cloning(q);
    return 0;
}

编译后出现警告，所以我将返回值改成这样:

char *b = s;
return b;

这样就可以解决warning了。但是我发现在函数 cloning() 中，sizeof(s) 是 5，而 strlen(s) 是 7。如果我将 char s[strlen(q)] 简单地更改为 char s[5]，输出仍然是不正确。谁能向我解释这个问题？非常感谢。

Answer 1

char s[strlen(q)] 是一个局部变量，因此当您返回它的地址时，它会导致未定义的行为。因此，您可以使用 strdup() 或 malloc() 来动态分配数组，从而确保当您从函数返回时数组 s 在堆上可用。返回的数组也需要被 free() 编辑，否则会发生内存泄漏:)

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

char * cloning(char * q){
    char *s = malloc(strlen(q)+1);
    // if you write char s[strlen(q)], it is defined locally, and thus on return gives an undefined behaviour
    int i;
    for(i = 0; i < strlen(q)+1; i++)
        s[i] = q[i];
    return s;
}

int main(){
    char q[] = "hello";
    char *s = cloning(q);
    free(s);
    return 0;
}