NSDictionary 作为属性

Question

我使用以下代码收到指向“NSDictionary *dw = [NSDictionary DictionaryWithContentsOfFile:path];”行的内存泄漏

NSDictionary    *_allData;

@property (nonatomic, retain) NSDictionary  *allData;

@synthesize allData = _allData;

+ (NSString*)getNSPath

{
 NSArray *paths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES);

 NSString *documentsDirectory = [paths objectAtIndex:0];

 NSString *path = [documentsDirectory stringByAppendingPathComponent:@"alarm.plist"];

 return path;
}


- (NSDictionary *)allData
{
 NSString *path = [saveAlarm getNSPath];
 NSDictionary *dw = [NSDictionary dictionaryWithContentsOfFile:path];


 _allData = [NSDictionary dictionaryWithDictionary:dw];

    return _allData;
}

数据在 pList 中发生变化，当我要求检索那里的新内容时通过财产然后它泄漏。 有什么建议如何说清楚吗？或者说如何实现这种不泄露的事情呢？

谢谢

Answer 1

您需要在重新分配 _allData 之前释放它。分配时还需要保留它。

编辑：结合罗伯特的改进来摆脱不需要的 NSDictionary。

EDIT2：因为您要跨 API 边界返回一个对象，所以需要将其重新调整为自动释放的对象。

- (NSDictionary *)allData
{
     NSString *path = [saveAlarm getNSPath];
     [_allData release];
     _allData = [[NSDictionary dictionaryWithContentsOfFile:path] retain];

    return [_allData autorelease];
}

您发布的代码有点奇怪，因为您正在创建一个名为 allData 的属性，告诉它使用 _allData 作为 ivar （使用 @synthesize），然后实现一个设置 ivar 的自定义 getter。如果将该属性声明为只读，则可以删除 @synthesize 语句。

如果您仅在此方法中使用 _allData 而不是此类中的其他任何地方，则可以完全摆脱它。这是一个更简单的版本，可以做同样的事情：

- (NSDictionary *)allData
{
     NSString *path = [saveAlarm getNSPath];
     return [NSDictionary dictionaryWithContentsOfFile:path];
}

Answer 2

为什么不替换

NSDictionary *dw = [NSDictionary dictionaryWithContentsOfFile:path];


_allData = [NSDictionary dictionaryWithDictionary:dw];

为

_allData = [NSDictionary dictionaryWithContentsOfFile:path];

然后你不必担心 dw NSDictionary 的自动释放，这可能会导致你的泄漏。