从 C 函数中创建的数组返回数组元素？

Question

这是在 C 中正确的做法吗？

char *func1() {
    char *str[3] = { "so", "is", "amazing" }; 

    return str[1]; 
}

char *func1() 返回一个指针，指向内存中存储字符串的位置。但是，当func1() 返回时，str 数组不会超出范围，我是否没有返回指向不存在的对象的指针？一个悬空指针（这个名字对吗？）

所以我有两个解决方案使str 成为全局的，这样它就永远不会超出范围，并且指针总是指向一个有效的内存地址，但这看起来很脏。

其他解决方案

char *func2() {
    char *str[3] = { "so", "is", "amazing" };
    int str1Len = strlen(str[1]); 
    char *ret = (char *)malloc(str1Len) ; // create a block of mem to hold the str[1]. 
    strncpy(ret, str[1], str1Len);
    return ret; 
}

这是正确的吗？这样做func1() 或func2() 的正确方法是什么？ 还是有更好的办法？

我使用func1() 已经有一段时间了，它没有给我带来任何问题，这是否意味着func1() 正在做正确的事情？和func2() 是不必要的？

Answer 1

在您的第一个函数中，返回str[1] 没有问题，因为它不是指向局部变量，而是指向字符串文字的指针。请注意，字符串文字应声明为const:

const char *func1(void) {
    const char *str[3] = { "so", "is", "amazing" };
    return str[1]; 
}

您的第二个函数返回一个指向已分配空间的指针。这个指针需要在某个时候被释放。分配不正确，您应该为最终的'\0' 分配额外的 1 个字节。您可以使用strcpy 将字符串复制到分配的空间，或者简单地使用strdup() 两者兼而有之：

char *func2(void) {
    const char *str[3] = { "so", "is", "amazing" };
    int size = strlen(str[1]) + 1; 
    char *ret = malloc(size);
    return strcpy(ret, str[1]);
}

或者简单地说：

char *func2(void) {
    const char *str[3] = { "so", "is", "amazing" };
    return strdup(str[1]);
}

永远不要使用strncpy，它不会像你认为的那样做。对于程序员和阅读您的代码的人来说，这很容易出错。

Answer 2

1. 这是在 C 中正确的做法吗？

   不，不是！通常，您需要创建局部变量的副本，尽管在这种情况下，指针指向字符串文字，因此它可以工作。

2. 你分配和复制字符串错误，你应该这样做

   size_t size =  strlen(str[1]) + 1; 
   char *result = malloc(size) ; // create a block of mem to hold the str[1]. 
   if (result != NULL)
       memcpy(ret, str[1], size);
   

Answer 3

虽然返回指向字符串文字的指针是可以接受的，但不是返回本地指针，但这可能是错误的方法。

如果str 没有改变，则设置为static const char * const str[] = ....，它同时符合数组和char []（每个字符串字面量）都指向const。 static 使其成为永久性的，但仍具有本地范围。它在您的程序代码开始之前设置一次。相反，非static 版本将为每次调用设置数组。

请注意，您必须返回 const char * 以保持 const 正确性。 我强烈建议这样做；如果不能，请仅省略第一个 const。

注意：不要将malloc和朋友（或一般void *）的结果转换为C。