iPhoneでメモリリークを修正する方法がわからない

Question

Leaksツールを実行していて、辞書のmutableDeepCopyで大量のリークが検出されましたが、コードに何が間違っているかわかりません。助言がありますか？

@interface RootViewController : UIViewController{ 

    NSDictionary *immutableDictionary; 
    NSMutableDictionary *mutableDictionary; 
} 

ここでここで楽器に

self.mutableDictionary = [self.immutableDictionary mutableDeepCopy]; 

を強調表示のコードの行は、ここで辞書

@interface NSDictionary(MutableDeepCopy) 
    -(NSMutableDictionary *)mutableDeepCopy; 
@end 

の可変コピーを作成するための方法であるが、私は「、メソッドの実装です漏れが100％漏れているというコードを強調しました

- (NSMutableDictionary *) mutableDeepCopy { 
    NSMutableDictionary *dictionaryToReturn = [NSMutableDictionary dictionaryWithCapacity:[self count]]; 
    NSArray *keys = [self allKeys]; 

    for(id key in keys) { 
     id value = [self valueForKey:key]; 
     id copy = nil; 
     if ([value respondsToSelector:@selector(mutableDeepCopy)]) { 
      copy = [value mutableDeepCopy]; 
     } else if ([value respondsToSelector:@selector(mutableCopy)]) { 
      copy = [value mutableCopy]; //This is the Leak 
     } 
     if (copy == nil) { 
      copy = [value copy]; 
     } 
     [dictionaryToReturn setValue:copy forKey:key]; 
    } 
    return dictionaryToReturn; 
} 

Answer 1

アップルのMemory Management Rulesに照らしてこれを分析する必要があります。この行を皮切り

：

self.mutableDictionary = [self.immutableDictionary mutableDeepCopy]; 

私はmutableDeepCopyは私が所有するオブジェクトを返すことを期待するので、いくつかの点で私はそれを解放するか、自動解放する必要があります。例えば

NSMutableDeepCopy* temp = [self.immutableDictionary mutableDeepCopy]; 
self.mutableDictionary = temp; 
[temp release]; 

または

self.mutableDictionary = [[self.immutableDictionary mutableDeepCopy] autorelease]; 

だから今、私たちはmutableDeepCopyを見てする必要があります。名前に「コピー」があるので、実際には返されたオブジェクトを解放することを「忘れる」ことを意味する「所有」オブジェクトを返す必要があります。最初の行に返されたオブジェクトを作成するときには、dictionaryWithCapacity：は所有していないオブジェクトを提供しているので、すでにそれを実行していません。

NSMutableDictionary *dictionaryToReturn = [[NSMutableDictionary alloc] initWithCapacity:[self count]]; 

これであなたはそれを所有しています。

mutableDeepCopyは、mutableDeepCopy、mutableCopy、およびcopyから返されたオブジェクトをまったく同じ方法で扱うことができるため、ルールに従うことが重要です。 3つのケースでは、配列に挿入するオブジェクトコピーを所有しています。あなたが所有しているので、あなたはそれを解放しなければなりません。そうしないと、漏れてしまいます。だから、ループの最後に、あなたが必要です

[copy release]; 

それはリークを停止します。

Answer 2

mutableCopyは、オブジェクトの保持カウントを増やします（setValue:forKey:）。これは、dictionaryToReturnがdeallocされている場合、mutableCopyがまだ呼び出されていたオブジェクトの保持カウントが1であることを意味します。

は、代わりにこれをやってみてください。

copy = [[value mutableCopy] autorelease]; 

Answer 3

あなたの財産が宣言されている方法は？これがretainまたはcopyの場合、これはリークしません。

問題は、名前がmutableDeepCopyの場合、保持されているオブジェクトが返され、実際にはオートレリースされたオブジェクトは返されないということです。

編集： そしてmutableDeepCopy自体で、あなたは辞書に追加した後copy変数を解放する必要があります。