9

我正在编写一个应用程序,用户在其中为自己拍照,然后通过一系列视图使用导航控制器调整图像。如果用户使用前置摄像头拍照(在支持它的设备上设置为默认设置),这非常有效,但是当我重复该过程时,我完成了大约一半,并在抛出内存警告后崩溃。

在 Instruments 中进行分析后,我发现我的应用程序内存占用在使用较低分辨率的前置摄像头图像时保持在 20-25 MB 左右,但是当使用后置摄像头时,每次视图更改都会再增加 33 MB 左右,直到它在大约 350 MB 时崩溃( 4S)

下面是我用来处理将照片保存到文档目录的代码,然后读取该文件位置以将图像设置为UIImageView. 此代码的“读取”部分通过多个视图控制器 (viewDidLoad) 循环使用,以便将我保存的图像设置为每个视图中的背景图像。

我已经删除了我所有的图像修改代码,以将其减少到试图隔离问题的最低限度,但我似乎找不到它。就目前而言,应用程序所做的只是在第一个视图中拍摄一张照片,然后将该照片用作大约 10 个视图的背景图像,并在用户浏览视图堆栈时进行分配。

现在显然更高分辨率的照片会使用更多的内存,但我不明白的是为什么低分辨率的照片似乎没有使用越来越多的内存,而高分辨率的照片却不断地使用越来越多的内存直到崩溃。

我如何保存和阅读图像:

- (void) imagePickerController:(UIImagePickerController *)picker didFinishPickingMediaWithInfo:(NSDictionary *)info
{
    UIImage *image = [info objectForKey:@"UIImagePickerControllerOriginalImage"];    
    jpgData = UIImageJPEGRepresentation(image, 1);

    NSArray *paths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES);  
    NSString *documentsPath = [paths objectAtIndex:0];
    filePath = [documentsPath stringByAppendingPathComponent:@"image.jpeg"];
    [jpgData writeToFile:filePath atomically:YES];    

    [self dismissModalViewControllerAnimated:YES];
    [disableNextButton setEnabled:YES];

    jpgData = [NSData dataWithContentsOfFile:filePath];
    UIImage *image2 = [UIImage imageWithData:jpgData];
    [imageView setImage:image2];
}

现在我知道我可以在保存图像之前尝试缩放图像,我计划接下来研究它,但我不明白为什么这不能按原样工作。也许我错误地认为 ARC 在离开堆栈顶部时会自动释放视图及其子视图。

任何人都可以解释为什么我要堆积我的设备内存吗?(希望我完全忽略了一些简单的事情)我是否设法将ARC扔出窗外?

编辑:我如何在其他视图中调用图像

- (void)loadBackground
{
    NSArray *paths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES);  
    NSString *documentsPath = [paths objectAtIndex:0];
    NSString *filePath = [documentsPath stringByAppendingPathComponent:@"image.jpeg"];
    UIImage *image = [UIImage imageWithContentsOfFile:filePath];
    [backgroundImageView setImage:image];

}

我的视图控制器之间的导航是如何建立的:

在此处输入图像描述

编辑2:

我的基本声明是什么样的:

#import <UIKit/UIKit.h>
#import <AVFoundation/AVFoundation.h>
@interface PhotoPickerViewController : UIViewController <UIImagePickerControllerDelegate, UINavigationControllerDelegate>
{
    IBOutlet UIImageView *imageView;
    NSData *jpgData;
    NSString *filePath;
    UIImagePickerController *imagePicker;
    IBOutlet UIBarButtonItem *disableNextButton;
}


@end

如果相关,我如何调用我的图像选择器:

- (void)callCameraPicker
{


    if ([UIImagePickerController isSourceTypeAvailable:UIImagePickerControllerSourceTypeCamera] == YES)
    {
        NSLog(@"Camera is available and ready");

        imagePicker.sourceType =  UIImagePickerControllerSourceTypeCamera;
        imagePicker.delegate = self;
        imagePicker.allowsEditing = NO;
        imagePicker.cameraCaptureMode = UIImagePickerControllerCameraCaptureModePhoto;

        NSArray *devices = [AVCaptureDevice devicesWithMediaType:AVMediaTypeVideo]; for (AVCaptureDevice *device in devices) 
        {
            if([[UIScreen mainScreen] respondsToSelector:@selector(scale)] && [[UIScreen mainScreen] scale] == 2.0) 
            {
                imagePicker.cameraDevice = UIImagePickerControllerCameraDeviceFront;


            }
        }


        imagePicker.modalTransitionStyle = UIModalTransitionStyleCoverVertical;

        [self presentModalViewController:imagePicker animated:YES];

    }


    else
    {
        NSLog(@"Camera is not available");
        UIAlertView *cameraAlert = [[UIAlertView alloc] initWithTitle:@"Error" 
                                                              message:@"Your device doesn't seem to have a camera!" 
                                                             delegate:self cancelButtonTitle:@"Dismiss" 
                                                    otherButtonTitles:nil];
        [cameraAlert show];

    }
}

编辑 3:我记录了viewDidUnload ,实际上它没有被调用,所以我现在正在调用并loadBackground在. 我希望这会有所帮助,但没有任何区别。viewWillAppearbackgroundImageViewviewDidDisappear

- (void)viewWillAppear:(BOOL)animated
{
    [self loadBackground];
}


- (void)viewDidDisappear:(BOOL)animated
{
    NSLog(@"ViewDidDisappear");
    backgroundImageView = nil;
}
4

3 回答 3

6

UIImage和之间的关系UIImageView不一定对每个人都直观。

UIImage是图像数据的高级表示 - 就显示数据而言,它没有任何作用。

UIImageView用于UIImage允许您显示图像。

没有理由为什么 的多个实例UIImageView不能显示相同的UIImage. 这是很好且高效的,因为图像数据只有一个内存表示,被多个视图共享。

您似乎正在做的是UIImage通过从磁盘加载它来为您的每个视图创建一个新视图。因此,在两个方面,这是一个糟糕的通用设计:一遍又一遍地实例化实际上相同的内容UIImage,以及反复从磁盘重新加载相同的图像数据。

您的内存问题实际上是一个单独的问题,您没有正确释放您一直加载到UIImage对象和UIImageViews.

从理论上讲,您应该能够获取第一个UIImage您从中获取的信息,UIImagePickerController然后将该引用简单地传递给您的视图,而无需从磁盘重新加载。

如果您因为更高级别的功能要求而需要从磁盘保存和重新加载(例如,因为图像正在被用户更改并且您想继续保存它),您需要确保您完全拆除了以前的UIView, 通过从它的视图层次结构中删除它。在视图的 dealloc 方法中设置断点以确认它正在被删除和解除分配是很有帮助的,并确保将任何iVar对子视图的引用(看起来你backgroundImageView是 iVar)设置为 nil。如果您没有正确地拆除它backgroundImageView,它会继续持有对UIImage您设置为它的 image 属性的引用。

于 2012-05-14T18:17:48.903 回答
1

有几件事对您发布的代码感到好奇:

  1. 您的视图回调实现都没有调用super. 那很糟!额外确保你调用了 super in viewDidUnloadand (如果你实现了它) didReceiveMemoryWarning
  2. 确保didReceiveMemoryWarning以有意义的方式实施!
  3. 真的不应该一遍又一遍地重新创建该图像!我假设您没有编辑实际图像,因为您在其上使用了 JPEG 压缩——即使是 100% 质量——每次保存都会使您的图像质量下降……</li>
  4. 检查您的实现,viewDidUnload确保将您的每个 IBOutlets 设置为nil.

ARC 不是 Pixie Dust™!它只是为您节省了一些打字时间,它并没有使您免于设计和维护您的对象图!

从您的问题中,我至少看到了这些引用您的图像的图表:

image 1 <- image-view 1 <- view-controller 1 <- navigation-controller <- key window <- application

image 1 <- image-view 1 <- view 1 <- view-controller 1 <- navigation-controller <- key window <- application

对于每个视图控制器重复此操作,并在视图控制器、视图、图像视图和图像上进行索引移位。虽然您必须为视图控制器提供单独的视图和图像视图,但我想不出为什么您需要同一图像的多个副本。

因此,内存消耗的第一把斧头显然是不再创建相同图像数据的所有副本——我估计这将为您节省一半的内存。

接下来的事情是,如果不再引用,ARC 只能释放对象消耗的内存。

内存方面,视图并不完全是轻量级对象,当您构建一个深度导航堆栈时,您最终会得到大量的视图。

因此,您还需要取消对这些视图的任何不需要的强引用。

必须发生这种情况的级别是视图控制器。发生这种情况的最晚时间是在视图控制器的.viewDidUnload

为什么是视图控制器?

根据您的描述,图像本身仅由 UIImageView 引用 - 这是一个糟糕的选择,恕我直言,但我离题了……<br> UIViewController旨在“知道”何时需要它的视图以及何时可以安全地处理它— 这就是它实现didReceiveMemoryWarningand的原因viewDidUnload

如果内存压力很高并且视图控制器的视图不在“屏幕上”,那么根实现didReceiveMemoryWorning将放开它的视图并viewDidUnload在之后调用它自己。

这就是为什么您必须在这两种方法super的实现中调用 to 。

此外,这就是为什么如果你有强大的 IBOutlets 引用视图控制器视图的子视图,你必须将它们 nil ,viewDidUnload否则系统无法回收它们占用的内存。

它的核心UIViewController是一个庞大的有限状态机。所有这些“something-will/did-whatever”回调都用于在这些状态之间进行转换,并且大多数默认实现都会做一些非常重要的簿记以保持所有状态井井有条。

如果你没有在你的覆盖中调用它们,你最终会处于不一致的状态和坏事——比如这种内存不足的崩溃——发生。

于 2012-05-19T07:44:19.113 回答
0

只需创建单独的文件夹并将 Capture 图像保存在其中。成功操作后,使用 nsfilemager 清除文件夹数据(或)文件夹。

于 2012-05-14T14:34:36.730 回答