Что вызывает утечку памяти?

Question

Что вызывает утечку в этом коде? Я действительно не могу этого понять. В строках: 1: NSMutableArray * days = [[NSMutableArray alloc] init]; 2: [dic setObject: days forKey: key]; 3: [days addObject: значение];

Весь метод:

-(void) addValueToDictionary: (NSMutableDictionary *) dic withValue: (NSNumber *) value forKey: (NSString *) key { 
NSMutableArray * days = [dic objectForKey:key]; 
if (days == nil) { 
    NSMutableArray * days = [[NSMutableArray alloc]init]; 
    [days addObject:value]; 
    [dic setObject:days forKey:key]; 
    [days release]; 
    days = nil; 
} 
else { 
    [days addObject:value]; 
} 

}

BR // Кристоффер

Answer 1

Проверьте, не выпущен ли dic. Вы должны NSLog keepCount раньше, чем считаете окончательные версии, и убедитесь, что они равны 1 перед окончательной версией.

Кроме того, запустите сборку и анализ, чтобы убедиться, что вы правильно освобождаетесь. Встроенная функция Build and Analyze не обнаруживает столько утечек, что работает с проверкой-сборкой со всеми проверками, поэтому изучите установку scan-build в Xcode.

Using an external Xcode Clang Static Analyzer binary, with additional checks

Answer 2

Вы пытались изменить имя переменной NSMutableArray дней * в пределах если? Из-за этого вы не получите предупреждение?

Answer 3

Там нет ничего плохого в том, что определенной части коды (кроме немногих сомнительного переопределения дней во внутреннем объеме). Где-то еще вы сохраняете, но забываете освободить объект, который вы помещаете в словарь.

Answer 4

Вы должны получить предупреждение о повторном объявлении days. Это может привести к утечке утечки, если вы используете статический анализатор. Модифицированный метод ниже. В основном меняется стиль кодирования с добавлением небольшого защитного кода.

-(void) addValueToDictionary: (NSMutableDictionary *) dic withValue: (NSNumber *) value forKey: (NSString *) key 
{ 
    if (nil == dic || nil == key || nil == value) return; // bail out on nil parameters 
    if (![dic objectForKey:key]) { 
     NSMutableArray * days = [[NSMutableArray alloc] init]; 
     [dic setObject:days forKey:key]; 
     [days release]; 
    } 
    [[dic objectForKey:key] addObject:value]; 
} 

Answer 5

изменения инициализации NSMutableArray в ...

NSMutableArray * days = [NSMutableArray array];