iOS内存管理困惑

Question

我在实用程序类中有一个静态方法：

+ (Division *) getDefaultDivision
{
    Division *defaultDivision = [[Division alloc] init];
    defaultDivision.Id = 0;
    defaultDivision.name = @"Accounting";
    defaultDivision.slug = @"accounting";

    return defaultDivision;
}

在我的ViewController中，我做类似的事情：

Division *div = [[Division alloc] init];
div = [Utilities getDefaultDivision];

但是当我分析时，它说“分配给第x行并存储到defaultDivision的对象的潜在泄漏”。

如果我使用：

Division *defaultDivision = [[[Division alloc] init] autorelease];

它可以工作一次，但是当我再次使用它时，它会崩溃。

只是想知道在这里应该做什么？

Answer 1

如果这是您的真实代码；

Division *div = [[Division alloc] init];
div = [Utilities getDefaultDivision];

您首先要分配一个Division并将其保存在div中，然后从getDefaultDivision中获得一个新分区，也将其存储在div中而无需释放第一个分区。

Answer 2

无论如何实现“ getDefaultDivision”，此代码都存在泄漏：

Division *div = [[Division alloc] init];
div = [Utilities getDefaultDivision];

第1行分配内存，并分配div指向该内存。 您必须在某个时候释放此内存。 但是在第2行之后，这变得不可能了，因为div现在有了一个新值-并且指向第1行中分配的内存的指针丢失了 。 这是泄漏。 除非您了解原因，否则您将陷入困境。

至于这种方法：

+ (Division *) getDefaultDivision
{
    Division *defaultDivision = [[Division alloc] init];
    defaultDivision.Id = 0;
    defaultDivision.name = @"Accounting";
    defaultDivision.slug = @"accounting";
    return defaultDivision;
}

有时称为“工厂”方法-一种静态实用程序方法，用于分配，初始化和返回对类的新实例的引用。 此处的最佳做法是在工厂方法中使用自动释放。 例如：

Division *defaultDivision = [[[Division alloc] init] autorelease];

为什么是最佳做法？ 根据Apple的内存管理指南，只有其中包含以下术语的方法才应返回对调用者负责释放的对象的引用：

alloc, allocWithZone:, copy, copyWithZone:, mutableCopy, mutableCopyWithZone:

-来自http://developer.apple.com/library/IOs/#documentation/General/Conceptual/DevPedia-CocoaCore/MemoryManagement.html

由于“ getDefaultDivision”不是“ alloc”或“ copy”方法（它是访问器方法），因此它不应返回指向对象的指针，调用者以后必须释放该对象以避免泄漏。 将新分配的返回内存标记为自动释放是遵循此合同的一种方法。