在重用之前释放并在退出时释放的对象的潜在泄漏

Question

使用 xcode 分析器时，我收到关于对象潜在泄漏的警告。这个警告令人困惑，我需要一些解释为什么我会收到这个错误。下面是 mediaSources 保存指向相关对象的指针的代码：

在 .h 文件中，创建指向 MediaSources 类的指针并赋予保留属性：

@interface RootViewController : UIViewController <...>
{
    ...

    MediaSources                *mediaSources;

    ...
}

@property (nonatomic, retain)   MediaSources            *mediaSources;

.m文件中（rootViewController）是一个可以多次调用的方法。因此，我在每个条目上释放对象并分配一个新对象。 MediaSources 对象执行后台任务，所以在我知道它完成之前我不想释放它。如果我在分配类的行上使用 autoRelease，它会崩溃。 ：

-(void) getSelectedMediaSources
{
    [self setMediaSources: nil];    // release old stuff and nilify 
    [self setMediaSources: [[MediaSources alloc] init]]; 
    [self.mediaSources checkForMediaSourceUpdates];
}

同样在.m文件中，mediaSources也被合成，在dealloc中释放

@synthesize mediaSources;

...

- (void)dealloc {
    ...
    [mediaSources release];
    ...

    [super dealloc];
}

请解释我收到此警告的原因。我不明白怎么可能有泄漏。 Dealloc 应该释放这个对象的最后一个副本。

响应 checkForMediaSourceUpdates 的代码请求。这会变得有点复杂，但下面是本质：

(void) checkForMediaSourceUpdates
{    
    NSString *s = [NSString  stringWithFormat:@"http://www.mywebsite.com/mediaSources/%@/mediaSources.plist", countryCode];

    NSURL *url = [NSURL URLWithString:s];
    NSURLRequest *req = [NSURLRequest requestWithURL:url  cachePolicy:NSURLRequestReloadIgnoringLocalAndRemoteCacheData timeoutInterval:60.0];
    MyDownloader *d = [[MyDownloader alloc] initWithRequest:req];
    [self.connections addObject:d];
    [[NSNotificationCenter defaultCenter] addObserver:self  selector:@selector(checkForMediaSourceUpdatesDownloadFinished:) name:@"connectionFinished" object:d];
    [d.connection start];
    [d release];
}

-(void) checkForMediaSourceUpdatesDownloadFinished: (NSNotification *) n 
{
    MyDownloader *d = [n object];
    NSData *data = nil;
    if ([n userInfo]) {
        NSLog(@"In checkForMediaSourceUpdatesDownloadFinished: MyDownloader returned an error");
    }
    else {
        data = [d receivedData];

        // do something with the data
    }
}

myDownloader 类执行输入 NSURLRequest 中指定的文件下载。完成下载后，该类会生成一个名为“connectionFinished”的NSNotification。此类的用户必须注册此通知并处理此类的所有清理操作。如果下载失败，该类将生成一个 NSNotification，也称为“connectionFinished”，但添加了指示发生错误的 userInfo。同样，此类的用户必须注册此通知并处理此类的所有清理操作。

Answer 1

根据定义，您将自动释放的对象传递给合成的 setter。 setter 本身保留了对象，因此下面这行是错误的：

[self setMediaSources: [[MediaSources alloc] init]]; 

应该是：

[self setMediaSources: [[[MediaSources alloc] init] autorelease]]; 

你也不需要在之前用nil 调用你的setter。当通过 setter 设置不同的对象时，旧对象被释放。合成的 setter 看起来像：

- (void) setMediaSources:(MediaSources *)mediaSources {
  if (_mediaSources != mediaSources) {
    [_mediaSources release];
    _mediaSources = [mediaSources retain];
  }
}

问题是：为什么每次调用getSelectedMediaSources 时都需要分配一个新的MediaSource？释放自动释放的MediaSource 时，您期望发生什么崩溃？您是将其设置为另一个对象的委托还是将其注册到NSNotificationCenter？如果是这样，请不要忘记取消代理或将其从通知中心删除。

Answer 2

正如其他答案中已经提到的，您应该像这样将 autoreleased 实例传递给 setter：

[self setMediaSources: [[[MediaSources alloc] init] autorelease]]; 

您因为这样做而看到的崩溃似乎源于您在checkForMediaSourceUpdates 中进行后台处理的事实。如果要在此方法中进行后台处理，则应确保self 在后台任务的整个持续时间内保持有效。在开头手动调用[self retain]; 并在结尾手动调用[self release]; 是完全有效的。 （注意：GCD 块会自动保留/释放 self 对象，如果您从块内引用它）。

编辑：我希望你现在知道问题所在，mAu 称它为。在执行请求时，MediaSources 实例上只有一个保留计数（通过 RootViewController 对象）。在请求完成之前，您再次调用该方法，导致MediaSources 对象被释放。因此，当请求完成并且NSNotification 尝试发送回调消息时，应用程序崩溃，因为观察者实例已经消失。

有两种方法可以解决这个问题（使用更适合您的情况的方法）：

1. 在checkForMediaSourceUpdates 和[self release]; 的末尾添加[self retain]; checkForMediaSourceUpdatesDownloadFinished:
2. 或者，在-[MediaSources dealloc] 中添加[[NSNotificationCenter defaultCenter] removeObserver: self]。

顺便说一句，您应该考虑why do you need to allocate a new MediaSource on every call to getSelectedMediaSources? 以及为什么每次您可能有一个正在运行的请求时都启动一个新请求。你的设计有很大的问题。

Answer 3

[self setMediaSources: [[MediaSources alloc] init]]; 

您要保留新的 MediaSources 对象两次，一次是使用 alloc，另一次是使用 setMediaSources。你只释放一次。

试试：

[self setMediaSources: [[[MediaSources alloc] init] autorelease]]; 

从 cmets 跟进：

是的，您可以直接设置变量，但这并不能解决您的根本问题。听起来您现有的代码因为额外的保留而有效，并且在您平衡保留/释放时失败。您需要找出谁在保留对旧 mediaSources 对象的弱引用，并在释放它之前中断该引用。