I have a NSTimer
object that I need to invalidate if a user taps on a button or if they exit a view.
So I have:
[myNSTimer invalidate];
I have had the same problem while I was trying to pause & restart a timer. To stop it:
[timer invalidate];
[timer release];
timer = nil;
And to start/ restart it:
timer = [[NSTimer scheduledTimerWithTimeInterval:3 target:self selector:@selector(aSelector) userInfo:nil repeats:YES] retain];
Now it doesn't crash when I invalidate it.
What I usually do is nil-out the member variable holding the timer, either when it fires, or when I invalidate it. Then you can just do:
[myNSTimer invalidate];
myNSTimer = nil;
and it'll do the right thing.
Swift:
if timer.valid {
timer.invalidate()
}
Once you invalidate the timer, simply call release
on it (assuming you've retained the reference you're holding on to) and then nil out your reference. That way when you exit the view, trying to invalidate the timer a second time will just call that method on nil instead, which does nothing.
Alternatively, you can use -[NSTimer isValid]
to check if it's valid before invalidating, but there's really no reason to hold onto your reference after invalidating it the first time anyway. Also, if your real problem is that you haven't retained your reference and so the first -invalidate
actually leaves you with a reference pointing at a released object, then calling -isValid
won't help anyway.