【问题标题】:Return char[]/string from a function [duplicate]从函数返回 char[]/string [重复]
【发布时间】:2013-01-03 05:11:19
【问题描述】:

我对 C 语言编码相当陌生,目前我正在尝试创建一个函数,该函数返回一个 c 字符串/字符数组并分配给一个变量。

到目前为止,我观察到返回 char * 是最常见的解决方案。所以我尝试了:

char* createStr() {
    char char1= 'm';
    char char2= 'y';
    char str[3];
    str[0] = char1;
    str[1] = char2;
    str[2] = '\0';
    char* cp = str;
    return cp;
}

我的问题是如何使用返回的 char* 并将它指向的 char 数组分配给 char[] 变量?

我试过了(都导致 noob-drowning 错误):

  1. char* charP = createStr();
  2. char myStr[3] = &createStr();
  3. char* charP = *createStr();

【问题讨论】:

  • 更好的讨论,即使它较新

标签: c arrays string char return


【解决方案1】:

请注意,您并未动态分配变量,这几乎意味着函数中 str 内的数据将在函数结束时丢失。

你应该有:

char * createStr() {

    char char1= 'm';
    char char2= 'y';

    char *str = malloc(3);
    str[0] = char1;
    str[1] = char2;
    str[2] = '\0';

    return str;

}

然后,当您调用函数时,将接收数据的变量的类型必须与函数返回的类型匹配。所以,你应该有:

char *returned_str = createStr();

值得一提的是,必须释放返回值以防止内存泄漏。

char *returned_str = createStr();

//doSomething
...

free(returned_str);

【讨论】:

  • sizeof(char) 保证为 1 并且没有理由对malloc 返回进行类型转换
  • @Aniket +1 感谢您的评论。有时朋友告诉我,但我想我当时不明白这个问题,后来又没有想到这个问题:D
【解决方案2】:

如果你想从一个函数返回一个char*,确保你malloc()它。堆栈初始化的字符数组在返回时没有任何意义,因为在从该函数返回后访问它们是未定义的行为。

改成

char* createStr() {
    char char1= 'm';
    char char2= 'y';
    char *str = malloc(3 * sizeof(char));
    if(str == NULL) return NULL;
    str[0] = char1;
    str[1] = char2;
    str[2] = '\0';
    return str;
}

【讨论】:

  • 值得一提的是,必须释放返回值以防止内存泄漏。 char* astr = createStr(); doSomething(astr); free(astr);
  • 这就是我要找的!我有点困惑,因为从函数返回指针可能会导致内存泄漏。你的回答解决了我的问题。所以最终释放 char *astr 持有的内存。 (例如,在 Java 中。我知道这不是问题,因为垃圾收集器会小心处理)
【解决方案3】:
char* charP = createStr();

如果你的函数是正确的,那就是正确的。不幸的是,您正在返回指向函数中局部变量的指针,这意味着一旦函数返回,它就是指向未定义数据的指针。您需要对函数中的字符串使用诸如 malloc 之类的堆分配,以使您返回的指针具有任何意义。那你需要记得稍后释放它。

【讨论】:

    【解决方案4】:

    您可以在方法中使用静态数组,以避免函数结束时丢失数组:

    char * createStr() 
    {
        char char1= 'm';
        char char2= 'y';
    
        static char str[3];  
        str[0] = char1;
        str[1] = char2;
        str[2] = '\0';
    
        return str;
    }
    

    编辑: 正如 Toby Speight 所提到的,这种方法不是线程安全的,并且调用该函数会导致数据覆盖,这在某些应用程序中是不需要的。因此,您必须在从函数返回后立即将数据保存在缓冲区中。 (但是因为它不是线程安全的方法,在某些情况下并发调用仍然会出现问题,为了防止这种情况你必须使用锁。进入函数时捕获它并在复制完成后释放它,我不喜欢使用这个方法,因为它混乱且容易出错。)

    【讨论】:

    • 可能值得一提的一些缺点(所有调用共享相同的存储,因此调用会覆盖以前的结果,并且没有什么是线程安全的)。
    【解决方案5】:

    包含“string.h”使事情变得更容易。解决问题的更简单方法是:

    #include <string.h>
        char* createStr(){
        static char str[20] = "my";
        return str;
    }
    int main(){
        char a[20];
        strcpy(a,createStr()); //this will copy the returned value of createStr() into a[]
        printf("%s",a);
        return 0;
    }
    

    【讨论】:

    • 为什么要复制? char *a = createStr(); 也一样。
    • 是的,你也可以使用它,这样可以节省一些内存,只是他要求一种方法来“将它(函数)指向的 char 数组分配给 char[] 变量”所以我就这么告诉他了。
    猜你喜欢
    • 2018-10-18
    • 2020-02-21
    • 2018-05-09
    • 2015-09-02
    • 2015-01-02
    • 1970-01-01
    • 2023-02-16
    • 1970-01-01
    • 2021-02-24
    相关资源
    最近更新 更多