Устранить утечку памяти NSMutableArray

Question

У моего объекта есть частные элементы NSMutableArray. Я использую следующий код для сортировки объектов элементов в порядке размера:

-(void)sortItems{ 
NSSortDescriptor *sortDescriptor = [[NSSortDescriptor alloc] initWithKey:@"size" ascending:YES];  
NSArray *sortDescriptors = [NSArray arrayWithObject:sortDescriptor]; 
NSArray *sortedArray = [items sortedArrayUsingDescriptors:sortDescriptors]; 
NSMutableArray* newArray = [[NSMutableArray alloc] initWithArray: sortedArray]; 
[self setItems:newArray]; 
[sortDescriptor release]; 

}

Очевидно, что это утечка памяти здесь, потому что каждый раз, когда я называю sortItems, я allocing новой памяти и назначение чтобы указать на него. Я пытался выпускать старую память следующим образом:

NSMutableArray* newArray = [[NSMutableArray alloc] initWithArray: sortedArray]; 
NSMutableArray* oldArray = [self items]; 
[self setItems:newArray]; 
[oldArray release]; 

Но это дает ошибку EXC_BAD_ACCESS. Я прочитал об обработке памяти в objC, и я убежден, что я делаю что-то принципиально неправильное здесь.

Любая помощь была бы принята с благодарностью!

Answer 1

Вы протечка нового массива, а не старый:

NSMutableArray* newArray = [[NSMutableArray alloc] initWithArray: sortedArray]; 
[self setItems:newArray]; 
[sortDescriptor release]; 
[newArray release]; // <-- add this 

Основное правило в том, что вы должны освободить все, что вы выделены, и вы, как правило, не должны заботиться о сохранении вещи, оставленные для кого-либо (т. е. [self setItems:]), те, кому что-то нужно сохранить, сделают это сами.

Я бы также рекомендовал сделать self.items изменчивым массивом и использовать [self.items sortUsingDescriptors:sortDescriptor для сортировки на месте без создания копии.

Answer 2

Есть ли причина, по которой вы не можете освободить newArray в первом примере?

NSSortDescriptor *sortDescriptor = [[NSSortDescriptor alloc] initWithKey:@"size" ascending:YES];  
NSArray *sortDescriptors = [NSArray arrayWithObject:sortDescriptor]; 
NSArray *sortedArray = [items sortedArrayUsingDescriptors:sortDescriptors]; 
NSMutableArray* newArray = [[NSMutableArray alloc] initWithArray: sortedArray]; 
[self setItems:newArray]; 
[newArray release]; 
[sortDescriptor release];