0

私は iOS ARC アプリ (リクエストに応じてより多くのコードを利用できます) に取り組んでおり、以前は大量の画像を生成して破棄していました。removeFromSuperview を呼び出して、使用されなくなった画像へのすべての参照を削除しようとしても、どこかにまだ画像への参照があると思いました。私は Leaks を試しましたが、Leaks は 17M 前後から始まり、時間の経過とともにメモリ使用量がほぼ直線的に増加したことを報告しました。

画像へのすべての参照をインスタンス変数に置き換えました。これにより、それらは小さくて有限の固定量のメモリを使用し、時計の針に使用される画像を削除する代わりに変換しました。残念ながら、これにより、時間の経過とともにメモリ使用量が 17M ではなく 5M から徐々に増加する結果となりましたが、それ以外は同じ問題であり、単により良い開始点に変換されました。

私のコードのトリミングされたバージョンは以下のとおりです。これについて何が漏れているのか (または、Leaks がリークを示さなかったので「疑似漏れ」)、コードが起動時に使用するメモリ境界に近づく方法を教えてください。

ありがとう、

- (void) renderScreen
{
int height = floor([[UIScreen mainScreen] bounds].size.height + .4);
int width = floor([[UIScreen mainScreen] bounds].size.width + .4);

if (height == 2048 || height == 2008 || height == 1024 || height == 1004 || height == 984)
{
    if (UIInterfaceOrientationIsPortrait([UIApplication sharedApplication].statusBarOrientation))
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default-Portrait.png"];
    }
    else
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default-Landscape.png"];
    }
}
else if (height == 1536 || height == 768 || height == 748 || height == 728)
{
    _backgroundImage = [UIImage imageNamed:@"Background-Default-Landscape.png"];
}
else if (height == 1136 || height == 1116 || height == 1096)
{
    if (UIInterfaceOrientationIsPortrait([UIApplication sharedApplication].statusBarOrientation))
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default-568.png"];
    }
    else
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default-Rotated-568.png"];
    }
}
else if (height == 960 || height == 940 || height == 920 || height == 480 || height == 460 || height == 440)
{
    if (UIInterfaceOrientationIsPortrait([UIApplication sharedApplication].statusBarOrientation))
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default.png"];
    }
    else
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default-Rotated.png"];
    }
}
else if ((height == 640 || height == 620 || height == 600) && (width == 1136 || width == 1116 || width == 1096))
{
    _backgroundImage = [UIImage imageNamed:@"Background-Rotated-568.png"];
}
else if ((height == 640 || height == 620 || height == 600 || height == 320 || height == 300 || height == 280) && (width == 960 || width == 940 || width == 920 || width == 480 || width == 470 || width == 410))
{
    if (UIInterfaceOrientationIsPortrait([UIApplication sharedApplication].statusBarOrientation))
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default-Portrait.png"];
    }
    else
    {
        _backgroundImage = [UIImage imageNamed:@"Background-Default-Rotated.png"];
    }
}
else
{
    _backgroundImage = [UIImage imageNamed:@"Background-Default-Portrait.png"];

}
if (!UIInterfaceOrientationIsPortrait([UIApplication sharedApplication].statusBarOrientation))
{
    int juggle = height;
    height = width;
    width = juggle;
}
NSUInteger centerX = width * .5;
NSUInteger centerY = height * .5;

_containerRect = CGRectZero;
_containerRect.size = [[UIScreen mainScreen] bounds].size;

self.view.backgroundColor = [UIColor colorWithPatternImage:_backgroundImage];
_backgroundView = [[UIImageView alloc] initWithImage:_backgroundImage];
[self.view addSubview:_backgroundView];
if (_changed)
{
    _containerView = [[UIView alloc] initWithFrame:_containerRect];
}

double timeStampSeconds = [[NSDate date] timeIntervalSince1970];
double hours   = fmod(timeStampSeconds / 86400, 24);
double minutes = fmod(timeStampSeconds /  3600, 60);
double seconds = fmod(timeStampSeconds,     60);
NSLog(@"Milliseconds: %lf, Hours: %.0f, minutes: %.0f, seconds: %.0f", timeStampSeconds * 1000.0, hours, minutes, seconds);
[_containerView removeFromSuperview];
_containerView = [[UIView alloc] initWithFrame:_containerRect];
_hourHandImage = [UIImage imageNamed:@"hour-hand.png"];
_hourHandView = [[UIImageView alloc] initWithImage:_hourHandImage];
_hourHandImage = [UIImage imageNamed:@"hour-hand.png"];
_hourHandView = [[UIImageView alloc] initWithImage:_hourHandImage];
[self.view addSubview:_hourHandView];

_hourHandView.layer.anchorPoint = CGPointMake(0.5f, 0.5f);
_hourTransform = CGAffineTransformMakeTranslation(centerX, centerY);
_hourTransform = CGAffineTransformTranslate(_hourTransform, -17, -127);
_hourTransform = CGAffineTransformRotate(_hourTransform, hours / 12.0 * M_PI * 2.0);
_minuteTransform = CGAffineTransformMakeTranslation(centerX, centerY);
_minuteTransform = CGAffineTransformTranslate(_minuteTransform, -10, -182);
_minuteTransform = CGAffineTransformRotate(_minuteTransform, minutes / 60.0 * M_PI * 2.0);
_hourHandView.transform = _hourTransform;
_minuteHandImage = [UIImage imageNamed:@"minute-hand.png"];
_minuteHandView = [[UIImageView alloc] initWithImage:_minuteHandImage];
_minuteHandView.transform = _minuteTransform;
[self.view addSubview:_minuteHandView];
_minuteTransform = CGAffineTransformRotate(_minuteTransform, minutes / 60.0 * M_PI * 2.0);
_secondHandImage = [UIImage imageNamed:@"second-hand.png"];
_secondTransform = CGAffineTransformMakeTranslation(centerX, centerY);
_secondTransform = CGAffineTransformTranslate(_secondTransform, -10, -189);
_secondTransform = CGAffineTransformRotate(_secondTransform, seconds / 60.0 * M_PI * 2.0);
_secondHandView = [[UIImageView alloc] initWithImage:_secondHandImage];
_secondHandView.transform = _secondTransform;
[self.view addSubview:_secondHandView];
}

- 編集 -

1 つのメソッドを 2 つのメソッドにリファクタリングしました。1 つは初期表示用、もう 1 つは増分更新用です。クロックが凍結され、シグネチャ ログ ステートメントが 1 回だけ呼び出されるため、増分更新メソッドは 1 回だけ呼び出されるようです。

更新部分については、次のようになりました。

- (void)render
{
    [self renderScreenInitial];
    [NSTimer scheduledTimerWithTimeInterval:0.002
                                     target:self
                                   selector:@selector(renderingTimer:)
                                   userInfo:nil
                                    repeats:YES];
}

- (void) renderScreenIncremental
{
    int height = floor([[UIScreen mainScreen] bounds].size.height + .4);
    int width = floor([[UIScreen mainScreen] bounds].size.width + .4);
    if (!UIInterfaceOrientationIsPortrait([UIApplication sharedApplication].statusBarOrientation))
    {
        int juggle = height;
        height = width;
        width = juggle;
    }
    double decibelAngle = M_PI / 4 + (_decibel / 60) * M_PI / 2;
    double decibelAngleDifference = decibelAngle - _previousDecibelAngle;
    _previousDecibelAngle = decibelAngle;
    NSLog(@"%lf %lf", _decibel, decibelAngle);
    _decibelNeedleView = [[UIImageView alloc] initWithImage:_decibelNeedle];
    // CGAffineTransform decibelTransform = CGAffineTransformMakeTranslation(centerX, centerY);
    CGAffineTransform decibelTransform = CGAffineTransformMakeRotation(decibelAngleDifference);
    decibelTransform = CGAffineTransformTranslate(decibelTransform, sin(decibelAngle - .06) * -298, -cos(decibelAngle - .06) * 298);
    _decibelNeedleView.transform = decibelTransform;
    [self.view addSubview:_decibelNeedleView];
    double timestampSeconds = [[NSDate date] timeIntervalSince1970];
    double timestampSecondsDifference = timestampSeconds - _previousTimestampSeconds;
    _previousTimestampSeconds = timestampSeconds;
    double hoursDifference   = fmod(timestampSecondsDifference / 86400, 24);
    double minutesDifference = fmod(timestampSecondsDifference /  3600, 60);
    double secondsDifference = fmod(timestampSecondsDifference,     60);
    NSLog(@"Milliseconds: %lf, Hours: %.0f, minutes: %.0f, seconds: %.0f", timestampSecondsDifference * 1000.0, hoursDifference, minutesDifference, secondsDifference);
    _hourHandView.transform = CGAffineTransformMakeRotation(hoursDifference);
    _minuteHandView.transform = CGAffineTransformMakeRotation(minutesDifference);
    _secondHandView.transform = CGAffineTransformMakeRotation(secondsDifference);
}

-(void)renderingTimer:(NSTimer *)timer {
    [self renderScreenIncremental];
}

助けてくれてありがとう。ディスプレイを一度更新した後、更新を継続しない理由がわかりますか?

ありがとう、

4

2 に答える 2

2

何よりもまず、Instruments の Allocations ツールを使用して、割り当てられているものとリリースされていないものを特定する必要があります。WWDC 2012 ビデオiOS アプリのパフォーマンス: メモリが必要なことをお勧めします。これは、とりわけこれを示しています。

Instrument の Allocations ツールを使用すると、割り当てられていて解放されていないものを推測するのではなく、ヒープショット/世代を使用して、ピッキングのある時間範囲内で割り当てられていて解放されていない実際のオブジェクトを特定できます。ドリルインして、問題のあるオブジェクトが最初に割り当てられた場所を確認することもできます。

それはさておき、コードをちらりと見ると、コンテナを削除して再度追加しているように見えますが、コンテナには何も入っていません。そして、時/分/秒針を追加していますが、それらを削除することはありません. おそらく、それらをコンテナーに追加するつもりでしたか?

また、あなたのコードは何をするかを説明していません_changedが、それが である場合、YES効果的に を作成し_containerView、そのスーパービューからそれを削除し (スーパービューに追加されたことはありませんが)、それを破棄して別の を作成してい_containerViewます。非常に奇妙な。

また、このコードから、あなたがrenderScreen何度も呼び出すことになると推測しています。それが本当なら、ビューの「作成」(背景、さまざまな画像などの追加)をビューの「変更」(transform値の調整)から分離することをお勧めします。可能であれば、ビューを一度追加してから、更新時に最小限の作業 (変換の変更) を行う必要があります。

于 2013-10-13T18:44:25.067 に答える