NSURlConnection取消导致内存泄漏

Question

这真是一个奇怪的错误。

我使用代码通过 NSURLConnection 下载文件，如果下载完成，我没有泄漏。 但是如果我取消下载，我就有1Mo内存没有释放。 我已经用仪器进行了测试，确定创建此泄漏的方法

- (void)connection:(NSURLConnection *)connection didReceiveData:(NSData *)data

真的很奇怪

这是我用于创建、管理和取消下载的代码

注意：此下载是核心数据管理对象的一部分，但我认为核心数据是不对我的泄漏负责。

- (void)loadURL:(NSURL *)url
{

    if (currentLoadingDatas) { // just bool for prevent multiple download

        return;
    }
    currentLoadingDatas = YES;


    NSURLRequest *request = [[NSURLRequest alloc] 
                         initWithURL: url
                         cachePolicy:             NSURLRequestReloadIgnoringLocalAndRemoteCacheData
                         timeoutInterval: 60
                         ];
    connectionDatas = [[NSURLConnection alloc] initWithRequest:request delegate:self];
    [request release];

}

#pragma mark NSURLConnection Delegates
- (void)connection:(NSURLConnection *)connection didReceiveResponse:(NSURLResponse *)response
{

    if (!self.transientData) {

        self.transientData = [[NSMutableData alloc] init];
    }
    [self.transientData setLength:0];



}

- (void)connection:(NSURLConnection *)connection didReceiveData:(NSData *)data
{
    [self.transientData appendData:data];


}



- (void)connectionDidFinishLoading:(NSURLConnection *)connection
{

    [self willChangeValueForKey:@"datas"];
[self setValue:self.transientData forKey:@"datas"];
    [self didChangeValueForKey:@"datas"];

    [self.transientData release];


    NSURLCache *sharedCache = [[NSURLCache alloc] initWithMemoryCapacity:0 diskCapacity:0 diskPath:nil];
    [NSURLCache setSharedURLCache:sharedCache];
    [sharedCache release];



}

- (void)connection:(NSURLConnection *)connection didFailWithError:(NSError *)error {


    NSURLCache *sharedCache = [[NSURLCache alloc] initWithMemoryCapacity:0 diskCapacity:0 diskPath:nil];
    [NSURLCache setSharedURLCache:sharedCache];
    [sharedCache release];


}

- (NSCachedURLResponse *)connection:(NSURLConnection *)connection     willCacheResponse:(NSCachedURLResponse *)cachedResponse
{
    return nil;
}

-(void)cancelDownload
{
    [self.connectionDatas cancel];


}

// fired by coreData 
-(void)willTurnIntoFault
{


    self.transientData = nil;
     [self cancelDownload];

    [super willTurnIntoFault];
}

// fired by coreData 
-(void)didTurnIntoFault
{

    [connectionDatas release];

NSURLCache *sharedCache = [[NSURLCache alloc] initWithMemoryCapacity:0 diskCapacity:0 diskPath:nil];
    [NSURLCache setSharedURLCache:sharedCache];
    [sharedCache release];

    [self.transientData release];
    [super didTurnIntoFault];
}

你能帮我找出问题所在吗？

多谢。

Answer 1

self.transientData 属性是如何声明的？
因为您使用以下方式进行初始化： self.transientData = [[NSMutableData alloc] init]; 并且如果该属性设置为保留该值，则需要释放它两次。

如果是这样，要设置属性，请使用：self.transientData = [[[NSMutableData alloc] init] autorelease];或简单地[NSMutableData data];。
其余的对我来说似乎还不错。

Answer 2

泄漏表明您的变量在该点被实例化，因此这不是泄漏实际发生的地方，而是泄漏开始的地方。您需要像这样在取消方法中释放transientData。

- (void)cancelDownload
{
  [self.connectionDatas cancel];
  self.transientData = nil;
}

您的编码风格存在一些不一致的问题，这可能会让您更难在精神上跟踪正在发生的事情。如果您坚持自己的标准，那么遵循流程应该会更容易。

这可能是发生另一次泄漏的地方

- (void)connection:(NSURLConnection *)connection didReceiveResponse:(NSURLResponse *)response
{
    if (!self.transientData) {
        self.transientData = [[NSMutableData alloc] init]; // leaky line
    }
    [self.transientData setLength:0];
}

[[NSMutableData alloc] init] 调用正在创建一个保留计数为 +1 的 NSMutableData 实例。然后，如果您将属性（这是最好的）与 retain 一起使用，则 self.transientData = 将采取另一个保留，并在保留计数中添加另一个 +1。此后仅发布一次，因此您会发生泄漏，因为 NSMutableData 仍会挂起。

在代码中，您可以使用以下模式

1. 创建实例并分配给局部变量
2. 将实例分配给 ivar
3. 释放局部变量实例

这是当我不在 init 方法中时我将使用的模式。因此，之前的方法应该更改为：

- (void)connection:(NSURLConnection *)connection didReceiveResponse:(NSURLResponse *)response
{
    if (!self.transientData) {
        NSMutableData *tmpTransientData = [[NSMutableData alloc] init];
        self.transientData = tmpTransientData;
        [tmpTransientData release];
    }
    [self.transientData setLength:0];
}

这样做的好处是不使用自动释放，因此您可以更明确地了解何时不再需要该对象，这在内存较小的设备上可能是首选。

另一个可能有利于整理的不一致之处是释放 ivars 的方式。您在代码中可以互换地完成此操作，

[self.transientData release];
self.transientData = nil;

我将在我的代码中使用后者（不在dealloc中）作为实例变量，因为synthesized setter将调用release 并将指针设置为 nil 这要安全得多。