带有 NSMutableArray 的 iphone EXC_BAD_ACCESS

Question

好的，所以我有一个 UIViewTable 和一个带有两个范围按钮的 UISearchBar。这个想法是，当我按下范围按钮时， UIViewTable 的数据源发生了变化，但我得到了 EXC_BAD_ACCESS 错误。

我的 UIViewController SearchViewController.m 中有以下代码:

- (void)searchBar:(UISearchBar *)searchBar selectedScopeButtonIndexDidChange: (NSInteger) selected scope
{
    MyAppDelegate *delegate = (MyAppDelegate *) [[UIApplicationsharedApplication] delegate];
    if (self.listData != nil) {
        [self.listData release];
    }
    if (selectedScope == 0) {
        self.listData = [delegate.data getListOne];
    }
    else {
        self.listData = [delegate.data getListTwo];
    }
}

- (void) viewDidLoad {
    MyAppDelegate *delegate = (MyAppDelegate*) [[UIApplication sharedApplication] delegate];
    self.listData = [delegate.data getListOne];

    //some other unrelated code
}

在我的 SearchViewController.h 中我有:

@property (nonatomic,retain) NSMutableArray *listData;

在我的 Data.m 中我有:

-(NSMutableArray *) getListOne {
     NSMutableArray *list = [[NSMutableArray alloc] initWithObjects:@"test1",
                                                                    @"test2",
                                                                    nil];
     [list autorelease];
     return list;
}

-(NSMutableArray *) getListTwo {
     NSMutableArray *list = [[NSMutableArray alloc] initWithObjects:@"test3",
                                                                    @"test4",
                                                                    nil];
     [list autorelease];
     return list;
}

它崩溃于:

self.listData = [delegate.data getListTwo];

我检查过当我设置它崩溃的属性时。我的理解是，当我在 Data.m 中创建新的 NSMutableArray 时，我会按我应该的方式为其分配自动释放。

当 View 加载时，我将它分配给我的 listData，并且由于我正在访问已保留的属性，因此引用计数会增加(因此它现在有 2 个挂起的自动释放)。

当我按下按钮更改数据源时，我也会检查 listData 是否存在(它总是会存在)，然后释放它，以便旧的 NSMutableArray 计数器将为 0(假设发生了自动释放)。

然后我得到一个新的 NSMutableArray 并将其设置为该属性...我的理解是否正确？我在这个简单的问题上花了太长时间:(

哦，我也确实创建了另一个未连接到 tableView 的 NSMutableArray，但仍然遇到同样的问题，如果我不在我的 if 语句中释放它，问题不存在，但我会发生内存泄漏？ ？我总是可以只保留数组并删除/添加对象，但我想知道为什么这不起作用:) 干杯

Answer 1

这是你的问题:

if (self.listData !=nil)
{
    [self.listData release];
}

您不需要执行此检查——由于您使用 retain 属性声明了 listData 属性，合成的 setter 会自动处理释放旧值。合成的 setter 看起来像这样:

- (void) setListData:(NSMutableArray *)listData
{
    [listData retain];
    [self->listData release];
    self->listData = listData;
}

这里注意几点:旧值被释放，新值被保留。此外，在自分配的情况下，保留发生在释放之前:如果您分配相同的值，您不希望它被过早地释放。另请注意，如果新值或旧值是 nil，则不会发生任何不良情况，因为 Objective-C 明确允许您将消息发送到 nil，但没有任何效果。

因此，这意味着无论何时设置属性，您都无需担心释放旧值——setter 会为您完成。因为您正在进行额外的释放，该对象在您实际使用完它之前就被释放了，所以一旦您在它被释放后使用它，您就会得到 EXC_BAD_ACCESS。