Perdita di memoria quando mantenendo proprietà

Question

Sto cercando di riprodurre un suono clic su ogni pulsante di scatto nella mia app Per questo ho creato una classe di utilità il cui .he .m è la seguente

file h

@interface SoundPlayUtil : NSObject<AVAudioPlayerDelegate,AVAudioSessionDelegate> 
{ 
    AVAudioPlayer *audioplayer; 
} 
@property (retain, nonatomic) AVAudioPlayer *audioplayer; 
-(id)initWithDefaultClickSoundName; 
-(void)playIfSoundisEnabled; 
@end 

di file .m

@implementation SoundPlayUtil 
@synthesize audioplayer; 

-(id)initWithDefaultClickSoundName 
{ 
self = [super init]; 
    if (self) 
{ 
    NSString* BS_path_blue=[[NSBundle mainBundle]pathForResource:@"click" ofType:@"mp3"]; 
    self.audioplayer =[[AVAudioPlayer alloc]initWithContentsOfURL:[NSURL fileURLWithPath:BS_path_blue] error:NULL]; 
    [self.audioplayer prepareToPlay]; 
} 
return self; 
} 

-(void)playIfSoundisEnabled 
{ 
if ([[NSUserDefaults standardUserDefaults] boolForKey:soundStatus]==YES) 
{ 
    [self.audioplayer play]; 
} 
} 

-(void)dealloc 
{ 
[audioplayer release]; 
[super dealloc]; 
} 
@end 

e sul pulsante di scatto su qualsiasi classe che sto facendo

SoundPlayUtil *obj = [[SoundPlayUtil alloc] initWithDefaultClickSoundName]; 
[obj playIfSoundisEnabled]; 
[obj release]; 

È funzionante e sono riuscito a riprodurre il suono. Il problema sorge quando ho analizzato il codice. Il compilatore mostra che c'è una perdita di memoria nel metodo initWithDefaultClickSoundName in .m della classe di utilità mentre invio il metodo di allocazione a self.audioplayer e non lo rilascia.

Qual è il posto migliore per liberare questo oggetto?

Answer 1

Il problema è quando si assegna l'oggetto che retainCount sarà 1, si assegna quell'oggetto a un oggetto proprietà di conservazione. Poi che sarà di nuovo a mantenere l'oggetto da cui il retainCount sarà 2.

Il codice setter di una proprietà mantenere è qualcosa di simile:

- (void)setAudioplayer: (id)newValue 
{ 
    if (audioplayer != newValue) 
    { 
     [audioplayer release]; 
     audioplayer = newValue; 
     [audioplayer retain]; 
    } 
} 

Modificare il:

self.audioplayer =[[AVAudioPlayer alloc]initWithContentsOfURL:[NSURL fileURLWithPath:BS_path_blue] error:NULL]; 

simili;

self.audioplayer =[[[AVAudioPlayer alloc]initWithContentsOfURL:[NSURL fileURLWithPath:BS_path_blue] error:NULL] autorelease]; 

o simile:

AVAudioPlayer *player = [[AVAudioPlayer alloc]initWithContentsOfURL:[NSURL fileURLWithPath:BS_path_blue] error:NULL]; 
self.audioplayer = player; 
[player release]; 

Answer 2

self.audioplayer =[[AVAudioPlayer alloc]initWithContentsOfURL:[NSURL fileURLWithPath:BS_path_blue] error:NULL]; 

Qui, si crea un nuovo oggetto, quindi assegnarlo a una proprietà mantenuto. Tuttavia, a parte la proprietà, non hai più alcun riferimento all'oggetto, quindi perde. Hai aumentato il conteggio di ritenzione due volte.

da risolvere, in ordine di preferenza:

1. Converti ad arco)

2. Creare una variabile locale, assegnarlo alla proprietà, quindi rilasciarlo.

   Object *object = [[Object alloc] init]; 
   self.property = object; 
   [object release]; 
   

3. aggiungere una chiamata autorelease per l'oggetto come si sta aggiungendo che: self.property = [[[Object alloc] init] autorelease];