Problema invalidar NSTimer en dealloc

Question

siguiente es mi código:

archivo .h:

#import "Foundation/Foundation.h" 
@interface GObject:NSObject{ 
    NSTimer* m_Timer; 
} 
@property(nonatomic, retain) NSTimer* m_Timer; 

- (void)Initialize; 
- (void)TimerCallback:(NSTimer*)pTimer; 
@end 

archivo .m:

@implementation GObject 

@synthesize m_Timer 

- (void) Initialize{ 
    self.m_Timer = [NSTimer scheduledTimerWithTimeInterval:5.0 
         target:self 
         selector: @selector(TimerCallback:) 
         userInfo: nil 
         repeats: YES]; 

} 

- (void)TimerCallback:(NSTimer*)pTimer { 
    //Some Code 
} 
- (void)dealloc { 
    [m_Timer invalidate]; //--Crashes Here 
    [m_Timer release]; 
    m_Timer = nil; 
    [super dealloc]; 
} 
@end 

Ahora cuando el dealloc es llamado, el programa se bloquea en la línea que invalida el temporizador. Las siguientes dos líneas ni siquiera se llaman. Aparece un error de "EXC_BAD_ACCESS". ¿Alguien puede decirme por qué podría estar pasando eso, y cuál es la forma correcta de detener y liberar una variable de miembro de NSTimer en una clase?

Answer 1

Hice algunas investigaciones y pruebas, y pude encontrar la respuesta a mi propia pregunta. Ok, aquí va:

Cuando asignamos self como destino al NSTimer, el temporizador guarda una referencia a nuestro objeto. Si el temporizador se repite (o tiene un período de tiempo largo), no se invalidaría por sí mismo (o tomaría demasiado tiempo para invalidar automáticamente, si no se repite). Entonces, incluso si el objeto se lanza mientras tanto, no llamaría al método dealloc, ya que el conteo de retención sería al menos de 1. Ahora, estaba intentando forzar la ubicación de mi objeto llamando a lanzamientos repetidos hasta que retenga el conteo se convirtió en 0. Ese fue mi error.

Pero si no lo hace, su objeto se mantendrá con vida, y eventualmente tendrá una pérdida de memoria ya que pierde el resto de las referencias a ese objeto por varias versiones. El único que se conservará será el NSTimer. Esto suena como una situación de punto muerto. Mi código se bloqueó, porque cuando el delloc intentó invalidar el NSTimer, intentó liberar la referencia que estaba reteniendo. Pero como había sido un listillo y había reducido el conteo de retención a 0, eso causaría una excepción de memoria.

Para resolver esto, primero limpié mi acto y eliminé el código para deslocalizar forzosamente el objeto.Entonces, justo antes de querer que el objeto fuera asignado, llamé a la función invalidada de NSTimer. Esto liberó la instancia de destino que tenía el temporizador. Después de eso, al llamar al release en mi objeto, se lo asignó con éxito.

La línea inferior es, si su objeto tiene NSTimers que se repiten o no invalidan (superan) automáticamente, nunca los invalide en la función delloc. El delloc no se ejecutará hasta que el temporizador retenga la instancia de su objeto. En su lugar, tiene una función de limpieza para invalidar los temporizadores, antes de liberar el objeto. Esa es la solución que se me ocurrió. Si hay uno mejor, ciertamente quiero saberlo.

Answer 2

Hay algunas cosas que debe resolver para limpiar esto.

Su declaración de clase está un poco desajustada. Si desea heredar de NSObject, la sintaxis correcta es:

@interface GObject : NSObject 

En su aplicación, se deben implementar - (id)init en lugar de - (void)Initialize. No hay ningún método de instancia - (void)Initialize ... hay un método estático + (void)initialize. Tenga en cuenta el + y la diferencia en mayúsculas, que es significativo): el método initialize se llama una vez en su programa antes de que la clase reciba su primer método.

En este caso, su método Initialize no se está llamando en absoluto (se escribe mal, y es un método de instancia en lugar de un método estático). En su lugar, se desea implementar init, que es el inicializador designado para casos NSObject:

- (id)init { 
    if (self = [super init]) { 
     self.m_Timer = [NSTimer scheduledTimerWithTimeInterval:5.0 
        target:self 
        selector: @selector(TimerCallback:) 
        userInfo: nil 
        repeats: YES]; 
    } 
    return self; 
} 

Por último, asegúrese de usar el símbolo @ antes de la instrucción de propiedad:

@property(nonatomic, retain) NSTimer* m_Timer; 

Y no se olvide para sintetizarlo en su implementación:

@implementation GObject 

@synthesize m_Timer; 

Answer 3

Tengo el mismo problema de pérdida de memoria al utilizar NSTimer repetido en uno de los controladores de vista. Este controlador de vista no se liberará cuando debería. Finalmente resolví este problema moviendo mi temporizador al código delegado de la aplicación principal donde no hay necesidad de liberar el temporizador.

Answer 4

Bueno, tengo que decir que puede consultar la documentación del desarrollador de apple, como la referencia de clase de Apple. Puede ver desde allí que cuando se invalida: se invoca el método, se llamará al método de liberación del temporizador antes de invalidar: el método retorna.

Answer 5

Otra solución sería crear un objeto separado que se ocupe de la devolución de llamada NSTimer (algunos TimerController). Ese objeto, si es necesario, puede llamar a los métodos en su controlador self.

Ahora el temporizador no mantener una referencia a self y cuando self consigue desasignar, - dealloc debe ser llamado y se puede invalidar y ajustar a cero el TimerController y el propio temporizador.