如何返回在函数中创建的char数组？

Question

我已经编程了很长一段时间,我才真正意识到.我之前创建了许多函数,它们将字符串作为char数组返回(或至少指向它们的指针).

有一天有人指出,当我的函数返回时,我的函数指向的char数组已超出范围,我现在基本上指向一个随机的内存(一个令人讨厌的悬空指针).

我暂时没有注意到这一点,因为输出到控制台时的char数组似乎没有损坏(可能是因为没有时间覆盖该数据).当我返回一个通过读取经常损坏的串口生成的字符串缓冲区(char数组)时,我注意到了这一点.

那么,我该怎么做才能做到最好？

我的错误代码如下:

#include <cstdlib>
#include <iostream>

using namespace std;

char* myBadFunction(){
    char charArray[] = "Some string\n";
    char* charPointer = charArray;
    return charPointer;
}


int main(int argc, char** argv) {

    cout << myBadFunction();

    return 0;
}

我知道我应该在调用函数之前在程序中分配内存或创建一个全局变量来放入返回的字符串,但是如果我的被调用函数被许多不同的程序使用,那么它应该如何知道传递的缓冲区的大小提前进入它,何时删除此内存？

以下代码也没有正确地执行我想要的操作:

#include <cstdlib>
#include <iostream>

using namespace std;

void fillArray(char* charPointer){
    char charArray[] = "Some string\n"; // Create string
    charPointer = charArray; // Not correct, want to fill predefined array with created string
    return;
}


int main(int argc, char** argv) {

    char predefinedArray[50] = {0};
    fillArray(predefinedArray);
    cout << predefinedArray;

    return 0;
}

我想填充指针解析指向的数组,但这不会发生在上面的代码中.

另外,我什么时候应该使用new []命令来创建我的数组？它需要吗？什么时候应该在上面调用delete [].

非常感谢,这显然是非常基本的,但我已经做了一段时间的错误.

Answer 1

最简单的方法是返回a std::string,如果需要访问内部char数组使用std::string::c_str().

#include <iostream>
#include <string>

using namespace std;

string myGoodFunction(){
    char charArray[] = "Some string\n";
    return string(charArray);
}


int main(int argc, char** argv) {  
    cout << myGoodFunction();
    return 0;
}

如果需要返回除char数组以外的其他内容,请记住指针可以用作迭代器.这允许您将数组封装在矢量或类似结构中:

vector<int> returnInts() {
    int someNums[] = { 1, 2, 3, 4 };
    return vector<int>(someNums, someNums + 4);
}

Answer 2

在C++中有两个返回数组的选项.你可以填写预先分配的内存(好),或者在函数中分配你自己的内存并返回它(坏).第一个首选的原因是它重新强制正确处理分配的内存.

一个基本的例子如下:

void fillArray(char* buffer, int sz) {
  char text[] = "hello there!";
  if (sizeof(text)>sz) {
    // overflow! Buffer is too small!
    return;
  }
  for (int n=0;n<sizeof(text);n++) {
    buffer[n] = text[n];
  }
}

int main() {
  char* buffer = new char[30]; // allocates a buffer of 30 bytes.
  fillArray(buffer,30);
  cout << buffer;
  delete [] buffer;
}

/* note that it would be easier to use static memory in this example */

想一想这个问题并不难.