定位内存泄漏/过度释放崩溃的根源

Question

谁能帮我弄清楚我应该在哪里发布exerciseArray？当我在调用 sortedArray 之前释放时，我在 dealloc 中发生崩溃。该代码被调用多次。

//Set exerciseArray
review.exerciseArray = [[workout assignedExercises] allObjects];

---

- (void)viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];
    NSLog(@"viewWillAppear");
    NSString *workoutName =  [event.assignedWorkout valueForKey:@"name"];
    self.title = workoutName ;

    NSSortDescriptor *sortDescriptor = [[[NSSortDescriptor alloc] initWithKey:@"index"
                                                                    ascending:YES] autorelease];
    NSArray *sortDescriptors = [NSArray arrayWithObject:sortDescriptor];
    [sortedArray release];
    sortedArray= [exerciseArray sortedArrayUsingDescriptors:sortDescriptors];
    NSLog(@"*****SORTEDARRAY****** %d",[sortedArray retainCount]);
    for (Exercise *ex in sortedArray){
        NSLog(@"%@ %@ %@",ex.name , ex.repCount, ex.weight);
    } 
    NSLog(@"*********************");
    [sortedArray retain];
    [self.tableView reloadData];
}

 - (void)dealloc {
    [super dealloc];

    [managedObjectContext release];
    [event release];
}

Answer 1

首先，您应该将 [super dealloc] 调用移至 dealloc 方法的末尾。首先处理您的自定义变量，然后您要做的最后一件事是将dealloc推送到超类以处理其余的清理工作。

---

现在，我担心其中的 [sortedArrayrelease] 和 [sortedArrayretain] 。为了节省在 sortedArray ivar 上实现 retain/release 的语义，您应该做的就是声明 的属性sortedArray 像这样：

// this goes in your @interface
@property (nonatomic, retain) NSArray *sortedArray;

// this goes in your @implementation
@synthesize sortedArray;

现在您可以轻松设置 sortedArray ，而不必担心保留或释放：

// note the use of self, this is required
self.sortedArray = [exerciseArray sortedArrayUsingDescriptors:sortDescriptors];

您可以删除 release 和 retain 调用现在，因为这是自动处理的。确保在 dealloc 方法中添加一行来清理变量......这

self.sortedArray = nil;

将为您调用数组上的 release 。

编辑：这也适用于exerciseArray，这是您的实际问题。无论何时涉及 Cocoa 类，您都可以使用 @property (retain) 和 @synthesize 来简化内存管理。对于 NSInteger 和其他基元类型，或者当您不想持有对该对象的拥有引用时，请改用 @property (assign) 。