0

我正在明智地上传图像块，在后台线程中，每个块的大小为 512kb，据我所知，我已经使用 release、nsautoreleasepool 处理了内存泄漏。下面是分块上传图片的代码。 

- (void)FetchDataFromDB : (NSNumber *) isOffline
{
 @autoreleasepool {
    @try {
    NSLog(@"FetchDatafromDB");
    isThreadStarted = YES;

    VYukaDBFunctions *vdb = [VYukaDBFunctions getInstance];

    NSMutableArray *fileNames = [vdb GetFileNames:[isOffline integerValue]];
    for(int j=0 ; j<[fileNames count] ; j++)
    {

        NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init];
        NSString * filename = fileNames [j] ;
        int _outgoingMsgId = [[vdb SelectMsgId:filename] intValue];
        int  _totalchunk =[[vdb SelectTotalChunk:filename]intValue];
        int currentChunk = [vdb GetCurrentChunk:filename];
    for( int i=currentChunk ; i <= _totalchunk ; i++)
    {

        NSAutoreleasePool *innerPool = [[NSAutoreleasePool alloc] init];

        NSString *AsyncRequest = [[NSString alloc] init];
        AsyncRequest = [vdb SelectAsyncRequest: i : _outgoingMsgId];

        if(![AsyncRequest isEqual:@""])
        {

        BOOL status = [self UploadChunkWise :AsyncRequest : 1 : i : vdb : filename : _outgoingMsgId];
      // AsyncRequest = NULL;
     //  [AsyncRequest release];
        if(status){
            if(i==_totalchunk)
            {
                NSLog(@"Deleting from medialist , FileName :%@", filename);
                [vdb DeleteFromMediaList : filename];

            }
        }
        else{

            [vdb DeleteFromMediaList : filename];
            break;
        }
        }

        [innerPool drain];
    }

        [pool drain];
    }

    [fileNames removeAllObjects];

   // [fileNames release];

    //recurssive call to check any pending uploads..
    if([[vdb GetFileNames:[isOffline integerValue]] count] > 0)
    {
        NSLog(@"Calling Recursively..");
        [self FetchDataFromDB:[isOffline integerValue]];
    }

    }
    @catch (NSException *exception) {

        NSLog(@"Exception caught on Uploading from FetchDataFromDB:%@", exception);

    }
    @finally {

    }
}

 NSLog(@"thread quit ");
 isThreadStarted = NO;
[NSThread exit];

}

-(BOOL) UploadChunkWise :(NSString *) AsyncRequest : (int) count : (int)currentChunk : (VYukaDBFunctions * ) vdb : (NSString *) currentFileName : (int) outgoingMsgId
{

NSHTTPURLResponse *response ;
NSError *error;
//Yes, http
NSMutableURLRequest *httpRequest = [[NSMutableURLRequest alloc] initWithURL:[NSURL     URLWithString:@"Url goes here"]];
NSData* data = [AsyncRequest dataUsingEncoding:NSUTF8StringEncoding];

[httpRequest setHTTPMethod:@"POST"];
[httpRequest setHTTPBody:data];

[httpRequest setValue:@"application/xml" forHTTPHeaderField:@"Content-Type"];
NSData *returnedData = [NSURLConnection sendSynchronousRequest: httpRequest returningResponse:&response error:&error] ;
NSString *result= [[NSString alloc] initWithData:returnedData encoding:NSASCIIStringEncoding];
[httpRequest release];

returnedData= NULL;
[returnedData release];
data = NULL;
[data release];

if ([result rangeOfString:@"success"].location != NSNotFound )
{
    NSLog(@" success");
    [vdb DeleteCurrentChunkFromOutgoingTable:currentChunk : outgoingMsgId];
    [result release];
    return YES ;

}
else if ([result rangeOfString:@"fail"].location != NSNotFound )
{
    [result release];
    if (count < 3) {

        return  [self UploadChunkWise :AsyncRequest : count+1 : currentChunk: vdb : currentFileName : outgoingMsgId ];
    }
    else
    {
        NSLog(@" failed");
        [vdb DeleteAllCurrentFileChunksFromOutgoingTable:currentFileName];
        return NO ;
    }

}

return NO;

}

我正在启动线程如下

 [NSThread detachNewThreadSelector:@selector(FetchDataFromDB:) toTarget:self withObject:[NSNumber numberWithInt:0]];

问题是在上传 9 到 12 个块后，我收到内存错误。我收到 4 到 5 次内存警告，然后应用程序崩溃。在控制台中，我首先在应用程序委托类中收到内存警告，然后是 4 个扩展 UIViewController 的类。为什么我在应用程序委托和其他类型为 UIViewController 的类中收到警告。为什么如果单独的线程给我内存错误，我必须释放其他类的对象？我在这里做错了什么？我不能使用 ARC，因为我已经将它与不使用 ARC 的旧代码集成在一起，我尝试明智地启用 ARC 类，但它不起作用。任何人都可以帮助我找出这段代码中是否存在任何内存泄漏。欢迎和赞赏建议。在此先感谢..

4

1 回答 1

1

两件事——首先，我看到了这个：

NSString *AsyncRequest = [[NSString alloc] init];
AsyncRequest = [vdb SelectAsyncRequest: i : _outgoingMsgId];

这应该合并为：

NSString *asyncRequest = [vdb SelectAsyncRequest: i : _outgoingMsgId];

相反，您正在创建一个新实例，然后立即生成或引用另一个实例。

第二：

您的代码很难阅读，并且不遵循 Objective-C smalltalk 约定。

变量名应以lowercase字母开头。方法名称也应该以小写字母开头。类名和函数应以大写字母开头。它使阅读变得困难，因为我和许多其他人都受过训练，可以看到大写字母并认为 CLASS NAME 而不是 POSSIBLE VARIABLE NAME。仅供参考

最后，您的一些方法采用多个参数，如上面的那个。您确实应该为每个参数添加一个前缀，以便轻松理解该参数的用途。这个：

[vdb SelectAsyncRequest: PARAMETER : PARAMETER];

如果是这样的话会更好看：

[vdb selectAsyncRequestForParameter: PARAMETER withOtherParameter:OTHERPARAM];

编辑：我也不认为你需要这么多自动释放池。整个事情已经包裹在一个大的自动释放池中。

EDIT2：我还看到很多release不必要的电话。在您的UploadChunkWise方法中，您正在调用 release on*data并且*returnedData它们都已经隐式地 autoreleased。将对象返回给您的方法将已经放弃所有权并“移交”给您。本质上，这些方法将这样做：

NSData *data = [[NSData alloc] init];
return [data autorelease];

当你得到它时，如果你想保留它，你必须retain自己去处理它，否则它会在你的方法返回时被销毁。

但是，您可以在NSString *result使用 创建的实例上调用 release 是正确的-init。

于 2013-08-29T17:39:26.197 回答