Objective-C 内存管理 - 很确定我做错了

Question

大约 3 个小时后，我终于设法修复了视图控制器中的内存泄漏。泄漏是由 UIPickerView 引起的，该 UIPickerView 在头文件中将其属性设置为“保留”。

以下代码设法修复它：

- (void)viewDidLoad {
    [super viewDidLoad];    
    myPicker = [[[UIPickerView alloc] initWithFrame:CGRectZero]autorelease];
}

- (void)dealloc {
    [super dealloc];
    [myPicker release];
    myPicker = nil;
}

请不要告诉我这段代码有多令人震惊……我知道这很糟糕。我有一个发布，和一个自动发布。问题是，如果我更改或删除上述任何部分，内存泄漏就会返回。

虽然我知道客观的 C 内存管理是如何工作的，但显然不知道......

为什么上面的代码修复了内存泄漏，正确版本的代码是什么样的？

-

编辑：

如果有人遇到同样的问题或感兴趣 - 问题是我班级中的其他对象之一被设置为“保留”而不是“分配”。 （如果你不拥有一个对象，它应该有属性分配，而不是保留）。

就像 Cannondale 所说，删除额外的保留可以解决所有问题，并且只需要一个版本。

Answer 1

您必须在代码中的其他位置对 myPicker 进行保留。一旦堆栈展开 viewDidLoad 调用，您的 myPicker 分配行将释放该内存（这就是 autorelease 告诉它做的事情）。

在那之后您必须在某处进行保留，否则您的 [myPicker 版本] 将尝试释放未分配的内存，结果无法预测。

你应该做的是在 viewDidLoad 中分配内存（所以删除自动释放）。确保不要在其他任何地方保留对象并从 dealloc 中释放 myPicker。

还有...bbum 所说的关于 dealloc ;)

Answer 2

Cannonade 说的。这应该有效：

myPicker = [[UIPickerView alloc] initWithFrame:CGRectZero];

你的dealloc 也被破坏了。对super 的调用始终必须在最后（考虑一下），这可能会导致未定义的行为。

- (void)dealloc {
    [myPicker release];
    myPicker = nil;
    [super dealloc];
}