ios - 内存泄漏 - UIImagePNGRepresentation

Question

我正在尝试将图像从 UIImagePicker 复制到文档目录。我正在使用 @"UIImagePickerControllerOriginalImage" 键从 UIImagePickerDelegate 的字典中获取原始图像。我正在使用 UIImagePNGRepresentation 将图像写入文件。当我添加(重复该过程)高分辨率图像(图像大小约为 20 mb)时，我遇到了内存问题。

我分析并使用了 Xcode 的内存泄漏功能，它放大了以下代码，这是造成泄漏的原因。

@autoreleasepool {
    imagesData = UIImagePNGRepresentation(images);
    [imagesData writeToFile:name atomically:NO];
    imagesData = nil;
    //[UIImageJPEGRepresentation(images, 1.0) writeToFile:name atomically:YES];
}

我在这里看到了很多关于 UIImagePNGRepresentation 引起的内存泄漏的问题。但是我还没有找到解决问题的合适方法。需要帮助。

Answer 1

我不知道 UIImagePNGRepresentation 有任何“泄漏”，但这肯定是对内存的过度使用，但这里有几个问题:

1. 首先，通过 UIImage 往返原始 Assets 然后使用 UIImagePNGRepresentation() 的过程效率相当低，最终可能会出现 NSData 比原始 Assets 大得多。例如，我选择了一张原始 Assets 为 1.5mb 的照片，UIImageJPEGRepresentation(compressionQuality 为 1.0)为 6mb，而 UIImagePNGRepresentation()大约10mb。 (这些数字在不同图像之间可能会有很大差异，但您已经了解了基本概念。)

   您通常可以通过使用 UIImageJPEGRepresentation 和 compressionQuality 小于 1.0(例如 0.8 或 0.9 提供最小的图像质量损失，但在NSData 站点)。但这是有损压缩。此外，您会在此过程中丢失一些图像元数据。

2. 我相信您同时在内存中保存了同一图像的多个副本:您同时拥有 UIImage 表示和 NSData 对象.

3. 不仅 Assets 的 NSData 表示大于它需要的大小，而且您还将整个 Assets 一次性加载到内存中。这不是必需的。

您可能会考虑将原始 Assets 从 ALAssetLibrary 直接流式传输到持久内存，而不使用 UIImagePNGRepresentation 或 UIImageJPEGRepresentation，并且不使用将它加载到 UIImage 中。相反，创建一个小缓冲区，通过 getBytes 用原始 Assets 的部分重复填充此缓冲区，同时使用 NSOutputStream 将此小缓冲区写入临时文件。您可以重复该过程，直到将整个 Assets 写入持久存储。此过程的总内存占用量远低于替代方法。

例如:

static NSInteger kBufferSize = 1024 * 10;

- (void)imagePickerController:(UIImagePickerController *)picker didFinishPickingMediaWithInfo:(NSDictionary *)info
{
    NSURL *url = info[UIImagePickerControllerReferenceURL];

    [self.library assetForURL:url resultBlock:^(ALAsset *asset) {
        ALAssetRepresentation *representation = [asset defaultRepresentation];
        long long remaining = representation.size;
        NSString *filename  = representation.filename;

        NSString *documentsPath = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES)[0];
        NSString *path = [documentsPath stringByAppendingPathComponent:filename];
        NSString *tempPath = [self pathForTemporaryFileWithPrefix:@"ALAssetDownload"];

        NSOutputStream *outputStream = [NSOutputStream outputStreamToFileAtPath:tempPath append:NO];
        NSAssert(outputStream, @"Unable to create output stream");

        [outputStream open];
        
        long long representationOffset = 0ll;
        NSError *error;

        uint8_t buffer[kBufferSize];

        while (remaining > 0ll) {
            NSInteger bytesRetrieved = [representation getBytes:buffer fromOffset:representationOffset length:sizeof(buffer) error:&error];
            if (bytesRetrieved < 0) {
                NSLog(@"failed getBytes: %@", error);
                [outputStream close];
                [[NSFileManager defaultManager] removeItemAtPath:tempPath error:nil];
                return;
            } else {
                remaining -= bytesRetrieved;
                representationOffset += bytesRetrieved;
                [outputStream write:buffer maxLength:bytesRetrieved];
            }
        }

        [outputStream close];
        
        if (![[NSFileManager defaultManager] moveItemAtPath:tempPath toPath:path error:&error]) {
            NSLog(@"Unable to move file: %@", error);
        }

    } failureBlock:^(NSError *error) {
        NSLog(@"assetForURL error = %@", error);
    }];
}

- (NSString *)pathForTemporaryFileWithPrefix:(NSString *)prefix
{
    NSString    *uuidString = [[NSUUID UUID] UUIDString];

    // If supporting iOS versions prior to 6.0, you can use:
    //
    // CFUUIDRef uuid = CFUUIDCreate(NULL);
    // assert(uuid != NULL);
    // NSString *uuidString = CFBridgingRelease(CFUUIDCreateString(NULL, uuid));
    // CFRelease(uuid);
    
    return [NSTemporaryDirectory() stringByAppendingPathComponent:[NSString stringWithFormat:@"%@-%@", prefix, uuidString]];
}