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

如果我初始化这个类，它会自动创建字典，如果我调用 set 或 get 方法之一，它似乎会保留字典，因为它会正确解除分配。如果类刚刚初始化，然后没有调用任何方法，它将抛出 EXC_BAD_ACCESS 错误。如果文件不存在时我没有弄错，我没有正确初始化字典，因为该方法以字典而不是 init 开头。虽然每次我运行这个文件都在那里，所以它总是使用文件找到的逻辑，我认为这会保留变量。

我没有正确初始化字典吗？

编辑 - 更改了便捷方法的代码以反映正确的方法。大家注意一下 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:] 返回一个自动释放的对象。如果您想将字典保留为 ivar 并且不让它在运行循环的下一个周期自动释放，您应该调用：

[currentEntry retain];

所以，基本上，将其更改为：

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

或：

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

第一个可能更有意义，因为便利类方法被设计为在您需要自动释放实例时使用。