c++可以多次删除指针吗？

Question

如果我有以下示例：

test.h

class MyClass
{
public:
MyClass();
std::string name1;
std::string name2;
std::string type1;
std::string type2;

void method1(MyClass &obj1);
void method2(MyClass &obj2);
}

test.cpp

MyClass *mainObject = new MyClass();

MyClass::MyClass()
{
}
void MyClass::method1((MyClass &obj1)
{
//do stuff
mainObject=&obj1; //we populate some of the MyClass variables


}
void MyClass::method2((MyClass &obj2)
{
//do stuff
mainObject=&obj2; //we populate the rest of MyClass variables
}

什么时候应该删除 test.cpp 中的 mainObject？我应该创建一个析构函数以便客户端删除它吗？

Answer 1

这是一个很好的例子，最好不要自己考虑。

使用shared_ptr<MyClass> mainObject;（新的 C++11 或 Boost 版本）。它会为你做delete。

请注意，method1() 和 method2() 也应该通过 shared_ptr 接受他们的论点。目前，他们正在做一件非常糟糕的事情：删除一个通过引用传递的对象。

Answer 2

多次删除指针变量（指向非0）比不删除更糟糕。因为前者会导致难以发现的错误和未定义的行为。

您的代码编写不正确。当您尝试使用&obj1 或&obj2 分配它时，您应该立即delete mainObject;。但请确保您只是第一次这样做。如果指针指向obj1 或obj2，请不要delete。

我从这个问题和你的previous question 中感觉到你来自Java/C# 背景。最好先读一本关于 C++ 的好书，你会发现大多数时候你不需要new/delete。

Answer 3

当你使用完指针指向的对象后，你应该删除指针。当指针指向单个对象时，您不应将其删除两次。如果指针指向您未使用 new 动态分配的对象，则不应删除该指针。

Answer 4

我想我会走一条稍微不同的路。

像这样：

测试.h

class MyClass
{
public:
  MyClass();
  std::string name1;
  std::string name2;
  std::string type1;
  std::string type2;

  void method1(MyClass &obj1);
  void method2(MyClass &obj2);
}

test.cpp

MyClass mainObject; // default c-tor called automatically.

MyClass::MyClass()
{
}
void MyClass::method1(MyClass & obj1)
{
  //do stuff

  //we populate some of the MyClass variables
  mainObject.name1=obj1.name1;
  mainObject.type1=obj2.type1;
}
void MyClass::method2(MyClass & obj2)
{
  //do stuff

  //we populate more of the MyClass variables
  mainObject.name2=obj1.name2;
  mainObject.type2=obj2.type2;
}

没有简单的方法可以只填充对象的一部分而不指定哪些部分。

但是，否则，如果您不使 mainObject 成为指针，那么您就不需要为它分配空间，这是自动完成的。 （但是，除非真的需要，否则我应该反对使用全局变量！）

我认为你正在尝试做的这种实现将完全避免使用堆，不需要新建/删除。

Answer 5

任何资源都应该有一个逻辑所有者，并且该所有者应该删除该资源。

在某些情况下，共享所有权是有意义的，这就是 boost::shared_ptr 和类似解决方案的用途。最后一个放弃所有权的就是删除资源的那个。

Answer 6

从所有 cmets 看来，您实际上可能想要以下内容：

static MyClass mainObject; // Not a pointer. Local to test.cpp

void MyClass::method1()
{
  //do stuff
  mainObject=*this; // Make a copy of the last object modified.
}
void MyClass::method2()
{
  //do stuff
  mainObject=*this; // Make a copy of the last object modified.
}

这样，无论你调用foo.method1()还是bar.method2，.左侧的对象都会复制到mainObject。根本不需要指针时髦，不需要new 也不需要delete。

Answer 7

什么时候应该删除 test.cpp 中的 mainObject？

不再使用时。

我应该创建一个析构函数以便客户端删除它吗？

如果必须释放类 MyClass 的某些资源，您只需要创建一个析构函数 - 显示的代码不是这种情况。您应该释放 (=delete) 的是 mainObject。但无论如何，method1(..) 和 method2(..) 正在覆盖 mainObject 指针，这会导致指针悬空（您无法再访问该对象）。

[编辑]
回答你的问题：

c++可以多次删除指针吗？

1. 指针通常不会分配给 new - 只有它们指向的对象。
2. 如果您的意思是“可以在同一个指针上多次调用delete 吗？”答案是否定的，会导致UB。 delete 在零指针上已定义且合法。