Xcode的「構建和分析」報告此代碼奇怪的內存泄漏

Question

有誰知道爲什麼xCode的「構建和分析」會報告此行爲「可能的內存泄漏」？

goodSound = [[SoundClass alloc] initializeSound：@「Good.wav」];

/////這裏有4個文件中的問題：

// Sounds.h 

#import <Foundation/Foundation.h> 
#import <AudioToolbox/AudioToolbox.h> 

@interface SoundClass : NSObject 
{ 
    SystemSoundID soundHandle; 
} 

-(id) initializeSound:(NSString *)soundFileName; 
-(void) play;  

@end 

/////////////

// Sounds.m 

#import "Sounds.h" 

@implementation SoundClass 

-(id) initializeSound:(NSString *)soundFileName 
{ 
    self = [super init]; 

    NSString *const resourceDir = [[NSBundle mainBundle] resourcePath]; 
    NSString *const fullPath = [resourceDir stringByAppendingPathComponent:soundFileName]; 
    NSURL *const url   = [NSURL fileURLWithPath:fullPath]; 

    OSStatus errCode = AudioServicesCreateSystemSoundID((CFURLRef) url, &soundHandle); 

    if(errCode == 0) 
     NSLog(@"Loaded sound: %@", soundFileName); 
    else 
     NSLog(@"Failed to load sound: %@", soundFileName); 

    return self;    
} 

////////////////////////////// 

-(void) play  
{ 
    AudioServicesPlaySystemSound(soundHandle); 
} 

///////////////////////////// 

-(void) dealloc 
{ 
    AudioServicesDisposeSystemSoundID(soundHandle); 
    [super dealloc]; 
} 

///////////////////////////// 

@end 

//// //////////

// MemTestViewController.h 

#import <UIKit/UIKit.h> 

@class SoundClass; 

@interface MemTestViewController : UIViewController 
{ 
    SoundClass *goodSound; 
} 

-(IBAction) beepButtonClicked:(id)sender; 

@end 

///////////

// MemTestViewController.m 

#import "MemTestViewController.h" 
#import "Sounds.h" 

@implementation MemTestViewController 

- (void)viewDidLoad 
{ 
    NSLog(@"view did load: alloc'ing mem for sound class"); 

    // "build and analyze" says this is possibly a memory leak: 
    goodSound = [[SoundClass alloc] initializeSound:@"Good.wav"]; 

    [super viewDidLoad]; 
} 


-(IBAction) beepButtonClicked:(id)sender 
{ 
    NSLog(@"beep button clicked"); 

    [goodSound play]; 
} 


- (void)didReceiveMemoryWarning 
{ 
    [super didReceiveMemoryWarning]; 
} 


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

@end 

Answer 1

靜態分析器是正確的;你的代碼是可能的內存泄漏。因爲靜態分析不能證明MemTestViewController的方法總是被調用（這樣做會解決暫停問題，畢竟），靜態分析器無法確定goodSound將被正確釋放。

現代Objective-C很少使用直接ivar訪問接受init和dealloc方法。你應該爲你的goodSound申報@property (retain)，即使它是私密的，並在你的課程中的其他地方使用，包括viewDidLoad方法。然後，靜態分析儀應能正確識別此模式，而不會將該線標記爲警告。

在一個側面說明，以防萬一你有興趣，你-[SoundClass initializeSound:]名稱應該像-[SoundClass initWithSoundFileName:]匹配約定，並應保護被寫入測試從[super init]一個nil回報：

-(id)initWithSoundFileName:(NSString*)soundFileName 
{ 
    self = [super init]; 

    if(self != nil) { 
     NSString *const resourceDir = [[NSBundle mainBundle] resourcePath]; 
     NSString *const fullPath = [resourceDir stringByAppendingPathComponent:soundFileName]; 
     NSURL *const url   = [NSURL fileURLWithPath:fullPath]; 

     OSStatus errCode = AudioServicesCreateSystemSoundID((CFURLRef) url, &soundHandle); 

     if(errCode == 0) 
     NSLog(@"Loaded sound: %@", soundFileName); 
     else 
     NSLog(@"Failed to load sound: %@", soundFileName); 
    } 

    return self;    
} 

總是最好添加這個警衛，尤其是因爲你正在調用C代碼，可能無法像Objective-C一樣處理nil/NULL。

Answer 2

這是一個可能的內存泄漏。如果視圖被卸載，則可以調用任何次數的viewDidLoad，在這種情況下，每次都會泄漏內存。爲了防範，您可以釋放viewDidUnload中的內存並將伊娃設置爲零，或者只有當伊娃實際爲零時，纔可以簡單地初始化viewDidLoad中的聲音。

所以：

- (void)viewDidLoad 
{ 
    if(!goodSound) 
    goodSound = [[SoundClass alloc] initializeSound:@"Good.wav"]; 
    [super viewDidLoad]; 
} 

和/或：

- (void)viewDidUnload 
{ 
    [goodSound release]; 
    goodSound = nil; 
    [super viewDidUnload]; 
}