Memory Leak kiedy zachowując własność

Question

Próbuję odtworzyć dźwięk kliknij na każdego kliknięcia przycisku w mojej aplikacji za to stworzył klasę Utility którego .h i .m jest następująca

.h plik

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

pliku .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 

i kliknięcia przycisku na dowolnej klasy robię

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

Działa poprawnie i udało mi się odtworzyć dźwięk. Problem powstał, gdy analizowałem kod. Kompilator pokazuje, że występuje przeciek pamięci w metodzie initWithDefaultClickSoundName w .m klasy narzędziowej, ponieważ wysyłam metodę alokacji do self.audioplayer i nie zwalniam jej.

Jakie jest najlepsze miejsce do uwolnienia tego obiektu?

Answer 1

Problem polega na tym, że przy przydzielaniu obiektu, który ma wartość retainCount, będzie 1, przypisujesz ten obiekt do obiektu właściwości retain. Potem ponownie zachować obiekt stąd retainCount będzie 2.

Kod setter właściwości zatrzymywania jest coś takiego:

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

Zmienianie:

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

podobne;

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

lub jak:

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]; 

Tutaj utworzyć nowy obiekt, a następnie przypisać je do zatrzymanego mienia. Jednak oprócz właściwości, nie masz żadnego odniesienia do obiektu ponownie, więc przecieka. Dwa razy zwiększyłeś liczbę zatrzymań.

Aby ustalić, w kolejności preferencji:

1. Konwersja do SM;)

2. Tworzenie zmiennej lokalnej, należy przypisać go do właściwości, a następnie zwolnij go.

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

3. Dodaj połączenia autorelease do obiektu jak dodajesz go: self.property = [[[Object alloc] init] autorelease];