删除通过取消引用新对象初始化的对象

Question

我有以下课程：

现在我一直在考虑以这种方式实现我的构造函数：

基本上取消引用新操作返回的对象。 现在......这感觉不对，我可能应该使用指针，但我想挑战自己，也许让它发挥作用并学习新的东西。

所以我的问题将是删除这个对象......我的 Dense 析构函数不起作用，因为我有这样的：

但显然我遇到了这个错误：

free(): 无效指针

那么我怎样才能用我实现的方式释放分配的内存呢？

谢谢。

Answer 1

没必要这么复杂。您可以直接从初始化列表中调用构造函数。这里绝对不需要new。你也不需要delete那些对象。如果Dense 被销毁，它们的析构函数将被自动调用。

Dense::Dense(): weights(), //Calls constructor Matrix()
                bias(), //Calls constructor Matrix()
                act(ActivationType::Relu), //Calls constructor Activation(ActivationType::Relu)
                input() //Calls constructor Matrix()
{}

---

此外，你试图用指针做什么也行不通。比如&bias就是成员变量bias的地址，与new Matrix()返回的指针无关。

Answer 2

那么我怎样才能用我实现的方式释放分配的内存呢？

你根本做不到，你已经造成了内存泄漏。

---

说明：

当你写作时：

int a = *(new int(5)); // Really bad

你基本上是：

• 在堆上分配一个int
• 取消引用右值
• 将其分配（复制）到堆栈上的变量

您从不关心保留new 语句的返回指针。 a 是另一个 int，它是您在堆上分配的副本。

您不能删除a，因为它是在堆栈上创建的。由于a 是另一个对象，所以a 的地址与您在new 在堆上分配的int 的地址不同。

所以删除&a 显然是禁止未定义的行为，并且不能以任何方式删除您使用new 创建的int。而且由于您没有存储 new 语句的返回指针，因此您无法检索它：这是内存泄漏。

---

如果您获得的是参考而不是副本，则可以将其删除：

int & a = *(new int(5));
delete &a; // fine

但这是一种过于复杂、效率较低且毫无意义的简单做法：

int a = 5; // destroyed at scope's end