这是内存泄漏吗？

Question

char *pointer1;
char *pointer2;

pointer1 = new char[256];
pointer2 = pointer1;

delete [] pointer1;

换句话说，我还必须做delete [] pointer2吗？

谢谢！

Answer 1

不，那个代码很好，不会泄漏内存。

你只需要使用delete []一次，因为你只使用一个new来为内存分配一个区域，即使有两个指向同一个内存的指针。

Answer 2

一个简单的规则：您需要与delete一样多的new s。更好的是，使用智能指针或容器之类的东西为你照顾它。

另一个小问题：pointer2在delete上致电pointer1后，正变成“悬空指针”。

Answer 3

这不是泄漏，但却在寻找麻烦。只要您删除pointer2，pointer1就会指向谁知道什么。这就是所谓的“悬垂指针”。使用它可以在最好的情况下导致段错误，并且在最坏的情况下可能导致在最终分配相同点的任何事物中出现神秘数据错误。

Answer 4

虽然它不会泄漏内存，但如果你想要显式，你应该将point1和point2都设置为NULL（并以这种方式初始化它们。）

Answer 5

此外，请考虑使用Boost库中的boost::shared_ptr<>。这是切片面包以来最棒的东西。

typedef boost::shared_ptr<TypeX> StrRef;

foo() {
  StrRef pointer1(new TypeX);

  while(something) {
    StrRef pointer2 = pointer1;
    // do stuff
  }

 return;
}

当指向它的最后一个指针超出范围时，将删除数据（TypeX）。如果您不需要引用计数，则可以使用内置auto_ptr<>类型执行类似的操作：

typedef auto_ptr<TypeX> StrRef;

foo() {
  StrRef pointer1(new TypeX);

  while(something) {
    TypeX * pointer2 = pointer1.get();
    subroutine(pointer2);
    if (condition) return;
  }

 return;
}

每当pointer1超出范围时，它都会删除数据。这样做的好处是你不必记得在底部的return语句之前放置一个删除，如果pointer1由于任何其他原因超出范围（即从循环的中间返回，或者subroutine()抛出异常，然后数据仍然会被正确释放。

我尚未测试此代码，因此您必须自行检查auto_ptr<>和boost::shared_ptr<>的文档。

我强烈建议尽可能使用Boost库。它是由专业人士编写的，它基本上是C ++扩展的临时区域。

Answer 6

delete删除new分配的内存。由于您只有一个new，因此只需要一个delete。

Answer 7

仅在使用新

时使用删除

好的做法是将指针设置为NULL，但如果你没有

，就不会有内存泄漏

Answer 8

不，你不必删除[]指针2，因为你没有为它分配内存！

语句pointer2 = pointer1使pointer2指向与pointer1相同的内存地址，不为其分配任何额外的内存。

Answer 9

每个new应该只有一个delete匹配{{1}}。如果删除了另一个指针，则会破坏该规则。

Answer 10

你在这里做的只是说明new char[256]分配的内存块将在同一时刻被pointer1和pointer2指向。

如果你之后写了delete[] pointer2;语句，那将是一个内存泄漏。

Answer 11

这是您思路上的差距：您不删除指针 - 删除内存。您只需使用指针来标识要释放的内存块。由于这两个变量指向同一个内存，因此删除一个就完全等同于删除另一个。换句话说，删除两者与删除其中一个相同 - 这显然是错误的。