自定义 NSDictionary getter 的正确、内存管理实现

Question

预先感谢...

所以在最近的一些内存泄漏和错误之后（记录这里），我一直在研究我的内存管理，现在我正在尝试在我拥有的辅助类中为 NSDictionary 编写一个自定义 getter。

自定义 getter 的原因是：

首先，它是一个计算字典，因此对于性能问题，我想返回缓存的对象，除非有必要重新生成它（我下面的示例在计算上并不太昂贵，但类中的其他示例将是）。

其次，我想延迟实例化。我只想在另一个类“请求”时生成字典，这就是我在 getter 中检查 nil 的原因。

那么，问题。下面的代码代表了我第一次（研究过）编写自定义 setter/getter 的尝试，我通常只是综合。

1）这是正确的实现和正确的内存管理吗？

2）这是最好的吗？

3) BOOL installedStandardsChangedSinceLastRead 是标记需要重新计算的最佳方法吗？如果类中的其他方法更改了某些内容，就会发生这种情况。但我是否应该让这些方法 nil out _installedStandards 来代替，因为这也会触发重新计算？

4）最后，如果我没有只是让其他方法将 iVar 清零以将其标记为重新计算，我如何确保不会泄漏？我会使用setter（即self.installedStandards = nil），直接nil iVar（即_installedStandards = nil），还是其他东西？

（哦，如果我直接取消 iVar，我需要先释放它吗？似乎正确的是 [_installedStandards release]; _installedStandards = nil？在这种情况下我不能只需使用self.installedStandards = nil？）

进入代码！ 提醒一下，这是一个更复杂的类的简化版本（请参阅顶部的链接），其中包含许多这些 setter/getter。我需要确保在充实它之前正确执行。

.h 文件

@interface InstalledStandardTracker20 : NSObject {

    NSDictionary *_installedStandards;
    BOOL _installedStandardsChangedSinceLastRead;
}

@property (nonatomic, retain) NSDictionary *installedStandards;
@property (nonatomic) BOOL installedStandardsChangedSinceLastRead;

@end

@implementation

@implementation InstalledStandardTracker20

@synthesize installedStandardsChangedSinceLastRead = _installedStandardsChangedSinceLastRead;

- (void)refreshInstalledStandards {
    NSUserDefaults *currentDefaults = [NSUserDefaults standardUserDefaults];
    self.installedStandards = [currentDefaults objectForKey:@"installedStandards"];
    self.installedStandardsChangedSinceLastRead = NO;
}

- (NSDictionary *)installedStandards {
    if (!_installedStandards || self.installedStandardsChangedSinceLastRead) {
        [self refreshInstalledStandards];
    }
    return _installedStandards;
}

- (void)setInstalledStandards:(NSDictionary *)newInstalledStandards {
    [newInstalledStandards retain];
    [_installedStandards release];
    _installedStandards = newInstalledStandards;
}

Answer 1

这个问题很难回答，因为课堂之外没有背景。该代码似乎没有内存泄漏。另外，您不需要实施 -setInstalledStandards:。由于您在保留属性上调用 @synthesize，生成的 setter 看起来就像这样。本质上，您正在使用完全相同的方法重写该方法。

对于您的另一个问题，清空 iVar 与内存中保存的对象无关。一旦释放 iVar，指针中包含的内存地址处的对象就不再保证存在。通过清空 iVar，您只需将该内存地址重置为指向 nil。这可以避免您收到 EXC_BAD_ACCESS 运行时异常，因为程序的其他部分不会尝试访问这个现在未定义的地址空间。

从本质上讲，你总是必须释放它。虽然将其置空是一种很好的做法，但只要您在知道该指针在内存中包含有效对象之前不尝试访问该指针，就不需要这样做。我希望这是有道理的。