xcode - UIScrollView のメモリ割り当てと解放

Question

私は、スクロールビューを持ついくつかのページを使用する、非常にシンプルで簡単なアプリに取り組んでいます。ViewDidLoad に画像の配列があり、次に現在のページのプラスとマイナス 1 を示す可変配列があります。スーパービューから余分な画像を削除することになっているパージページメソッドがあります。大量の画像がある実機でこれを実行すると、50 枚程度の画像をスクロールした後にメモリ警告が表示され、アプリがクラッシュします。これを計測器で実行したところ、スワイプするたびにメモリが大幅に増加していることがわかりました。配列を作成するときに、imageNamed と imageWithContentsOfFile を使用しましたが、どちらの方法でもほぼ同じ結果が得られます。ここには複数のスクロールビューの質問があることに気づきましたが、どういうわけかこれを乗り越えることができないようです. 控えめに言ってもイライラします。誰かがこれを新鮮な目で見て、私の問題に光を当ててくれることを願っています. よろしくお願いします。

編集: ARC の使用について言及するのを忘れていました。はい、ARC を使用しています。

コード: .h ファイル

#import <UIKit/UIKit.h>

@interface FirstViewController : UIViewController <UIScrollViewDelegate>

@property (nonatomic, strong) IBOutlet UIScrollView *scrollView;
@property (nonatomic, strong) IBOutlet UIPageControl *pageControl;

@end

および .m ファイル

#import "FirstViewController.h"

@interface FirstViewController ()

@property (nonatomic, strong) NSArray *pageImages;
@property (nonatomic, strong) NSMutableArray *pageViews;

- (void)loadVisiblePages;
- (void)loadPage:(NSInteger)page;
- (void)purgePage:(NSInteger)page;
@end

@implementation FirstViewController


@synthesize scrollView = _scrollView;
@synthesize pageControl = _pageControl;

@synthesize pageImages = _pageImages;
@synthesize pageViews = _pageViews;

- (id)initWithNibName:(NSString *)nibNameOrNil bundle:(NSBundle *)nibBundleOrNil
{
    self = [super initWithNibName:nibNameOrNil bundle:nibBundleOrNil];
    if (self) {
        // Custom initialization
    }
    return self;
}

- (void)didReceiveMemoryWarning
{
    // Releases the view if it doesn't have a superview.
    [super didReceiveMemoryWarning];

    // Release any cached data, images, etc that aren't in use.
}

- (void)loadVisiblePages {
    // First, determine which page is currently visible
    CGFloat pageWidth = self.scrollView.frame.size.width;
    NSInteger page = (NSInteger)floor((self.scrollView.contentOffset.x * 2.0f + pageWidth)     / (pageWidth * 2.0f));

// Update the page control
self.pageControl.currentPage = page;

// Work out which pages you want to load
NSInteger firstPage = page - 1;
NSInteger lastPage = page + 1;

// Purge anything before the first page
for (NSInteger i=0; i<firstPage; i++) {
    [self purgePage:i];
}

// Load pages in our range
for (NSInteger i=firstPage; i<=lastPage; i++) {
    [self loadPage:i];
}

// Purge anything after the last page
for (NSInteger i=lastPage+1; i<self.pageImages.count; i++) {
    [self purgePage:i];
}

}

- (void)purgePage:(NSInteger)page {
    if (page < 0 || page >= self.pageImages.count) {
    // If it's outside the range of what you have to display, then do nothing
    return;
    }

    // Remove a page from the scroll view and reset the container array
    UIView *pageView = [self.pageViews objectAtIndex:page];
    if ((NSNull*)pageView != [NSNull null]) {
        [pageView removeFromSuperview];
        [self.pageViews replaceObjectAtIndex:page withObject:[NSNull null]];


    }

}


- (void)loadPage:(NSInteger)page {
    if (page < 0 || page >= self.pageImages.count) {
        // If it's outside the range of what you have to display, then do nothing
        return;
    }

    // 1
    UIView *pageView = [self.pageViews objectAtIndex:page];
    if ((NSNull*)pageView == [NSNull null]) {
        // 2
        CGRect frame = self.scrollView.bounds;
        frame.origin.x = frame.size.width * page;
        frame.origin.y = 0.0f;

        // 3
        UIImageView *newPageView = [[UIImageView alloc] initWithImage:[self.pageImages objectAtIndex:page]];
        newPageView.contentMode = UIViewContentModeScaleAspectFit;
        newPageView.frame = frame;
        [self.scrollView addSubview:newPageView];
        // 4
       [self.pageViews replaceObjectAtIndex:page withObject:newPageView];

    }
}

- (void)scrollViewDidScroll:(UIScrollView *)scrollView {
    // Load the pages that are now on screen
    [self loadVisiblePages];
}

- (void)viewDidLoad {
    [super viewDidLoad];

    // 1
    self.pageImages = [NSArray arrayWithObjects:
                       [UIImage imageNamed:@"BBD1.jpg"],
                       [UIImage imageNamed:@"BBD2.jpg"],
                       [UIImage imageNamed:@"BBD3.jpg"],
                       [UIImage imageNamed:@"BBD4.jpg"],
                       [UIImage imageNamed:@"BBD5.jpg"],
                       [UIImage imageNamed:@"BBD6.jpg"],
                       [UIImage imageNamed:@"BBD7.jpg"],
                  .....
                       [UIImage imageNamed:@"BBD292.jpg"],
                       [UIImage imageNamed:@"BBD293.jpg"],
                       [UIImage imageNamed:@"BBD294.jpg"],
                       [UIImage imageNamed:@"BBD295.jpg"],
                       [UIImage imageNamed:@"BBD296.jpg"],
                       [UIImage imageNamed:@"BBD297.jpg"],
                       [UIImage imageNamed:@"BBD298.jpg"],
                       [UIImage imageNamed:@"BBD299.jpg"],
                       [UIImage imageNamed:@"BBD300.jpg"],                       


                       nil];

    NSInteger pageCount = self.pageImages.count;

    // 2
    self.pageControl.currentPage = 0;
    self.pageControl.numberOfPages = pageCount;

    // 3
    self.pageViews = [[NSMutableArray alloc] init];
    for (NSInteger i = 0; i < pageCount; ++i) {
        [self.pageViews addObject:[NSNull null]];
    } 
}

- (void)viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];

    // 4
    CGSize pagesScrollViewSize = self.scrollView.frame.size;
    self.scrollView.contentSize = CGSizeMake(pagesScrollViewSize.width *     self.pageImages.count, pagesScrollViewSize.height);

    // 5
    [self loadVisiblePages];
}

- (void)viewDidUnload
{
    [super viewDidUnload];
    // Release any retained subviews of the main view.
}

- (BOOL)shouldAutorotateToInterfaceOrientation:    (UIInterfaceOrientation)interfaceOrientation
{
    if ([[UIDevice currentDevice] userInterfaceIdiom] == UIUserInterfaceIdiomPhone) {
        return (interfaceOrientation != UIInterfaceOrientationPortraitUpsideDown);
    } else {
        return YES;
    }
}

@end

時間を割いてこれを見てくれたことに本当に感謝しています。誰かが私を正しい方向に向けることができることを願っています.

ありがとう！

score 3 · Accepted Answer

実際の画像を配列にロードすると、名前だけを配列に格納して次のように画像をロードするよりも、はるかに大きなオーバーヘッドが発生します。

UIImage *image = [[UIImage alloc] initWithContentsOfFile:[self.pageImages objectAtIndex:page]];
UIImageView *newPageView = [[UIImageView alloc] initWithImage:image];
[image release];

この時点で、newPageView のみがメモリ内の画像への参照を保持し、その newPageView がそのスーパービューから削除されると、メモリ内の画像が解放されます。

score 0 · Accepted Answer

ARC を使用しているかどうかはわかりませんが、ARC がないとメモリ管理に明らかな問題があります。loadPage:at コメント "// 3" では、保持カウント 1 で作成しnewPageView、それをサブビューself.scrollViewおよび配列に追加しself.pageViewsます。したがって、全体の保持カウントは +3 になります。andpurgePage:から削除しますが、全体の保持カウントは +1 のままです。したがって、各ページのは忘れられますが、割り当ては解除されません。self.scrollViewself.pageViewsUIImageView

newPageViewプロジェクトで ARC を使用するか、代わりに作成する行を次のように変更することで、これを修正できます。

UIImageView *newPageView = [[[UIImageView alloc] initWithImage:[self.pageImages objectAtIndex:page]] autorelease];

xcode - UIScrollView のメモリ割り当てと解放

2 に答える 2

Related

Reference