Objective C NSMutableDictionary內存管理

Question

我有一個模型類，保持跟蹤記錄由多個視圖構建。它有一個NSMutableDictionary，它有我最終寫入數據庫的字段和值。它被保存到plist並在需要時裝回。我以爲我記錄了我的記憶，但是當我嘗試發佈字典時，它會拋出EXC_BAD_ACCESS。這是我的接口：

#import <Foundation/Foundation.h> 


@interface CurrentEntryModel : NSObject { 
NSMutableDictionary *currentEntry; 
} 

@property (nonatomic, retain) NSMutableDictionary *currentEntry; 
- (void) setValue: (NSString *)value; 
- (NSString *) getValue; 

@end 

我的理解是currentEntry應該保留，我將不得不在dealloc期間釋放它。

這裏是我的實現（這是不是整個類只是相關部分）：

#import "CurrentEntryModel.h" 


@implementation CurrentEntryModel 

@synthesize currentEntry; 

-(id) init { 
    if (self = [super init]) 
    { 
    //check for file 
    NSFileManager *fileManager = [NSFileManager defaultManager]; 
    NSString *file; 
    file = @"location.plist"; 

    if ([fileManager fileExistsAtPath:file]){ 
     NSLog(@"file exists"); 
     currentEntry = [[NSMutableDictionary alloc] initWithContentsOfFile:file]; 

    }else { 
     NSLog(@"file doesn't exist"); 
     currentEntry = [[NSMutableDictionary alloc ] initWithCapacity:1]; 

     NSDate *testDate = [NSDate date]; 

     [currentEntry setObject:testDate forKey:@"created"]; 

     [currentEntry writeToFile:file atomically:YES]; 

    } 

} 
return self; 
} 

- (void) setValue: (NSString *)value { 
[currentEntry setObject:value forKey:@"location"]; 
} 

- (NSString *) getValue { 
return [currentEntry objectForKey:@"location"]; 
} 


- (void) dealloc{ 
[currentEntry release]; 
[super dealloc]; 

} 

@end 

如果我初始化這個類會自動創建字典，如果我叫組的一個或得到它看起來像字典保留，因爲它會正確釋放。如果該類剛剛初始化，然後沒有方法被調用，它將拋出EXC_BAD_ACCESS錯誤。如果我沒有錯誤，當文件不存在時，我不正確地初始化字典，因爲該方法以字典而不是初始化開始。雖然每次運行該文件時都會使用該文件，但我認爲這將保留該變量。

我不正確初始化字典嗎？

編輯 - 更改便利方法上的代碼以反映正確的方式。每個人都注意到Squeegy必須說的話。

Answer 1

這是壞壞壞。

else { 
     NSLog(@"file doesn't exist"); 
     currentEntry = [[NSMutableDictionary alloc ] dictionaryWithCapacity:1]; 

dictionaryWithCapacity:是NSMutableDictionary一類方法，它返回一個自動釋放的對象，而你沒有retain它。所以運行循環結束，字典被自動釋放。然後你在你的dealloc中運行[currentEntry release]，它會爆炸，因爲該對象已被釋放。

您可能不想用initWithCapacity:代替。始終將alloc與以init開頭的方法配對。

---

此外，當使用像這樣的保留屬性時，我通常讓屬性找出這個對我來說，只適用於自動釋放對象。你只需要記住更少的規則，並且有更少的陷阱。

- (id)init { 
    // ... 
    self.currentEntry = [NSMutableDictionary dictionWithContentsOfFile:file]; 
    // ... 
} 

- (void)dealloc { 
    //... 
    self.currentEntry = nil; 
    //... 
} 

這種方式，你從來沒有打電話retain或release直接在物體上。根據我的經驗，這會導致更少的混淆錯誤。但這也是許多ObjC程序員的風格，並非所有人都同意。

Answer 2

約書亞 -

+ (id)dictionaryWithCapacity:(NSUInteger)numItems 

是NSDictionary中的類的方法。所以，當你調用它，它應該是：

[NSMutableDictionary dictionaryWithCapacity:1]; 

不：

[[NSMutableDictionary alloc] dictionaryWithCapacity:1]; 

此外，[的NSMutableDictionary dictionaryWithCapacity：]返回一個自動釋放的對象。如果你想保持字典作爲伊娃，而不是讓它自動釋放對運行循環的下一個循環，你應該叫：

[currentEntry retain]; 

所以，基本上，將其更改爲：

currentEntry = [[NSMutableDictionary alloc] initWithCapacity:1]; 

或：

currentEntry = [[NSMutableDictionary dictionaryWithCapacity:1] retain]; 

第一種可能更有意義，因爲設計的connivence類的方法，當你想要一個自動釋放的情況下使用。