1

写真ライブラリから写真をインポートするアプリに取り組んでいます。写真のインポート中に大量のメモリを使用し、160 枚の写真をインポートした後にクラッシュします。

なぜこれが起こっているのかわかりません。どんな助けでも大歓迎です。

これが私のコードです

- (void)agImagePickerController:(AGImagePickerController *)picker    didFinishPickingMediaWithInfo:(NSArray *)info{

[pickerView dismissViewControllerAnimated:YES completion:nil];
[self.navigationItem setHidesBackButton:YES];
[self.navigationItem.rightBarButtonItem setEnabled:NO];

MBProgressHUD * hud =  [MBProgressHUD showHUDAddedTo:self.view animated:YES];
hud.labelText=@"Importing Your Photo(s)";



// USING GRAND CENTRAL DISPATCH

firstQueue=dispatch_queue_create("com.import.photos", nil);
dispatch_async(firstQueue, ^{

    [self beingBackgroundUpdateTask];

    // Open connection
    //[albumObject openDB];


    // Start the loop from here ******************************************************
    int j=0;
    for(j=0;j<info.count;j++){

        // update the hud label
        hud.labelText=[[NSString alloc] initWithFormat:@"Importing %i of %i",j+1, [info  count]];


        // Create new image from returned array
        ALAssetRepresentation *rep = [[info objectAtIndex: j] defaultRepresentation];
        UIImage * image  = [UIImage imageWithCGImage:[rep fullScreenImage]];
        //rep=nil;

        // resize the image for the thumbnail and save it in new image
        UIImage *newImage=[image  resizedImageWithContentMode:UIViewContentModeScaleAspectFill bounds:CGSizeMake(75, 75)  interpolationQuality:kCGInterpolationDefault];


        NSDate *date = [NSDate date];

        NSDateFormatter *dateFormat = [[NSDateFormatter alloc]init];
        [dateFormat setDateFormat:@"DD_MM_YY_HH_mm_ss"];
        NSString *dateString = [dateFormat stringFromDate:date];

        int random=rand();
        // create new file names
        NSString *fileName=[[NSString alloc] initWithFormat:@"%d%@",random,dateString];
        NSString *s_fileName=[[NSString alloc]  initWithFormat:@"small_%d%@",random,dateString];
        // where to save this image
        NSString *imagePath=[self.documentsDirectory  stringByAppendingPathComponent:fileName];
        NSString *s_imagePath=[self.documentsDirectory stringByAppendingPathComponent:s_fileName];


        NSLog(@"type: %@",[rep.url pathExtension]);

        if ([[rep.url pathExtension] isEqualToString:@"JPG"]) {
            // its jpg
            NSData *rawImage=UIImageJPEGRepresentation(image, 1.0);
            NSData *thumb=UIImageJPEGRepresentation(newImage, 1.0);


            image=nil;
            newImage=nil;
            date=nil;
            dateFormat=nil;
            dateString=nil;
            random=nil;

            // Create the images
            [rawImage writeToFile:imagePath atomically:YES];
            [thumb writeToFile:s_imagePath atomically:YES];

            imagePath=nil;
            s_imagePath=nil;
            rawImage=nil;
            thumb=nil;

        }

        if ([[rep.url pathExtension] isEqualToString:@"PNG"]) {
            // its PNG
            // save it in NSData to it can be saved in the directory
            NSData *rawImage=UIImagePNGRepresentation(image);
            NSData *thumb=UIImagePNGRepresentation(newImage);




            image=nil;
            newImage=nil;
            date=nil;
            dateFormat=nil;
            dateString=nil;
            random=nil;

            // Create the images
            [rawImage writeToFile:imagePath atomically:YES];
            [thumb writeToFile:s_imagePath atomically:YES];

            imagePath=nil;
            s_imagePath=nil;
            rawImage=nil;
            thumb=nil;

        }





        // Now save it to database


        [albumObject savePhoto:fileName toFID:PassedID toName:s_fileName  albumName:singletonObj.currentAlbum photosName:singletonObj.currentPhotos];


        [display_photos addObject:[MWPhoto photoWithFilePath:[self.documentsDirectory  stringByAppendingPathComponent:fileName]]];

        UIImage *img=[UIImage imageWithContentsOfFile:[self.documentsDirectory  stringByAppendingPathComponent:s_fileName]];

        [collection_photos addObject:img];

        img=nil;

        NSArray *pObj=[[NSArray alloc] initWithObjects:fileName,s_fileName, nil];
        [photos addObject:pObj];

        pObj=nil;
        fileName=nil;

        s_fileName=nil;


    } j=nil;

    // Update the album photo count to reflect the changes
    [albumObject updateAlbumCount:PassedID photocount:[photos count]  albumName:singletonObj.currentAlbum];
    photos=[albumObject getPhotos:PassedID photosName:singletonObj.currentPhotos];

    // End the loop here ***************************************************************

    dispatch_async(dispatch_get_main_queue(), ^{
    [self.collectionView reloadData];
    [MBProgressHUD hideHUDForView:self.view animated:YES];
    [self.navigationItem setHidesBackButton:NO];
    [self.navigationItem.rightBarButtonItem setEnabled:YES];

    });

    });



}
4

1 に答える 1

3

ループでは、多くの余分なオブジェクトが生成されており、ループ全体が終了するまで解放されません。ただし、メモリが不足するまで余分なオブジェクトが積み重なるため、決して終了しません。防御の第一線は、ループの内部を@autoreleasepoolブロックでラップすることです。

例として、私の本のこのセクションを見てください: http://www.apeth.com/iOSBook/ch12.html#_autoreleaseそのセクションの一番下までスクロールします。

for /* loop iterator goes here */ {
    @autoreleasepool {
        // everything inside the for loop goes here
    }
}

編集:別の提案があります:画像の配列などで終わっていますか? よくわかりません。その場合、これらすべての最終的な結果としてイメージをメモリに保存する代わりに、イメージをディスクに保持し、代わりにその URL またはファイル パスを保存します。そうすれば、必要なときにのみ画像を取得できます。画像はメモリ内で多くのスペースを必要とし、サイズが大きくなると指数関数的に増加します。

于 2013-04-27T17:01:21.803 に答える