Convirtiendo objetos de auto-liberación a ARC

Question

OK, entonces Apple nos trajo ARC, lo cual es genial. Después de refacturar mi Aplicación a ARC, casi todo funciona bien y ahora es mucho más fácil de desarrollar y mantener.

Solo hay un problema que todavía no puedo resolver.

Mi programa de administración de trabajos muestra información detallada de propuestas, pedidos, etc. en sus propias ventanas. Así que tengo una clase especial donde WindowControllers consigue asignan y se iniciaron con initWithWindowNibName y luego la ventana se muestra con SHOWWINDOW:

DetailWindowController *proposalWindowController = [[DetailWindowController alloc] initWithWindowNibName:@"ThePorposalWindow"]; 
[proposalWindowController showWindow:nil]; 

Antes ARC la instancia de la WindowController hizo la liberación como se muestra en la documentation:

- (void)windowWillClose:(NSNotification *)notification 
{ 
    [self autorelease]; 
} 

Pero ahora con ARC esto ya no es posible y lo que empeora aún más, en mi clase especial donde se asigna e inicia el WindowController, ARC lanza el mismo windowController porque no hay ningún puntero al WindowController.

Mi idea era copiar el windowController en una matriz mutuable:

[proposalWindowArray addObject:proposalWindowController]; 
[[proposalWindowArray lastObject] showWindow:nil]; 

Y en el método windowControllers delegado windowWillClose puedo enviar una notificación a mi clase especial:

- (void)windowWillClose:(NSNotification *)notification 
{ 
    [[NSNotificationCenter defaultCenter] postNotificationName:@"ProposalWindowWillClose" object:[[self window] windowController] userInfo:nil]; 
} 

En mi clase especial Escucho la notificación y elimino el objeto de la matriz:

- (void) proposalWindowWasClosed: (NSNotification *) notification 
{ 
    [proposalWindowArray removeObjectIdenticalTo:[notification object]]; 
} 

Funciona, pero sigo sin creer que esta sea la forma correcta.

¿Alguien tiene el mismo problema o un consejo para mejorarlo?

Answer 1

Probablemente usaría un enfoque de delegado en lugar de notificaciones. En general, es mejor tener un objeto externo que realice un seguimiento de las ventanas abiertas. Los objetos autoretenidos, como su sistema antiguo, rompen los puntos básicos de la propiedad del objeto y hacen que sea difícil encontrar cosas (como "darme una lista de ventanas abiertas").Los Non-Singletons que simplemente están "flotando" por ahí a menudo vuelven para morderte en tu arquitectura (he tenido que arreglar esto con frecuencia).

Dicho esto, a veces la autopropiedad es al menos conveniente, y en el peor de los casos, no es el fin del mundo. Por lo tanto, auto-propio. La única diferencia es que debe hacerlo de forma explícita en lugar de coincidir con una fuga y una sobreexplotación (que es lo que hacía su código anterior).

Cree una propiedad privada strong. Asignar self a él. Eso creará un bucle de retención que lo mantendrá cerca hasta que establezca la propiedad en nil.

Answer 2

Sin hacks, no hay una manera elegante de mantener un objeto retenido que no sea una referencia fuerte en algún otro objeto. Por ejemplo, puede mantener un NSMutableArray/NSMutableSet estático, agregar su controlador allí y eliminarlo en windowsWillClose:. Esto será más corto que publicar una notificación. Para que esto sea reutilizable, cree un singleton WindowControllerRegistry con una matriz, donde puede agregar controladores como este, y que escuchará automáticamente NSWindowWillCloseNotification y los eliminará de su matriz, liberando así la propiedad.

Como solución rápida, puede realizar retain/autorelease llamadas de non-ARC file:

my_retain(self); 
my_autorelease(self); 

// ArcDisabled.mm 
void my_retain(id obj) { [obj retain]; } 
void my_autorelease(id obj) { [obj autorelease]; } 

Answer 3

Creo que su enfoque alternativo debe ser correcta, pero no creo que necesita la segunda notificación. Usted debe ser capaz de hacer:

- (void)windowWillClose:(NSNotification *)notification 
{ 
    [proposalWindowArray removeObjectIdenticalTo:self]; 
} 

Suponiendo que el "proposalWindowArray" es una NSMutableArray estática.

Answer 4

Tuve este mismo problema cuando cambié a ARC. Su solución funciona, pero la está haciendo demasiado complicada. En esencia, puede hacer lo que hacía antes haciendo que la ventana se libere cuando se cierra, pero de forma compatible con ARC.

La solución es simplemente crear una propiedad de su clase dentro de la misma clase. Para su ejemplo, en DetailWindowController, debe agregar la siguiente propiedad:

@property (strong) DetailWindowController  *theWindowController; 

A continuación, cuando se crea la ventana con el código anterior, añadir una línea así:

DetailWindowController *proposalWindowController = [[DetailWindowController alloc] initWithWindowNibName:@"ThePorposalWindow"]; 
[preferenceController setTheWindowController:proposalWindowController]; 
[proposalWindowController showWindow:nil]; 

Entonces, finalmente, tener ARC libera la ventana cuando está cerrada como lo hizo con la liberación automática pre-ARC, en la clase DetailWindowController, simplemente haga:

- (void)windowWillClose:(NSNotification *)notification 
{ 
    // Let ARC tear this down and clean it up 
    [self setTheWindowController:nil]; 
}