当分配给保留属性时，我应该保留自动释放的对象吗？

Question

在我保留的属性之一的 getter 中，我将 NSArray 分配并分配给我的模型对象（“models”）。然后我使用“sortedArrayUsingSelector:”方法对该 NSArray 进行排序。根据文档，这会返回一个自动释放的 NSArray。然后我将其重新分配给我的“模型”数组。

起初，我从未保留过这个自动释放的排序数组，并且发现当我从堆栈中弹出这个 viewController 时，我的应用程序会崩溃，说我正在尝试减少已释放对象或类似对象的引用计数。然后我添加了一个保留，如下面的代码所示，一切正常。

我的问题是，即使我在其声明中保留此属性并在我的 dealloc 中释放它，我应该保留自动释放的对象是否正确？

- (NSArray *)models {

    if (!models) {
        models = [[NSArray alloc] initWithArray:[self.modelDictionary allKeys]];
        models = [[models sortedArrayUsingSelector:@selector(compare:)] retain];
    }
    return models;

}

Answer 1

一般来说，如果您希望自动释放的对象保留下来，则必须保留它。

此代码存在几个问题：

- (NSArray *)models {

    if (!models) {
        models = [[NSArray alloc] initWithArray:[self.modelDictionary allKeys]];
        models = [[models sortedArrayUsingSelector:@selector(compare:)] retain];
    }
    return models;
}

• 您正在泄漏第二个 models= 行代码中 alloc/initd 的数组

• 您说“在其声明中保留此属性，但在上面您直接分配给实例变量的代码，绕过可能由 @synthesize

  生成的 setter 方法，

• 它是一个getter方法，可以改变所获取的变量的状态...而这种延迟初始化可能很有吸引力，但必须非常小心地完成。核心数据竭尽全力——通过相当多的内部复杂性——以允许以这种方式进行惰性状态填充。上面的代码还意味着 models 属性的键值观察器在 getter 执行时不会看到变化。然而，如果您从 getter 手动调用 willChangeValueForKey:/didChangeValueForKey:，您最终会从 getter 内部发送更改通知，很可能会导致某些内容消耗value before 调用 getter 的人首先检索该值。听起来很混乱？这可能会或可能不会引起问题。根据我的经验，它很可能有一天会导致问题。

  它

---

一些建议：

• 如果可以的话，使用核心数据。为了使 Core Data 非常高效并与系统很好地集成，我们投入了大量的工程工作。

• 将 getter/setter 与其余逻辑分开。更有可能的是，您的应用程序中存在一个逻辑点，您需要“获取”模型。当你这样做时，告诉对象通过其他一些方法来出错，然后获取模型。这可能看起来很痛苦，但与查询该图以获取信息相比，对象图的构建在很大程度上是一项专门的任务（或使用 Core Data，它有一个解决方案）。

Answer 2

除非我遗漏了一些东西，否则看起来您只是将结果设置为实例变量之一。我认为您应该使用您的属性。

self.models =  [NSArray arrayWithArray:[self.modelDictionary allKeys]];

如果没有 self. 部分，您就不会使用setter 方法，因此从不保留任何内容。

Answer 3

谢谢你！我明白你的意思。我想我很困惑，并认为在 getter 中调用 self.models 会创建无限循环。我现在明白了，这只是我从 getter 内部调用 getter，而不是从 getter 内部调用 setter。我将代码更改为：

- (NSArray *)models {
    if (!models) {
        self.models = [[NSArray alloc] initWithArray:[self.modelDictionary allKeys]];
        self.models = [models sortedArrayUsingSelector:@selector(compare:)];
    }
    return models;
}

一切都编译并正常工作...这看起来是一个更正确的解决方案吗？感谢您抽出时间。