iPhone NSCFString 在 fetchRequest 中泄漏

Question

在以下代码中：

- (NSMutableArray *) fetchNotesForGroup: (NSString *)groupName {

 // Variables declaration
 NSMutableArray *result;
 NSFetchRequest *fetchRequest;
 NSEntityDescription *entity;
 NSSortDescriptor *sortDescriptor;
 NSPredicate *searchPredicate;
 NSError *error = nil;

 // Creates the fetchRequest and executes it
 fetchRequest = [[[NSFetchRequest alloc] init] autorelease];
entity = [NSEntityDescription entityForName:@"Note" inManagedObjectContext:managedObjectContext];
[fetchRequest setEntity:entity];
sortDescriptor = [[[NSSortDescriptor alloc] initWithKey:@"noteName" ascending:YES] autorelease];
[fetchRequest setSortDescriptors:[NSArray arrayWithObject:sortDescriptor]];
[fetchRequest setReturnsDistinctResults:YES];
searchPredicate = [NSPredicate predicateWithFormat:@"categoryName like %@", groupName];
[fetchRequest setPredicate:searchPredicate];
[fetchRequest setPropertiesToFetch:[NSArray arrayWithObject:@"noteName"]];
result = [[managedObjectContext executeFetchRequest:fetchRequest error:&error] mutableCopy];

 // Variables release

 return result;
}

... I 获取给定类别名称的注释。当我运行 Instruments 时，它说 NSCFString 正在泄漏。

我知道泄密对于 iPhone 开发者来说是很严重的……但我不知道如何堵住这个漏洞。

有什么线索吗？欢迎所有帮助。

多谢！

Answer 1

您的问题是这样的：

result = [[managedObjectContext executeFetchRequest:fetchRequest error:&error] mutableCopy];
// Variables release
return result;

mutableCopy 返回一个拥有引用（即具有+1 保留计数的对象），您负责（自动）释放该引用。你不这样做，然后你放弃了引用，这意味着你已经泄漏了数组。

请改用 return [result autorelease];。

Answer 2

首先，仪器可能并不总是准确的。
它可以报告某些特殊情况下的泄漏，只是因为它看不到您实际上在其他地方释放了对象。

也有可能CF的某些部分有泄漏。

我在您的代码中看到的是您正在使用自动释放的对象。
你的临时对象将有很长的生命周期。
如果您在方法返回之前显式释放它们，则会追加什么？

您还将返回一个对象的副本。
您必须确保该对象在某个时刻被释放。
返回一个自动释放的对象，并让调用方法决定是否应该保留该对象可能会更好。

Answer 3

由于您没有创建任何字符串，因此泄漏字符串很可能是最终出现在提取请求中的字符串之一，并且很可能是实际泄漏的提取请求。 （Instruments 中显示的调用堆栈应该证实了这一点。）

我认为您没有实际的泄漏，但是通过这样做：

 fetchRequest = [[[NSFetchRequest alloc] init] autorelease];

...您允许 fetchRequest 超出其定义的范围，Instruments 会将其解释为泄露。

自动释放实际上使对象的生存时间比直接释放更长，因为它会导致对象挂起，直到外部自动释放池耗尽。例如，如果您在对象 A 内创建对象 B 并将其返回给自动释放的对象 C，则在对象 A 被释放后，对象 B 仍将保持活动状态很长时间，即使对象 C 从未保留它。 （尽管它最终会在不可预测的时刻消亡。）

自动释放并不是一种方便的保留方法。它具有保留在其他对象之间传递的对象的特定目的。如果您不这样做，请不要使用自动释放。

如果您这样做：

fetchRequest = [[NSFetchRequest alloc] init];
// ...
[fetchRequest release];

...您的泄漏将会消失。

但是，您可能想要这样做：

return [result autorelease];

...以确保结果数组的寿命足够长，以便由另一个对象保留。