Obj-C，在线分配的对象的潜在泄漏，UIBarButtonItem alloc

Question

我遇到分析器泄漏，但这与我在其他地方使用的代码相同，没有问题。我知道我正在使用 alloc，因此我必须释放，但我是在 dealloc 中执行此操作。

我做错了什么？

头文件：

@interface myViewController : UIViewController <UITableViewDataSource, 
               UITableViewDelegate> {

    UIBarButtonItem *addButton;
}
@property (nonatomic, retain) UIBarButtonItem *addButton;

主文件：

@synthesize addButton;
- (void)viewDidLoad {

    NSMutableArray* buttons = [[NSMutableArray alloc] initWithCapacity:3];


    addButton = [[UIBarButtonItem alloc]
                            initWithBarButtonSystemItem:UIBarButtonSystemItemAdd 
                            target:self action:@selector(btnNavAddPressed:)];
    addButton.style = UIBarButtonItemStyleBordered;
    [buttons addObject:addButton];

    [tools setItems:buttons animated:NO];
    [buttons release];

    self.navigationItem.rightBarButtonItem = [[UIBarButtonItem alloc] 
                                initWithCustomView:tools];

    addButton.enabled = FALSE;

- (void)dealloc {
    [addButton release];

Answer 1

当您使用属性并向其分配指定的属性时，将确定在分配给该属性时保留计数是否递增。在您的情况下，您指定了“retain”，这意味着处理属性分配的 setter 函数将自动增加对象的保留计数。

但是，当您编写时，

 addButton = [[UIBarButtonItem alloc]
                            initWithBarButtonSystemItem:UIBarButtonSystemItemAdd 
                            target:self action:@selector(btnNavAddPressed:)];

您正在创建一个已经保留计数== 1的对象，因此当您分配它时，它将具有保留计数2。执行此操作的正确方法是创建一个临时变量并创建对象，然后将临时变量分配给该物业随后释放了温度。变量：

UIBarButtonItem* tmp = [[UIBarButtonItem alloc]
                            initWithBarButtonSystemItem:UIBarButtonSystemItemAdd 
                            target:self action:@selector(btnNavAddPressed:)];
self.addButton = tmp;
[tmp release];

当然我会推荐一个比“temp”更具描述性的名称作为变量名称。

Answer 2

您没有使用 setter，代码应该是：

self.addButton = [[[UIBarButtonItem alloc]
                        initWithBarButtonSystemItem:UIBarButtonSystemItemAdd 
                        target:self action:@selector(btnNavAddPressed:)] autorelease];

通过使用名称与属性名称不同的 ivar 可以避免此类问题。这是在 @synthesize 语句中完成的：

@synthesize addButton = _addButton;

这样任何 self 的遗漏都会导致错误消息。

这是一个完整的实现（除了 tools 未定义），属性 addButton 在所有地方都是句柄：

@interface myViewController : UIViewController <UITableViewDataSource, UITableViewDelegate> {
}
@property (nonatomic, retain) UIBarButtonItem *addButton;
@end

@implementation myViewController
@synthesize addButton = _addButton;

- (void)viewDidLoad {
    NSMutableArray* buttons = [NSMutableArray array];

    self.addButton = [[UIBarButtonItem alloc]
                 initWithBarButtonSystemItem:UIBarButtonSystemItemAdd 
                 target:self action:@selector(btnNavAddPressed:)];
    self.addButton.style = UIBarButtonItemStyleBordered;
    [buttons addObject:self.addButton];

    [tools setItems:buttons animated:NO];

    self.navigationItem.rightBarButtonItem = [[[UIBarButtonItem alloc] initWithCustomView:tools] autorelease];

    self.addButton.enabled = FALSE;
}
- (void)dealloc {
    [_addButton release];
}
@end

Answer 3

上述两个答案都有误导性。您不需要使用 setter，直接将对象分配给 iVar 就完全没问题了。但是，您确实需要释放您分配或保留的任何内容。您遇到的问题在这里：

self.navigationItem.rightBarButtonItem = [[UIBarButtonItem alloc] initWithCustomView:tools];

这一行分配一个 UIBarButtonItem 实例并将其设置为 navigationItem 的 rightBarButtonItem 属性。这意味着 navigationItem 正在保留 UIBarButtonItem 并且它负责该保留。您有责任在 alloc 中释放它，但您没有。将代码更改为：

self.navigationItem.rightBarButtonItem = [[[UIBarButtonItem alloc] initWithCustomView:tools] autorelease];

这样泄漏就消失了。

Answer 4

您没有利用声明的属性，但我没有发现 addButton 有任何问题。泄漏似乎更多的是：

self.navigationItem.rightBarButtonItem = 
                             [[UIBarButtonItem alloc] initWithCustomView:tools];

只需添加 autorelease ，泄漏就会消失。