处理工厂方法中的内存泄漏

Question

我正在开发一个目标 C 框架，最终将作为静态库发布。但是，当我将该库集成到泄漏工具中的实际应用程序（通过添加静态库）时，我发现存在一些内存泄漏。

这是一个示例场景。

@implementation Test

@synthesize testNumber

+(Test) createTestInstance {

    Test *test = [[Test alloc] init];
    test.testNumber = [[NSDecimerNumber alloc] initWithInt:1];

    return test;
}

-(void) dealloc {
    [testNumber release];
}

@end

虽然我在 dealloc 中释放了 testNumber 变量，但我在 Leaks 工具中的 alloc 位置看到内存泄漏。这里可能有什么问题？

此外，由于这是一个提供给用户调用的库，因此从库代码中释放这些变量是最佳实践吗？

谢谢

Answer 1

我在这里看到两个问题。如果 testNumber 是一个保留属性，则使用以下语句过度保留它：

test.testNumber = [[NSDecimerNumber alloc] initWithInt:1];

alloc-init 和属性访问器都保留该对象。因此，应该是：

test.testNumber = [[[NSDecimerNumber alloc] initWithInt:1] autorelease];

不用说，你还需要在dealloc方法中释放testNumber。

另外，我知道 createTestInstance 是一个创建 Test 对象的便捷构造函数，它应该根据 对象所有权策略（仅名称以“alloc”、“new”、“copy”或“mutableCopy”开头的方法返回您拥有的对象）：

+ (id)createTestInstance {

    Test *test = [[[self alloc] init] autorelease];
    test.testNumber = [[[NSDecimerNumber alloc] initWithInt:1] autorelease];

    return test;
}

最后，按照@的建议Josh Caswell，便利构造函数应该返回 id 而不是特定的类。来自 Objective-C 编程语言：

方便的返回类型
出于同样的原因，构造函数也是 id
它是初始化方法的 id，如
在“约束和约定。”

此外，他们应该使用 self 而不是硬编码的类名来分配初始化实例，以便正确处理子类化（此处为 self指的是类对象本身，因为这是一个类方法）。