MemoryManagement lors de l'obtention d'un NSMutableArray de la classe NSObject à la classe UIViewController

Question

J'ai problème avec le code ci-dessous une fuite de mémoire ...

@property (nonatomic, retain) NSMutableArray *childrensArray; 


-(void)connectionDidFinishLoading:(NSURLConnection *)connection { 

NSLog(@"Connection finished loading."); 
// Dismiss the network indicator when connection finished loading 
[UIApplication sharedApplication].networkActivityIndicatorVisible = NO; 

// Parse the responseData of json objects retrieved from the service 
SBJSON *parser = [[SBJSON alloc] init]; 

NSString *jsonString = [[NSString alloc] initWithData:responseData encoding:NSUTF8StringEncoding]; 
NSDictionary *jsonData = [parser objectWithString:jsonString error:nil]; 
childrensArray = [jsonData objectForKey:@"Children"]; 

// Callback to AttendanceReportViewController that the responseData finished loading 
[attendanceReportViewController loadChildren]; 

[connection release]; 
[responseData release]; 
[jsonString release]; 
[parser release]; 
} 

Dans le viewController ce qui suit fuites mémoire aussi ...

@property (nonatomic, retain) NSMutableArray *childrensArray; 


- (void)loadChildren { 

// Retrieve a array with dictionaries of children from ServiceGetChildren 
self.childrensArray = [[serviceGetChildren.childrensArray copy] autorelease]; 

int total = [childrensArray count]; 
totalLabel.text = [NSString stringWithFormat:@"%d", total]; 

[theTableView reloadData]; 
} 

Answer 1

Vous ne publiez que childrensArray lorsque l'instance est désallouée. Vous devez également libérer la variable d'instance avant de le placer:

- (void)loadChildren { 
    // Retrieve a array with dictionaries of children from ServiceGetChildren 
    [childrensArray release]; 
    childrensArray = [serviceGetChildren.childrensArray copy]; 
} 

Une meilleure façon serait d'utiliser réellement votre propriété:

- (void)loadChildren { 
    // Retrieve a array with dictionaries of children from ServiceGetChildren 
    self.childrensArray = [[serviceGetChildren.childrensArray copy] autorelease]; 
} 

(Notez que le autorelease)

Ceci a l'avantage de déclencher des notifications KVO si jamais vous les utilisez.