2012-04-20 9 views
2

XMLファイル(プロジェクトのリソースとして添付)のすべての情報をコレクションに解析するXMLパーサーがあります。初めて私はNSMutableArrayを介してそれを行い、それをNSMutableDictionaryに変換し、すべて正常に動作します。 その後、私はそれがNSMutableArrayを持つ必要はないことを認識しました。私はちょうどNSMutableDictionaryに直接XMLを解析することができます。そして、私はそれを行いました(1つのプロパティの問題とコレクションにオブジェクトを追加する方法)が、今問題が発生しています。非常に奇妙なリーク。それは起こり得ず、できなかった。それが起こるたびに、それはハッシングとキー/バリューの問題についての何かが言います。コードの パーツ:NSMutableArrayとNSMutableDictionaryの奇妙な問題

- (void)parser:(NSXMLParser *)parser didStartElement:(NSString *)elementName namespaceURI:(NSString *)namespaceURI qualifiedName:(NSString *)qName attributes:(NSDictionary *)attributeDict{ 
     self.currentElement = elementName; 
     // category 
     if(self.targetBranch==XML_KEY_CATEGORY && [elementName isEqualToString:XML_KEY_CATEGORY]) { 
      self.categoryItem = [[CategoryItem alloc] initWithId:[attributeDict objectForKey:@"id"] andParentId:[attributeDict objectForKey:@"parentId"]]; 
      //NSLog(@"didStartElement with id %@",self.categoryItem.categoryId); 
     } 
    ... 
     } 

     - (void)parser:(NSXMLParser *)parser foundCharacters:(NSString *)string{ 
     ... 
      // categories 
      if (self.targetBranch==XML_KEY_CATEGORY && [self.currentElement isEqualToString:XML_KEY_CATEGORY]) { 
       //NSLog(@"adding category to array"); 
       if (self.categoryItem.categoryName) 
        self.categoryItem.categoryName = [NSString stringWithFormat:@"%@%@",self.categoryItem.categoryName,string]; //[string copy]; 
       else 
        self.categoryItem.categoryName = string; //[string copy]; 
     ... 
     } 


     - (void)parser:(NSXMLParser *)parser didEndElement:(NSString *)elementName namespaceURI:(NSString *)namespaceURI qualifiedName:(NSString *)qName{ 
      //NSLog(@"ended element: %@", elementName); 

      // category 
      if (self.targetBranch==XML_KEY_CATEGORY && [elementName isEqualToString:XML_KEY_CATEGORY]) { 
       //NSLog(@"adding category to array"); 
       //[xmlParsedInfo addObject:self.categoryItem]; 
       //  if (self.categoryItem.categoryId && self.categoryItem.categoryId && self.categoryItem.categoryId!=NULL 
     //  && ![xmlParsedData objectForKey:self.categoryItem.categoryId]){ 
     //   //NSLog(@"adding dictionary element %@",self.categoryItem.categoryId); 
     //   [xmlParsedData setObject:self.categoryItem forKey:self.categoryItem.categoryId]; 
        [xmlParsedInfo addObject:self.categoryItem]; 

       } 
     ... 
     } 

     -(NSDictionary*) getCategoriesForParentId:(NSString*)parentId_ fromUrl:(NSString *)strUrl{ 
      NSLog(@"getCategoriesForParentId: %@",parentId_); 
      targetBranch = XML_KEY_CATEGORY; 
      parentId = parentId_; 
      xmlParsedInfo = [[NSMutableArray alloc] init]; 
     // xmlParsedData = [[NSMutableDictionary alloc] init]; 
      [self parseXMLFileAtURL:strUrl]; 
      NSLog(@"category for parent id count is %i",xmlParsedInfo.count); 
      //return xmlParsedData; 
      NSDictionary *result = [self convertCategoryItemArrayToDictionary:xmlParsedInfo]; 
      [xmlParsedInfo release]; 
      return result; //[self convertCategoryItemArrayToDictionary:xmlParsedInfo]; 
     } 

     - (NSDictionary *) convertCategoryItemArrayToDictionary:(NSMutableArray *)array 
     { 
      NSMutableDictionary *mutableDictionary = [[NSMutableDictionary alloc] init]; 
      for (CategoryItem *catItem in array) { 
       [mutableDictionary setObject:catItem forKey:catItem.categoryId]; 
      } 
      return (NSDictionary *)[mutableDictionary autorelease]; 
     } 

私は、XMLパーサが正常に動作し、何の問題がまだ存在しない言ったように。しかし、辞書に関連するコメントのセクションが表示されます。現在の状態ではうまくいきますが、コメント配列関連のコードとアンコメント辞書が関連していれば - それは漏れるでしょう。ほとんどの時間は、私のような2つのリークを取得しています:

# Category Event Type Timestamp RefCt Address Size Responsible Library Responsible Caller 
0 CFBasicHash (key-store) Malloc 1928615936 1 0x9019600 512 MyApp -[XMLParser parser:didEndElement:namespaceURI:qualifiedName:] 
# Category Event Type Timestamp RefCt Address Size Responsible Library Responsible Caller 
0 CFBasicHash (value-store) Malloc 1928608768 1 0x900c600 512 MyApp -[XMLParser parser:didEndElement:namespaceURI:qualifiedName:] 

およびそれらのリークは、次のコード行に直接つながる:

 [xmlParsedData setObject:self.categoryItem forKey:self.categoryItem.categoryId]; 

そこでここで辞書に問題は何ですか?私はそのハッシュ/キー関連とそのNSMutableDictionary内部の問題を見ることができます。

私はエミュレータでコードを試しています。だから私はまだ問題が本当のデバイスで持続するかどうかは言えません。たぶんそのエミュレータに関連するバグ(キーボードリークのような)かもしれません。
私は解析しているXMLは静的で、プロジェクト内の単なるファイルです。このXMLファイルには71のカテゴリがあります。したがって、すべての実行アプリは同じデータで同じことを行います。だからイムは問題が奇妙だと言っている。最初の実行時には、リークはありません。リークはありません。 2番目の試行では、1または2になる可能性があります。次のすべての実行では、常に2(上に示した)のリークが表示されます(+1は辞書自体につながりますが、キーや値のリーク、 。

CategoryItem.h 
@interface CategoryItem : NSObject { 
    NSString * categoryName; 
    NSString * categoryId; 
    NSString * parentId; 
} 
@property (nonatomic,retain) NSString * categoryName; 
@property (nonatomic,retain) NSString * categoryId; 
@property (nonatomic,retain) NSString * parentId; 

- (id)init; 
- (id)initWithId:(NSString *)id_ andParentId:(NSString *)parentId_; 
- (id)initWithName:(NSString *)name_ andId:(NSString *)id_ andParentId:(NSString *)parentId_; 
- (BOOL)isEqualToItem:(CategoryItem *)anItem; 
- (NSUInteger)hash; 

@end 

CategoryItem.m 

#import "CategoryItem.h" 

@implementation CategoryItem 

@synthesize categoryName; 
@synthesize categoryId; 
@synthesize parentId; 

-(id)init{ 
    self = [super init]; 
    return self; 
} 

-(id)initWithName:(NSString *)name_ andId:(NSString *)id_ andParentId:(NSString *)parentId_{ 
    self = [super init]; 
    if(self){ 
     self.categoryName = name_; 
     self.categoryId = id_; 
     self.parentId = parentId_; 
    } 
    return self; 
} 

-(id)initWithId:(NSString *)id_ andParentId:(NSString *)parentId_{ 
    self = [super init]; 
    if(self){ 
     self.categoryId = id_; 
     self.parentId = parentId_; 
    } 
    return self; 
} 

- (void)dealloc { 
    [categoryName release]; 
    [categoryId release]; 
    [parentId release]; 
    [super dealloc]; 
} 

- (BOOL)isEqual:(id)other { 
    if (other == self) 
     return YES; 
    if (!other || ![other isKindOfClass:[self class]]) 
     return NO; 
    return [self isEqualToItem:other]; 
} 

- (BOOL)isEqualToItem:(CategoryItem *)anItem { 
    if (self == anItem) 
     return YES; 
    if (![(id) self.categoryName isEqual:anItem.categoryName]) 
     return NO; 
    if (![(id) self.categoryId isEqual:anItem.categoryId]) 
     return NO; 
    if (![(id) self.parentId  isEqual:anItem.parentId]) 
     return NO; 
    return YES; 
} 

- (NSUInteger)hash { 
    NSString *string4Hash = [NSString stringWithFormat:@"%@%@%@",self.categoryName,self.categoryId,self.parentId]; 
    NSUInteger hash = [string4Hash hash]; 
    return hash; 
} 

@end 

を追加しました: 私が見つけた解決策は、私の中で(私のためとして愚かな)である - (NSDictionaryの*)getCategoriesForParentId:

リターンこの:

NSDictionary *result; 

    result = (NSDictionary *)[xmlParsedData copy]; 
    [xmlParsedData release]; 

    return result; 

OR!

return [(NSDictionary *)xmlParsedData autorelease]; 

} 

したがって、NSDictionaryMutableをコピーするのがポイントです...私はなぜそれが助けになるのか理解できません。保持しないでコピー! getCategoriesForParentId()を呼び出すメソッドは、NSDictionary(変更可能ではない)を要求します。これは、ALLOC +のinitを経由して、追加することなく、私も(NSDictionaryの*)のようなものを返すようにしようとした

- (void)dealloc { 
    [currentElement release]; 
    [targetBranch release]; 

    xmlParsedInfo = nil; 
    xmlParsedData = nil; 
... 

とのdeallocを持って、それがXMLParserの作成リリース辞書[xmlParsedDataが保持]または(NSDictionaryの*)xmlParsedDataを取得した後のXMLParserを作成しますNSDictionary * result - しかし、そこに漏れがあった。

私はxmlParsedDataをプロパティ+ @ synthesizeとして使用していないようです。だから、私はそれをプロパティ(retain)にし、現在は[xmlParsedData release]をdeallocにして、getCategoriesForParentIdから[xmlParsedData retain]を返します。まだ漏れている。

答えて

1

これが問題なのかどうかわかりませんが...categoryItemクラスを投稿できますか?または...あなたはそれにハッシュメソッドを実装しましたか?

+0

はい、私はハッシュ+に等しい実装しました。しかし、ハッシュの実装についてコメントしても同じ問題が発生します。だから問題ではない。 – Stan

5

あなたのコードはARCを使用していないようですが、質問に明記してください。

に入力するたびにメモリがリークし、didStartElementブランチになり、self.categoryItemに新しいエレメントが割り当てられます。あなたは新しいオブジェクトを割り当てていますが、それをプロパティに割り当てるともう1つretainが発行されます。次にオブジェクトを割り当てるときは、プロパティはreleaseを発行しますが、alloc以降は参照カウンターがもう1つ必要になります。

それがうまく動作させるために、問題作成時に自動解放または解放あなたはプロパティに割り当てられた後:

self.categoryItem = [[[CategoryItem alloc] 
     initWithId:[attributeDict objectForKey:@"id"] 
     andParentId:[attributeDict objectForKey:@"parentId"]] autorelease]; 
+1

+1ですが、 'self.categoryItem'は* any * kindのプロパティであればリークします。 assignプロパティの場合、解放されずに次の 'didStartElement:'に上書きされます。 – JeremyP

+0

それを指摘していただきありがとうございます、あなたは絶対に正しいです:最終的にコードは、プロパティのタイプに関係なく(またはそれがプロパティではなくても)すべての原因でメモリをリークします。 – MrTJ

+0

FOA私はARCあなたはどういう意味ですか?あなたの提案について:第一に、これが問題ならば、私は1〜2リーク以上を見なければならない。毎回71のカテゴリレコードが作成されています。だから私はCategoryItemの71リーク分を取得する必要がありますが、そのようなリークはありません。しかし、とにかく私はあなたのコードを試しましたが、運はありません。 SetterはCategoryItemのオブジェクトへのリンクを1つ解放しますが、リンクは1つのままにするか、辞書でどのように使用できますか? CategoryItemへの最後のリンクは辞書で解放されると思います。 – Stan