良好实践/不良实践 - 指针和设置值 - Objective C

Question

我仍在思考指针。我编写了以下代码，并对它的优点表示怀疑。

我需要我的视图控制器能够为其 UIBarButtonItem 设置自定义标题，具体取决于它将哪个新视图控制器推送到导航控制器的堆栈上。我没有在每次尝试设置新标题之前使用条件代码来检查 self.navigationItem.backBarButtonItem 是否存在（因为默认情况下，除非您创建它，否则 self.navigationItem.backBarButtonItem 为零），我这样做：

它有效，但谁能指出为什么我不应该这样做： 我有一个自定义 getter 来延迟实例化导航栏后退按钮：

- (UIBarButtonItem *)myBackButton {
if (!myBackButton) {
    myBackButton =  [[[UIBarButtonItem alloc] initWithTitle:self.title style:UIBarButtonItemStylePlain target:nil action:nil] retain];
    self.navigationItem.backBarButtonItem = myBackButton;
}
return myBackButton;

}

现在，在 .m 文件的其余部分中，每当我想要更改后退按钮的标题时（例如，在将新的视图控制器推送到导航控制器的堆栈之前） ），我只是这样做：

self.myBackButton.title = @"Some Custom Title";

或：

self.myBackButton.title = self.title;

我相信这是有效的，因为 self.myBackButton 和 self.navigationItem.backBarButtonItem 指向同一个对象。这种类型的间接对象设置是否可取？如果没有，处理栏按钮场景的更好方法是什么？

非常感谢， 马特

Answer 1

您应该阅读内存管理指南来了解分配，保留，...

您正在分配后退按钮（=您拥有它）并且您也保留它（=您拥有它两次），因此您也必须调用释放两次。

不需要延迟实例化后退按钮，它只是一小块内存，我假设您总是希望它可见。更好的方法是在 UIViewController 的 init 方法中执行此操作。

正确的方法是……

self.navigationItem.backBarButtonItem = [[[UIBarButtonItem alloc] initWithTitle:self.title style:UIBarButtonItemStylePlain target:nil action:nil] autorelease];

总结内存管理指导规则……

• 每个名称以alloc、new、copy或mutableCopy开头的方法都会创建一个对象，这是你的（=你取得所有权，你必须在某个地方释放它）
• 如果你想释放它（阅读放弃所有权），你应该在这个对象上调用release或autorelease，好的做法是将ivar也设置为nil

Answer 2

您可能想将该方法更改为类似的方法。

- (UIBarButtonItem *)myBackButton {
    if (!myBackButton) {
        myBackButton =  [[[UIBarButtonItem alloc] initWithTitle:self.title style:UIBarButtonItemStylePlain target:nil action:nil] autorelease];
    }
return myBackButton;
}

然后，在其他地方（可能在您的 viewDidLoad/appear 中）调用

self.navigationItem.backBarButtonItem= self.myBackButton;

我已将保留更改为自动释放，因为任何带有 init、copy 或 new 的方法都会增加保留计数。如果它不包含 init copy 或 new，它应该返回一个自动释放的实例。

另外，当您将其设置为 backbuttonItem 时，无论如何它都会被保留。

值得注意的是，当通过点表示法访问属性时，您实际上正在经历 setter 和 getter。 ie

self.myBackButton.title =@"title";

与

[[self myBackButton] setTitle:@"title"];

您几乎应该总是遍历 setter 和 getter 而不是将其作为局部变量访问。即使用

self.myBackButton 而不是 myBackButton，因为这将允许您在整个过程中进行任何所需的设置，并且还知道如果有任何代码正在访问它，它将通过该方法而不是直接更改变量。

Answer 3

是的，您不清楚内存管理规则，但更多的是您的问题：不，不必费心实现此后退按钮访问器，已经有一个后退按钮访问器，它是 self.navigationItem.backBarButtonItem。

您建议的这种方法是惰性访问器，当您有一个在调用访问器之前不需要的对象时，可以使用该方法。它向不需要知道的调用者隐藏了对象创建的详细信息。

然而，在这种情况下，导航视图控制器在正确的时间需要对象（按钮），并且应该在有意义的地方创建对象。这是在视图控制器中，我认为按钮应该在 init、loadView 或者 viewWillAppear 中创建。在这种情况下，按钮创建的详细信息应该在您拥有按钮的任何一个位置中明确显示。

因此，无论您想在何处使用 self.myBackButton.title<，请取出该访问器/code> 而是使用 self.navigationItem.backBarButtonItem ，最后确保在必须访问它之前调用的位置（通常是您的 init ）对其进行初始化。

Answer 4

您的问题可以归结为：按下后退按钮后会发生什么 - 当它控制的视图被释放时。在这些情况下检查后退按钮的保留计数。如果它保持稳定，那就没问题（假设程序中没有其他模块分配不同的后退按钮实例）。否则，每次需要时都必须创建一个新的后退按钮。