工科呆萌男 发表于 2020-1-13 22:35:53

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;
        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

本帖最后由 superbe 于 2020-1-13 23:00 编辑

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

工科呆萌男 发表于 2020-1-13 23:08:33

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


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

Croper 发表于 2020-1-13 23:11:53

跟怎么删除没关系，void set(stringy &dst, char *src)
{
      int n = strlen(src);
      dst.str = new char;//这里溢出了，str的长度应该设为n+1
      strcpy(dst.str, src);
      dst.ct = n;
}

工科呆萌男 发表于 2020-1-13 23:21:29

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

多谢！

superbe 发表于 2020-1-13 23:33:18

strlen()得到的长度没有包括末尾的\0，而strcpy()拷贝字符串时连同\0也拷贝在内，所以分配的内存应该是strlen()+1。
页: [1]
查看完整版本: C++ Primer Plus 第8章 编程题4