Release UINavigationController in ARC

necromonger · 21. Februar 2017, 19:28

Hallo Forum-Mitglieder,

ich habe eine kurze Frage. Ich habe einen UINavigationscontroller und lade zusätzlich einen Navigationscontroller hinzu. Wenn ich jetzt vom 2 in den 1 Navigationcontroller zurück springe, ist laut Speicherverbrauchsanzeige
in Xcode der Navigationscontroller immer noch im Speicher. Wie kann ich diesen in ARC releasen.

So lade ich den zweiten Navigationscontroller:

Quellcode

FileViewer_Collection_ViewController* ViewerController = [self.storyboard instantiateViewControllerWithIdentifier:@"fileviewer_controller"];
ViewerController.Pfad = [filename.filename_url path];
ViewerController.title = filename.filename_str;
[self.navigationController pushViewController:ViewerController animated:YES];

Wäre über einen Tipp oder Rat sehr Dankbar!

Gruss
necromonger

macmoonshine · 21. Februar 2017, 19:33

necromonger schrieb:

Wenn ich jetzt vom 2 in den 1 Navigationcontroller zurück springe, ist laut Speicherverbrauchsanzeige
in Xcode der Navigationscontroller immer noch im Speicher. Wie kann ich diesen in ARC releasen.

Woher weist du, dass das genau dein Viewcontroller ist? Hast du dir deine App mal mit Instruments angesehen?

Du kannst in ARC kein Objekt explizit aus dem Speicher werfen. Dafür steht das A in ARC.

necromonger · 21. Februar 2017, 19:46

Hallo macmoonshine,

ich gehe davon aus, dass der Navigationscontroller aus dem Speicher nicht mehr freigegeben wird. Mit Instruments habe ich es noch nicht versucht, ich muss ehrlich sagen
ich blicke auch nicht ganz durch bei Instruments. Wenn ich vom 1 in den 2 Navigationscontroller springe erhöht sich immer weiter der verbrauchte Speicher im Arbeitsspeicher, um
so öfter ich hin und her springe!

Gruss
necromonger

macmoonshine · 21. Februar 2017, 19:59

Hast du noch andere Referenzen auf den Navigationcontroller.

Deine Beschreibung hört sich nach einem Retain-Cycle an. Könnten Code-Blöcke mit self sein.

necromonger · 21. Februar 2017, 20:03

In dem Navigationscontroller ist ein UICollectionView mit einem delegate = self und zwei NSNotification mit addObserver = self;

Das heißt er schließt den UINavigationsController nicht, weil diese Referenzen nicht abgeschloßen werden?

macmoonshine · 21. Februar 2017, 20:38

Delegates sind in der Regel auch schwache Referenzen. Daran sollte es auch nicht liegen.

Das Notification-Center hält seine Observer nicht. Wenn ARC einen Observer wegräumt gibt es stattdessen einen Satz heißer Ohren. Außer du verwendest Code-Blöcke als Observer. Wenn die self enthlten, hast du schnell einen Retain-Cycle.

necromonger · 21. Februar 2017, 20:54

Danke für deine Hilfe. Also Code-Blöcke enthalten die Observer nicht. Ich führe aber in dem UINavigationcontroller eine Dispatch_async aus.

Quellcode

dispatch_async(dispatch_get_global_queue( DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^(void){
dispatch_async(dispatch_get_main_queue(), ^(void){
UIAlertController *alertController = [UIAlertController
alertControllerWithTitle:@"Erzeuge"
message:nil
preferredStyle:UIAlertControllerStyleActionSheet];
UIAlertAction *cancelAction = [UIAlertAction
actionWithTitle:NSLocalizedString(@"Abbrechen", @"Cancel action")
style:UIAlertActionStyleCancel
handler:^(UIAlertAction *action)
{
NSLog(@"Cancel action");
}];
UIAlertAction *okAction = [UIAlertAction
actionWithTitle:NSLocalizedString(@"neuen Ordner", @"OK action")
style:UIAlertActionStyleDefault
handler:^(UIAlertAction *action)
{
[self createnewfolder_controller];
}];
[alertController addAction:cancelAction];
[alertController addAction:okAction];
alertController.popoverPresentationController.barButtonItem = _action_toolbar_add_button;
alertController.popoverPresentationController.sourceView = self.view;
[self presentViewController:alertController animated:YES completion:nil];
});
});

Alles anzeigen

Könnte das zu diesem Problem führen?

macmoonshine · 21. Februar 2017, 20:59

Kannst du den Code mal einrücken.

Wie dem auch sei. Die ersten beiden Zeilen sind so schon mal wenig sinnvoll. Du kannst die erste und letzte Zeile getrost streichen.

Die Code-Blöcke der Alert-Actions halten self solange, bis der Alert-Controller das Zeitliche segnet.

necromonger · 21. Februar 2017, 21:02

Quellcode

dispatch_async(dispatch_get_global_queue( DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^(void){
dispatch_async(dispatch_get_main_queue(), ^(void){
UIAlertController *alertController = [UIAlertController alertControllerWithTitle:@"Erzeuge" message:nil preferredStyle:UIAlertControllerStyleActionSheet];
UIAlertAction *cancelAction = [UIAlertAction actionWithTitle:NSLocalizedString(@"Abbrechen", @"Cancel action") style:UIAlertActionStyleCancel
handler:^(UIAlertAction *action)
{
NSLog(@"Cancel action");
}];
UIAlertAction *okAction = [UIAlertAction actionWithTitle:NSLocalizedString(@"neuen Ordner", @"OK action") style:UIAlertActionStyleDefault
handler:^(UIAlertAction *action)
{
[self createnewfolder_controller];
}];
[alertController addAction:cancelAction];
[alertController addAction:okAction];
alertController.popoverPresentationController.barButtonItem = _action_toolbar_add_button;
alertController.popoverPresentationController.sourceView = self.view;
[self presentViewController:alertController animated:YES completion:nil];
});
});

Alles anzeigen

macmoonshine · 21. Februar 2017, 21:15

Quellcode

???

necromonger · 21. Februar 2017, 21:17

Habe die 1. Zeile komplett entfernt.

Im viewDidload habe ich noch folgende Zeile um eine Funktion aus dem AppDelegate abzufragen:

Quellcode

_appDelegate = (AppDelegate*)[[UIApplication sharedApplication] delegate];

Könnte das Probleme machen?

macmoonshine · 21. Februar 2017, 21:20

Du hast doch das Problem, dass ARC anscheinend Viewcontroller nicht freigibt. Was sollte das App-Delegate damit zu tun haben?

necromonger · 21. Februar 2017, 21:25

Ich komme einfach nicht weiter! Kann ich Dir den Code vom NavigationController zukommen lassen. Vielleicht kannst du mir ja helfen, wo der Fehler liegt.

macmoonshine · 21. Februar 2017, 22:00

Nö

Du kannst übrigens mit __weak typeof(self) theSelf = self; eine schwache Referenz auf self erzeugen.

necromonger · 24. Februar 2017, 20:20

Ich wollte nicht anmassend sein, ich dachte nur weil du oben geschrieben hast (Quellcode ???). Ich habe schon etwas gegoogelt, leider ohne Erfolg. Kannst du mir ein Buch oder eine Tutorialwebsite empfehlen, wie ich so einen Fehler mit Instruments suchen kann.

Danke

MCDan · 25. Februar 2017, 00:38

Ausgehend von dem Source Code in Deinem 1. Post: Bist Du sicher, dass FileViewer_Collection_ViewController ein Navigation Controller ist?

Eigentlich lassen sich keine Navigation Controller auf den Stack eines Navigation Controllers pushen. Da sollte es zur Laufzeit eine Fehlermeldung geben und der Controller sollte sich dann nicht auf den Stack pushen lassen.

necromonger · 25. Februar 2017, 12:30

Ok, hier das Problem etwas genauer beschrieben:

Das Projekt sieht wie folgt aus (Interface):
[Blockierte Grafik: http://www.stachowiak-software.de/Forum/bild1.png]
Link zum Bild zur Ansicht

Der Navigationscontroller wird in einem Storyboard-Reference geladen. In dem Navigationscontroller werden Ordner und weiteres angezeigt. Wenn ich jetzt einen Ordner anklicke, wird ein zusätzlicher Navigationscontroller geladen. Mit folgendem Code

C-Quellcode

__weak typeof(self) weakSelf = self;
FileViewer_Collection_ViewController* ViewerController = [weakSelf.storyboard instantiateViewControllerWithIdentifier:@"fileviewer_controller"];
ViewerController = [weakSelf.storyboard instantiateViewControllerWithIdentifier:@"fileviewer_controller"];
ViewerController.Pfad = [filename.filename_url path];
ViewerController.title = filename.filename_str;
[weakSelf.navigationController pushViewController:ViewerController animated:YES];

Er lädt sich quasi selber noch einmal, mit der gleichen Klasse. Mein Problem stützt sich auf folgende Tatsache. Wenn ich jetzt von dem ersten zu einem neuen Navigationscontroller springe und wieder zurück springe, scheint sich der letzte Navigationscontroller nicht aus dem Speicher entfernen.

So sieht der Vorgang im Simulator aus:
[Blockierte Grafik: http://www.stachowiak-software.de/Forum/bild2.gif]
Link zum Bild zur Ansicht

Und das passiert in der Speicherverbrauchs-Anzeige bei Xcode:
[Blockierte Grafik: http://www.stachowiak-software.de/Forum/bild3.gif]
Link zum Bild zur Ansicht

In Instruments (wo ich noch nicht wirklich durchblicke!) kann ich folgendes feststellen, falls es richtig ist:
[Blockierte Grafik: http://www.stachowiak-software.de/Forum/bild4.gif]
Link zum Bild zur Ansicht

Ich würde davon ausgehen, dass die Funktion "Files_in_a_directory" nicht released wird oder? Leider werden die Bilder nicht angezeigt. Leider werden die Bilder anscheinend nicht angezeigt. Ich habe nochmals Links unter die Bilder gemacht!

MCDan · 25. Februar 2017, 12:48

Wie gesagt, mit pushViewController:animated: lässt sich kein weiterer UINavigationController auf den Stack eines UINavigationControllers pushen. Dies führt zu einer Fehlermeldung zur Laufzeit.

FileViewer_Collection_ViewController kann also keine Referenz auf ein UINavigationController sein.

Ist dieser Source Code 1 zu 1 aus dem Projekt?

Warum erzeugst Du 2 mal einen ViewController per [weakSelf.storyboard instantiateViewControllerWithIdentifier:@"fileviewer_controller"]?

necromonger · 25. Februar 2017, 12:53

Hallo MCDan,

leider werden die Bilder nicht angezeigt. Ich habe nochmals Links davon gemacht. Eine Fehlermeldung bekomme ich nicht. Und es funktioniert auch tadellos, nur das der Speicher
irgendwann überläuft. Auf den Bildern sieht man den Vorgang genau.

Danke für den Tip. Das habe ich komplett übersehen. Zeile entfernt! Trotzdem ohne Wirkung!

Gruss
David

necromonger · 25. Februar 2017, 23:05

Hallo MCDan,

Stimmt. FileViewer_Collection_ViewController ist ein UIViewController und kein UINavigationscontroller. Sorry, hatte mich vorher nicht korrekt ausgedrückt! Das wäre theoretisch so in Ordnung! Oder mache ich etwas beim Lade des 2. ViewControllers falsch oder habe ich einen Denkfehler?

Gruss
David