objective-c - 为什么这个对象释放不正常，我应该释放它吗？

您必须原谅我，因为我对 Obj-C 还很陌生，但我很困惑..

我有这个带有 12 个按钮的小音板应用程序..每个按钮都调用相同的 IBAction..

当用户点击按钮时，我在 player 变量上调用 alloc init(在类的接口(interface)部分声明)

这一切都很好，花花公子:

#pragma mark - IBActions

-(IBAction)userDidTapButtonid)sender {
    [player stop];

    NSURL *soundClip = [NSURL fileURLWithPath:[[NSBundle mainBundle] pathForResource"clip" ofType"mp3"]];

    player = [[AVAudioPlayer alloc] initWithContentsOfURL:soundClip error:nil];
    [player setNumberOfLoops:-1];
    [player play];
}

#pragma mark - Cleanup

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

但是这感觉就像当我反复调用 alloc init 时我让内存悬空(因为我将播放器指针分配给一个新变量而不释放旧变量..)

为了解决这个问题，我尝试在 IBAction 顶部添加:

-(IBAction)userDidTapButtonid)sender {
    [player stop];
    [player release];

    ... etc ...

这在我第一次单击按钮时起作用(这对我来说似乎很奇怪，因为它实际上是一个空指针，因为它还没有被分配和初始化(对吗？))但是当我再次点击按钮时它会抛出一个 EXC_BAD_ACCESS 信号..

为什么？

我分配了内存，我不应该也释放它吗？

我应该如何释放它？

提前感谢!

---

Best Answer-推荐答案

所以我会告诉你我会怎么做以及为什么。

在你的 .h 文件中声明 player ivar 具有这样的属性

// .h
@interface MyClass : UIViewController

@property (nonatomic, retain) AVAudioPlayer *audioPlayer;

// method signatures

@end

我将它命名为 audioPlayer 只是为了更明确(​​这是个人喜好)。

在你的实现文件中，你需要像这样synthesize这个ivar

// .m
@implementation MyClass

@synthesize audioPlayer = _audioPlayer;

// Do some stuff

@end

这将创建支持 ivar 和 getter 和 setter 签名 - (void)setAudioPlayerAVAudioPlayer *)audioPlayer 和 - (AVAudioPlayer *)audioPlayer;但在后台，他们将操纵 ivar _audioPlayer。

您在回复中提到您来自 Ruby，这可以比作像这样的 attr_accessor :audio_player 但在 Objective-C 中，它创建的 setter 和 getter 不能根据是否处理内存管理您将 assign/retain/copy 传入 @property 行。

Apple 在他们的大多数示例中都是这样做的，这意味着当您直接访问 ivar 或通过 getter/setter 时会更清楚。

我现在将您的 -(IBAction)userDidTapButtonid)sender 更改为如下所示

-(IBAction)userDidTapButtonid)sender 
{
  [self.audioPlayer stop];

  NSURL *soundClip = [NSURL fileURLWithPath:[[NSBundle mainBundle] pathForResource"clip" ofType"mp3"]];

  AVAudioPlayer *tmpPlayer = [[AVAudioPlayer alloc] initWithContentsOfURL:soundClip error:nil];;
  self.audioPlayer = tmpPlayer;
  [tmpPlayer release]; tmpPlayer = nil;

  [self.audioPlayer setNumberOfLoops:-1];
  [self.audioPlayer play];
}

每当我与 audioPlayer ivar 交互时，我都会使用 getter/setter。这意味着每次我设置 ivar 时都会进行内存管理(例如，它释放旧播放器并保留新播放器)。使用 getters/setters 的原因是 self.audioPlayer 将被编译为适当的调用，如下所示:

self.audioPlayer;             // compiled to -> [self audioPlayer];
self.audioPlayer = tmpPlayer; // compiled to -> [self setAudioPlayer:tmpPlayer];

现在整理并使 - (void)dealloc; 方法正确，我们应该直接使用 ivar 而不是通过 getter/setter 所以我必须使用 _audioPlayer 我们合成的 ivar 是这样的:

#pragma mark - Cleanup

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

关于objective-c - 为什么这个对象释放不正常，我应该释放它吗？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/6843669/