对象潜在泄漏警告 - 需要澄清

Question

在我分析我的代码后，Xcode 指出了潜在的泄漏，如下所示。

这是我应该关心的事情吗？

在此代码中，设置 doublyLinkedList 的类是唯一的所有者，并在整个程序执行过程中继续管理该对象。

Answer 1

您收到警告的原因是因为 new 调用返回一个保留对象，然后您的 setter 可能会对其执行另一个 retain （取决于它是合成的还是手动生成）。

另外，我建议您使用标准的 alloc/init 而不是 new，这样两阶段创建就很明显了。

这更好：

if (self) {
    DoublyLinkedList *dll = [[[DoublyLinkedList alloc] init] autorelease];
    self.doublyLinkedList = dll;
}

或者只是

if (self) {
    self.doublyLinkedList = [[[DoublyLinkedList alloc] init] autorelease];
}

Answer 2

您可能希望这样做：

if (self) {
    DoublyLinkedList *dll = [DoublyLinkedList new];
    self.doublyLinkedList = dll;
    [dll release];
}

在标头中，将 doublyLinkedList 声明为保留的 @property。

Answer 3

您有“潜在泄漏”，因为分析器发现您已经为 DoublyLinkedList 实例分配了内存（使用 new），将其放入名为 dll 的局部变量中，并且没有在同一范围内释放该内存。

假设您设置的 doublyLinkedList 成员碰巧也是声明为 retaining 的属性，那么您也会遇到实际泄漏，因为您过度保留了在此处创建的 DoublyLinkedList。

所有权规则表示您对此实例拥有一项声明，因为您调用了 new 来创建它。当您将实例传递给 setDoublyLinkedList: 时，它会被保留，然后您就有两个声明。当 init 方法结束时，您只有一个通过 ivar/property 对该实例的引用——您已经丢失了局部变量——这意味着您拥有的所有权声明多于您拥有的引用。这是一个很好的迹象，表明您将发生泄漏。

要修复泄漏，您需要在 init 方法结束之前放弃您的声明之一。您可以通过以下两种方式之一执行此操作：设置属性后立即使用 release：

DoublyLinkedList * dll = [DoublyLinkedList new];
[self setDoublyLinkedList:dll];
[dll release];

或 autorelease：

[self setDoublyLinkedList:[[DoublyLinkedList new] autorelease]];
// Or equivalent procedures involving a temp variable

但是，应该注意的是，在 中使用 setter init 可能有问题（另请参阅Mike Ash 的文章主题），因为访问器可能（可能）产生副作用，这取决于您的对象是否已完全设置。关于这个问题似乎有两个阵营，最好阅读它并得出自己的结论，但您可能会发现它简化了分配给 ivars 而不是使用属性的初始化方法：

if( self ){
    doublyLinkedList = [DoublyLinkedList new];
}

这在术语上是完全正确的的内存管理。

最后，如果 DoublyLinkedList 是一个您拥有其代码的类，您还可以考虑编写一个方便的构造函数，它将为您返回一个新的自动释放实例。 Cocoa 中的约定是简单地在类之后命名方法，使用标准方法名称大小写，如下所示：

+ (id) doublyLinkedList {
    return [[[self alloc] init] autorelease];
}

请注意，这是一个类方法：

if( self ){
    [self setDoublyLinkedList:[DoublyLinkedList doublyLinkedList]];
}

并参见 我对“自分配对象”的回答以解释这些构造函数。

Answer 4

如果您有一个名为“doublyLinkedList”的属性（基于给定代码的假设），并且它是“保留的”，您可以执行以下操作：

if (self) {
    DoublyLinkedList *dll = [[DoublyLinkedList alloc] init]
    self.doublyLinkedList = dll;
    [dll release];
}