iOS - 无法在 didSelectRowAtIndexPath 中访问 NSMutableArray？

Question

我无法在 didSelectRowAtIndexPath 中访问我的 NSMutableArray，尽管我可以在 cellForRowAtIndexPath 中访问它。

这是我的代码：-

- (void)viewDidLoad
{
    NSString *path = [[NSBundle mainBundle] pathForResource:@"drinks" ofType:@"plist"];
    self.drinkArray = [[NSMutableArray alloc] initWithContentsOfFile:path];
    NSLog(@"%@", self.drinkArray);
    [super viewDidLoad];
}

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
{
    static NSString *CellIdentifier = @"Cell";
    NSLog(@"I am inside cellForRowAtIndexPath");
    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {
        cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier] autorelease];
    }

    // Configure the cell.
    NSDictionary *dict = [self.drinkArray objectAtIndex:indexPath.row];
    cell.textLabel.text = [dict objectForKey:@"name"]; 
    cell.accessoryType = UITableViewCellAccessoryDisclosureIndicator;
    [dict release];
    return cell;
}

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{

    DrinkDetails *detailViewController = [[DrinkDetails alloc] initWithNibName:@"DrinkDetails" bundle:nil];
    // ...
    // Pass the selected object to the new view controller.
    //detailViewController.drink = [self.drinkArray objectAtIndex:indexPath.row];

    NSLog(@"%@", self.drinkArray);

    [self.navigationController pushViewController:detailViewController animated:YES];
    [detailViewController release];

}

有时 NSLog 会打印一些愚蠢的输出，有时它会给我一个错误“EXC_BAD_ACCESS”。

请看一下我的代码有什么问题。

任何帮助将不胜感激。

谢谢。

Answer 1

您有一个（实际上是两个）问题，但主要问题在您的 -cellForRowAtIndexPath: 方法中：

[dict release];

去掉这一行，它应该可以正常工作。

这解决了您的问题的原因是因为 -objectAtIndex: 只是返回一个指向内存中所请求对象的指针，因此您不会（也不应该）发送 -release 消息到该对象，因为 NSArray 在插入该对象时获得了该对象的所有权。向此对象引用发送 -release 可以有效地释放内存中的对象，现在 NSArray 中的该索引指向垃圾内存。 BAD BAD BAD

另一个问题是这里存在内存泄漏：

self.drinkArray = [[NSMutableArray alloc] initWithContentsOfFile:path];

您正在通过发送方式将 -retain 消息发送到您已经拥有所有权的对象引用-分配。 （当然，假设您的 @property 具有 retain setter 修饰符）

要解决此问题，只需将 -autorelease 消息发送到此实例：

self.drinkArray = [[[NSMutableArray alloc] initWithContentsOfFile:path] autorelease];

Answer 2

不要释放对象，除非该对象是使用 alloc 创建的，或者是通过以 new 或 copy 开头的方法获取的，或者显式 retain< /代码>'ed。 (NARC)

在这种情况下：

NSDictionary *dict = [self.drinkArray objectAtIndex:indexPath.row];

未归还保留，因此您不拥有所有权，不应释放它。

出于同样的原因：

self.drinkArray = [[NSMutableArray alloc] initWithContentsOfFile:path];

已分配，因此需要使用 convince 方法释放或获取它，该方法将返回自动释放的对象：

self.drinkArray = [NSMutableArray arrayWithContentsOfFile:path];

Answer 3

您不应该在 cellForRowAtIndexPath 中[dict release]。 objectAtIndex 不会保留它，因此当您释放它时，它可能会被从数组中清除。