无效和重新创建 NSTimer(s) 的问题

Question

我在启动和停止 NSTimers 时遇到问题。文档说计时器被 [timer invalidate] 停止；

我有一个这样声明的计时器对象

.h
NSTimer *incrementTimer;
@property (nonatomic, retain) NSTimer *incrementTimer;
.m
@synthesize incrementTimer;
-(void)dealloc {
 [incrementTimer release];
 [super dealloc];
 }

-通常。

在需要时，我的方法会执行以下操作：

-(void)setGenCount {
    if(!condition1 && condition2) {
        incrementTimer = [NSTimer scheduledTimerWithTimeInterval: 2.0 
                                                      target: self 
                                                    selector:@selector(incrementBatteryVoltage:) 
                                                    userInfo: nil 
                                                     repeats: YES]; 
    }
}

上面的一切都很好。但是，一旦那个计时器完成了它的工作，我希望它自己失效。我使计时器无效，因为可以调用一个相等的递减方法，如果它仍然处于活动状态，它将与 incrementTimer 对抗。 （以前，我注意到我的两个计时器，如果处于活动状态，通过增加和减少值（一种战斗）来作用于同一个 ivar （一种战斗）......没有崩溃）调用的选择器如下工作：

-(void)incrementBatteryVoltage:(NSTimer *)timer {
    if(battVoltage < 24.0) {
         generatorDisplay.battVoltage += 0.1;
      }
    if(battery1Voltage == 24.0) {
         [timer invalidate];
      }
  }

我有一个相等的方法来减少电池计数。 （前面提到过）
由于我的程序设计：界面模拟电压显示。当“机器”关闭时，我希望所有定时器都失效，不管电压值是多少。我通过检查计时器是否有效来做到这一点。

-(void)deEnergizeDisplays {

   if([decrementTimer isValid]) {
        [decrementTimer invalidate];
        decrementTimer = nil;
     }

    if([incrementTimer isValid]) {
       [incrementTimer invalidate];
       incrementTimer = nil;
    }

我遇到了许多“BAD_ACCESS”崩溃。错误的线路呼叫总是指向我的 [timer isValid] 呼叫。似乎如果计时器无效......指针 也不存在。我知道 [timer invalidate] 消息会禁用计时器，然后将其从运行循环中删除，然后将其释放。我的理解是：根据命名约定，它是一个自动释放的对象。

我的想法是：如果我要发送保留消息，引用不应该仍然存在吗？我尝试了几种组合，带走：

timer = nil;

甚至代替：

if([timer isValid])

我试过了：

if([timer != nil])

和：

if(timer)

我总是遇到同样的崩溃。感谢您在启动和停止 NSTimers 方面的任何帮助。

Answer 1

更新：见Darren's answer。问题是您在设置计时器时没有使用您的属性访问器。而不是：

incrementTimer = [NSTimer ...

你应该有：

self.incrementTimer = [NSTimer ...

self.propertyName = ... 语法将调用您的访问器方法，从而自动保留您发送给它的对象（因为您的属性设置为retain）。简单地调用propertyName = ... 并不 使用属性访问器。您只是直接更改 ivar 的值。

---

更新 #2： 在与 Peter Hosey（参见 cmets）进行了一次富有启发性的对话后，我删除了我之前的建议，即“永远不要保留或释放”您的计时器对象。我还完全重写了我之前的代码，因为我认为以下是更好的方法：

Controller.h：

NSTimer *voltageTimer;
float targetBatteryVoltage;
...
@property (nonatomic, retain) NSTimer *voltageTimer;

Controller.m：

@implementation Controller
@synthesize voltageTimer;

- (void)stopVoltageTimer {
    [voltageTimer invalidate];
    self.voltageTimer = nil;
}

- (void)setTargetBatteryVoltage:(float)target {
    [voltageTimer invalidate];
    targetBatteryVoltage = target;
    self.voltageTimer = [NSTimer scheduledTimerWithTimeInterval: 2.0
                                target: self
                              selector: @selector(updateBatteryVoltage:)
                              userInfo: nil
                               repeats: YES];
}

- (void)updateBatteryVoltage:(NSTimer *)timer {
    const float increment = 0.1;
    if (abs(battVoltage - targetBatteryVoltage) < increment) {
        [timer invalidate];
    }
    else if (battVoltage < targetBatteryVoltage) {
        generatorDisplay.battVoltage += increment;
    }
    else if (battVoltage > targetBatteryVoltage) {
        generatorDisplay.battVoltage -= increment;
    }
}

现在，您可以简单地设置一个目标电池电压，定时器的魔法就会在幕后发生：

[self setTargetBatteryVoltage:24.0];

您的关机方法如下所示：

- (void)deEnergizeDisplays {
    [self stopVoltageTimer];
}

Answer 2

您需要将retain 分配给incrementTimer 的值在setGenCount 中。您可以使用合成属性自动执行此操作，该属性可通过self. 访问：

self.incrementTimer = [NSTimer scheduledTimerWithTimeInterval: ...