引用计数和内存泄漏的错误减少

Question

使用“分析”，在 dealloc 中我得到： 调用者此时不拥有的对象的引用计数减少不正确

#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

您应该直接释放 ivar，而不是通过访问器：

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

你不拥有访问器返回的对象，所以你不应该释放它。

Answer 2

不要将-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，它详细介绍了所有不同的内存管理方法。