[已解决]C++ Primer Plus 第8章 编程题4

工科呆萌男

这段程序要求在set函数中使用new来申请空间，因此我又写了一段unset，用来释放内存。程序会出错，怎么才能在一个函数里面申请空间，在另一个函数释放内存呢？

#include <iostream>
#include <cstring>

using namespace std;

struct stringy {
        char * str;
        int ct;
};

void set(stringy &dst, char *src);
void unset(stringy &src);
void show(const stringy &src, int n = 1);

int main()
{
        stringy beany;
        char testing[] = "apple";

        set(beany, testing);
        show(beany);
        show(beany, 2);
        unset(beany);


        return 0;
}

void set(stringy &dst, char *src)
{
        int n = strlen(src);
        dst.str = new char[n];
        strcpy(dst.str, src);
        dst.ct = n;
}

void unset(stringy &src)
{
        delete[] src.str;
}

void show(const stringy &src, int n)
{
        cout << src.ct << endl;
        for (int i = 0; i < n; i++) {
                cout << src.str << endl;
        }
}

最佳答案

月排行榜 / 总排行榜

superbe

2020-1-13 22:58:36

void set(stringy &dst, char *src)
{
    int n = strlen(src);
    dst.str = new char[n+1];   // [n] 改成 [n+1]，加 1 是保存结束符 ‘\0’ 的
    strcpy(dst.str, src);
    dst.ct = n;
}

跳转到最佳答案楼层

superbe

void set(stringy &dst, char *src)
{
    int n = strlen(src);
    dst.str = new char[n+1];   // [n] 改成 [n+1]，加 1 是保存结束符 ‘\0’ 的
    strcpy(dst.str, src);
    dst.ct = n;
}

工科呆萌男

superbe 发表于 2020-1-13 22:58
void set(stringy &dst, char *src)
{
    int n = strlen(src);

请问，是不是因为delete的时候要通过'\0'来判断 delete多少个？

Croper

跟怎么删除没关系，

void set(stringy &dst, char *src)
{
        int n = strlen(src);
        dst.str = new char[n];  //这里溢出了，str的长度应该设为n+1
        strcpy(dst.str, src);
        dst.ct = n;
}

工科呆萌男

Croper 发表于 2020-1-13 23:11
跟怎么删除没关系，

多谢！

superbe

strlen()得到的长度没有包括末尾的\0，而strcpy()拷贝字符串时连同\0也拷贝在内，所以分配的内存应该是strlen()+1。