不正な減少はにdeallocで私が得る、「分析」で

Question

をリーク：呼び出し側によって、この時点で所有されていないオブジェクトの参照カウントの 不正な減少を

#import <AVFoundation/AVFoundation.h> 
@interface XYZViewController : UIViewController 
@property (retain) AVAudioRecorder *recorder; 
@end 
@implementation XYZViewController 
@synthesize recorder; 
- (void) dealloc 
{ 
    [self.recorder release]; 
    [super dealloc]; 
} 
- (void) viewDidLoad 
{ 
    NSURL *url = [NSURL fileURLWithPath:@"/dev/null"]; 
    NSDictionary *settings = [NSDictionary dictionaryWithObjectsAndKeys: 
          [NSNumber numberWithFloat: 44100.0],     AVSampleRateKey, 
          [NSNumber numberWithInt: kAudioFormatAppleLossless], AVFormatIDKey, 
          [NSNumber numberWithInt: 1],       AVNumberOfChannelsKey, 
          [NSNumber numberWithInt: AVAudioQualityMax],   AVEncoderAudioQualityKey, 
          nil]; 
    NSError *error; 
    self.recorder = [[[AVAudioRecorder alloc] initWithURL:url settings:settings error:&error] autorelease]; 
} 
@end 

私はそれを放つべきではないという意味ですか？ また、私はコードを 'プロファイル'しようとしていて、何からでもメモリリークが[[[AVAudioRecorder alloc] initWithURL:url settings:settings error:&error] autorelease]から出ます。

Answer 1

ではなくプロパティのアクセッサメソッドによって返されるオブジェクトに-releaseを送る、nilにプロパティ自体を設定します。

- (void)dealloc { 
    self.recorder = nil; 
    [super dealloc]; 
} 

あなたは、ストレージのセマンティクスを指定したため、コンパイラは、正しいことを行うために知っているだろうプロパティ宣言。 retainセマンティクスで宣言されたプロパティを合成すると、次のアクセサメソッドを書くことを効果的に等価です：

- (AVAudioRecorder *)recorder { 
    return recorder; 
} 

- (void)setRecorder:(AVAudioRecorder *)newRecorder { 
    [newRecorder retain]; 
    [recorder release]; 
    recorder = newRecorder; 
} 

あなたはself.recorder = nil書くとき、コンパイラは[self setRecorder:nil]に変換します。したがって、このようにプロパティをnilに設定すると、メモリリークやポインタの不具合を回避し、定型化の回数を減らし、コードの意図をより明確に表現できます。

最後に、宣言されたプロパティに関するセクションを持つThe Objective-C Programming Languageを再読み込みすることは決してありません。 Advanced Memory Management Programming Guideがあります。これは、メモリ管理のさまざまなアプローチをすべて詳しく説明しています。

Answer 2

あなたはむしろアクセサを通過するよりも、直接IVARを解放する必要があります

- (void)dealloc 
{ 
    [recorder release]; 
    [super dealloc]; 
} 

あなたはそれを解放するべきではありませんので、あなたは、アクセサの返されたオブジェクトを所有していません。