C ++如何删除本地分配的char数组？

Question

我编写了一个函数，该函数接收一个char指针作为参数，然后构建一个包含该参数char的新动态分配的char数组。然后，它返回新的char数组。 这是功能：

 char* read_string(char *pstr)

    {
        char *str;
        str = new char[strlen(pstr)];//allocate memory for the new char
        str[strlen(pstr)] = '\0';
        for(unsigned i=0;i<strlen(pstr);i++)//build the new char
            str[i]=pstr[i];
        return str;//then return it
    }

我主要有：

int main()

    {
        char *Pchar = read_string("Test");

        cout<<Pchar;// Outputs "Test"

        delete [] Pchar;//"Program received signal SIGTRAP, Trace/breakpoint trap." error
    }

我在main中声明一个char指针，然后使其指向从read_string函数返回的char数组。它输出我想要的内容，但是如果我想释放内存，它将给我运行时错误。如何释放内存如果我不再需要使用Pchar了吗？

编辑：谢谢大家提供的非常有帮助的答案。我已经成功解决了这个问题。

Answer 1

您的特定问题是一个错误的错误：

str = new char[strlen(pstr) + 1];
//                         ^^^^ need one more for the '\0'
str[strlen(pstr)] = '\0';

通常，由于这是C ++而不是C，因此最好返回一个智能指针，以便调用者知道该指针的所有权语义是：

std::unique_ptr<char[]> read_string(char *pstr)
{
    std::unique_ptr<char[]> str(new char[strlen(pstr) + 1]);
    // rest as before
    return str;
}

Answer 2

您需要分配更多的内存以为EOS字符留出空间：

str = new char[strlen(pstr)+1];

Answer 3

似乎是由于分配的字符串的长度不正确而发生了错误。 您必须使用以下记录来分配字符串

    str = new char[strlen(pstr) + 1];//allocate memory for the new char
    str[strlen(pstr)] = '\0';

该函数可以如下所示

char* read_string( const char *pstr )
{
    char *str;
    size_t n = strlen( pstr );

    str = new char[n + 1];//allocate memory for the new char

    strcpy( str, pstr );

    return str;
}