Konwertowanie obiektów zwalniających do ARC

Question

OK, więc Apple wprowadził do nas ARC, co jest świetne. Po refakturowaniu mojej aplikacji do ARC prawie wszystko działa dobrze i teraz jest o wiele łatwiej opracowywać i utrzymywać.

Jest tylko jeden problem, którego wciąż nie mogę rozgryźć.

Mój program zarządzania zadaniami wyświetla różne informacje o propozycjach, zamówieniach itd. W ich własnych oknach. Więc mam specjalną klasę gdzie WindowControllers zostaje przydzielony i zainicjowane z initWithWindowNibName a następnie okno jest pokazany z ShowWindow:

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

Przed ARC instancję WindowController zrobił uwalnianie jak pokazano na documentation:

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

Ale teraz z ARC nie jest to już możliwe i co jest jeszcze gorsze, w mojej specjalnej klasie, gdzie WindowController jest przydzielany i inicjowany, ten sam WindowController jest zwalniany przez ARC, ponieważ nie ma wskaźnika do windowController.

Mój pomysł był aby skopiować windowController się z mutuable tablicy:

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

A w metodzie windowControllers delegata windowWillClose mogę wysłać powiadomienie do mojego specjalnego Klasa:

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

W mojej klasie specjalnej Wysłuchuję powiadomienia i usuwam obiekt z tablicy:

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

Działa, ale nadal nie wierzę, że to jest poprawna metoda.

Czy ktoś ma ten sam problem lub napiwek, aby go poprawić?

Answer 1

Prawdopodobnie użyłbym raczej podejścia delegowanego niż powiadomienia. Generalnie lepiej jest mieć zewnętrzny obiekt, który śledzi otwarte okna. Samozachowujące się obiekty, takie jak stary system, łamią podstawowe punkty własności obiektu i utrudniają znajdowanie rzeczy (np. "Podaj listę otwartych okien").Non-Singleton, które są po prostu "pływające" często powracają, by cię ugryźć w twoją architekturę (musiałem to często naprawiać).

To powiedziawszy, czasami samoposiadanie jest co najmniej wygodne, aw najgorszym przypadku nie-na-końcu świata. Tak więc jesteś samowystarczalny. Jedyna różnica polega na tym, że musisz to zrobić jawnie, zamiast dopasowywać wyciek i nadpisywanie (co robił twój stary kod).

Utwórz prywatną właściwość strong. Przypisz do niego self. To utworzy pętlę zatrzymującą, która będzie Cię utrzymywać, dopóki nie ustawisz właściwości na nil.

Answer 2

Bez hacków nie ma eleganckiego sposobu na zatrzymanie obiektu poza silnym odniesieniem do niego w jakimś innym obiekcie. Możesz na przykład zachować statyczną wartość NSMutableArray/NSMutableSet, dodać tam swój kontroler i usunąć go w windowsWillClose:. To będzie krótsze niż wysyłanie powiadomienia. Aby było to możliwe do ponownego użycia, utwórz singleton WindowControllerRegistry z tablicą, do której dodajesz kontrolery, takie jak ten, i który automatycznie nasłuchuje NSWindowWillCloseNotification i usunie je z tablicy, zwalniając własność.

Jako szybki obejść ten problem, można wykonać retain/autorelease połączeń z 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

myślę, że alternatywne podejście powinno być poprawne, ale nie sądzę, trzeba drugie powiadomienie. Powinieneś być w stanie to zrobić:

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

Zakładając "proposalWindowArray" jest statycznym NSMutableArray.

Answer 4

Miałem ten sam problem, kiedy przestawiłem się na ARC. Twoje rozwiązanie działa, ale robisz to zbyt skomplikowane. Zasadniczo możesz zrobić to, co robiłeś wcześniej, przez samo otwarcie okna po zamknięciu, ale w sposób zgodny z ARC.

Rozwiązaniem jest proste utworzenie właściwości klasy w samej klasie. Na swoim przykładzie, w DetailWindowController, należy dodać następującą właściwość:

@property (strong) DetailWindowController  *theWindowController; 

Następnie podczas tworzenia okno z kodem powyżej, dodać jedną linię tak:

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

Wtedy wreszcie mieć ARC zwalnia okno, gdy jest ono zamknięte, podobnie jak w przypadku autorelease przed ARC, w klasie DetailWindowController, po prostu wykonaj:

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