ARC Problem mit Speicherfreigabe

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • ARC Problem mit Speicherfreigabe

    Moin zusammen,

    habe eine Frage, ob ich es momentan richtig mache! Ich habe mir die Doku durchlgesen und das nicht nur einmal.
    Ich weiß, dass man bei ARC eigentlich kein "nil" nutzen muss.
    Aber anders funktioniert es nicht. Hier mein Code:

    Quellcode

    1. - (void)viewDidLoad{
    2. [super viewDidLoad];
    4. //Slideshow
    5. slideshow = [[Slideshow alloc] initWithFrame:CGRectMake(0.0, 20.0, [[UIScreen mainScreen] bounds].size.width, [[UIScreen mainScreen] bounds].size.height) setVerzeichniss:@"Grunddaten/startbilder" setDateiendung:@"png"];
    6. [self.view insertSubview:slideshow atIndex:0];
    7. }
    9. - (void)viewDidDisappear:(BOOL)animated{
    11. //Slideshow
    12. [slideshow ende];
    13. [slideshow removeFromSuperview];
    14. slideshow = nil;
    15. }
    Alles anzeigen


    Speicher am Anfang 7,9 MB, nachdem ich im NavigationController push ausführe wird ja "viewDidDisappear" ausgeführt. Speicher sinkt auf 4,5 MB.

    Wenn ich den Code aber so verändere:

    Quellcode

    1. - (void)viewDidDisappear:(BOOL)animated{
    3. //Slideshow
    4. [slideshow ende];
    5. [slideshow removeFromSuperview];
    6. }


    Hier bleibt der Speicher auf 7,9 MB.
    Mach ich irgendwas komplett falsch?

    Vielen Dank schon mal für die Antworten...

  • slideshow ist vermutlich eine Instanzvariable oder Property, auch wenn Du fälschlicherweise keinen Setter/Getter benutzt. Daher erkennst du anscheinend auch nicht, dass ARC keine Veranlassung sieht, die Slideshow Instanz nicht durch das Entfernen vom Superview zu löschen. Es gibt ja schließlich noch eine strong Reference auf das Objekt.

    Michael

  • Hier der Code von Slideshow!

    Variable ist hier:

    Quellcode

    1. @interface Startseite ()
    2. {
    3. Slideshow *slideshow;
    4. }


    Slideshow.h

    Quellcode

    1. @interface Slideshow : UIView
    2. {
    3. int zaehler;
    4. int anz_bilder;
    5. NSString *ordner;
    6. NSString *dateiformat;
    7. UIImageView *iV_slideshow;
    8. }
    10. @property (nonatomic) BOOL aktiv;
    12. -(id)initWithFrame:(CGRect)frame setVerzeichniss:(NSString *)verzeichniss setDateiendung:(NSString *)dateiendung;
    14. //Einstellung
    15. -(void)einstellung;
    17. //Start
    18. -(void)start;
    20. //Ende
    21. -(void)ende;
    23. @end
    Alles anzeigen


    Slideshow.m

    Quellcode

    1. -(id)initWithFrame:(CGRect)frame
    2. setVerzeichniss:(NSString *)verzeichniss
    3. setDateiendung:(NSString *)dateiendung
    4. {
    5. self = [super initWithFrame:frame];
    6. if (self) {
    7. self.backgroundColor = [UIColor clearColor];
    9. self.aktiv = FALSE;
    11. //Verzeichniss
    12. ordner = verzeichniss;
    14. //Dateiendung
    15. dateiformat = dateiendung;
    17. //Slideshow
    18. iV_slideshow = [[UIImageView alloc]initWithFrame:CGRectMake(0, 0, frame.size.width, frame.size.height)];
    19. [self addSubview:iV_slideshow];
    20. }
    22. return self;
    23. }
    25. //Einstellung
    26. -(void)einstellung{
    28. //Anzahl Bilder
    29. anz_bilder = [Bilderimport anz_bilder:ordner];
    31. //Zähler
    32. zaehler = anz_bilder;
    33. }
    35. //Start
    36. -(void)start{
    38. if (iV_slideshow) {
    40. self.aktiv = TRUE;
    42. if (zaehler < anz_bilder) {
    43. zaehler++;
    44. }
    45. else{
    46. zaehler = 1;
    47. }
    49. [UIView transitionWithView:self
    50. duration:3.0
    51. options:UIViewAnimationOptionTransitionCrossDissolve
    52. animations:^{
    53. iV_slideshow.image = [UIImage imageWithContentsOfFile:[Bilderimport bilderzeichniss:ordner Filename:[NSString stringWithFormat:@"%d.%@",zaehler,dateiformat]]];
    54. }
    55. completion:^(BOOL finished){
    57. [NSTimer scheduledTimerWithTimeInterval:3.0
    58. target:self
    59. selector:@selector(start)
    60. userInfo:nil
    61. repeats:NO];
    62. }];
    63. }
    64. }
    66. //Ende
    67. -(void)ende{
    68. self.aktiv = FALSE;
    69. iV_slideshow = nil;
    70. }
    Alles anzeigen


    Hoffe hier ist das Problem zu erkennen!

  • Ich glaube Michael meinte dieses slideshow

    Quellcode

    1. slideshow = [[Slideshow alloc] initWithFrame:CGRectMake(0.0, 20.0, [[UIScreen mainScreen] bounds].size.width, [[UIScreen mainScreen] bounds].size.height) setVerzeichniss:@"Grunddaten/startbilder" setDateiendung:@"png"];


    Da es dort nicht deklariert ist, ist es sicher im ViewController deklariert und dort als iVar oder Property? Diese wird natürlich erst freigegeben wenn der ViewController freigegeben wird. Das passier aber nicht by viewDidDisappear, also bleibt die Slideshow auch erhalten.

    Wenn insertSubView() selber eine referenz hält (ich bin gerade niht sicher ab eigentlich ist das ja anzunehmen), dann solltest du slideshow einfach lokal deklarieren und damit wird DEINE REferenz direkt wieder freigegeben und die Refernz vom insertSubView dann beim viewDidDisappear

    Gruß

    Claus
    2 Stunden Try & Error erspart 10 Minuten Handbuchlesen.

    Pre-Kaffee-Posts sind mit Vorsicht zu geniessen :)

  • Thallius schrieb:

    Ich glaube Michael meinte dieses slideshow

    Quellcode

    1. slideshow = [[Slideshow alloc] initWithFrame:CGRectMake(0.0, 20.0, [[UIScreen mainScreen] bounds].size.width, [[UIScreen mainScreen] bounds].size.height) setVerzeichniss:@"Grunddaten/startbilder" setDateiendung:@"png"];


    Da es dort nicht deklariert ist, ist es sicher im ViewController deklariert und dort als iVar oder Property? Diese wird natürlich erst freigegeben wenn der ViewController freigegeben wird. Das passier aber nicht by viewDidDisappear, also bleibt die Slideshow auch erhalten.

    Wenn insertSubView() selber eine referenz hält (ich bin gerade niht sicher ab eigentlich ist das ja anzunehmen), dann solltest du slideshow einfach lokal deklarieren und damit wird DEINE REferenz direkt wieder freigegeben und die Refernz vom insertSubView dann beim viewDidDisappear

    Gruß

    Claus


    Danke für die Antwort! :)

    Über iVar:

    Quellcode

    1. @interface Startseite ()
    2. {
    3. Slideshow *slideshow;
    4. }


    Ich verstehe momentan diesen Part nicht:
    "dann solltest du slideshow einfach lokal deklarieren und damit wird DEINE REferenz direkt wieder freigegeben und die Refernz vom insertSubView dann beim viewDidDisappear"

    slideshow deklariere ich doch schon lokal in dem ViewController Startseite oder nicht?

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von kruemel1770 ()

  • Wie Michael schon gesagt hat, ist slideshow eine starke Referenz auf den View. Die verhindert das Löschen des Views bis Du oder die Runtime ihr ein neues Objekt (oder nil) zuweist. Dein Problem kannst Du auch über eine schwache Referenz lösen. Allerdings musst Du dann den View bis zum Einfügen über eine starke Referenz halten:

    Quellcode

    1. Slideshow *theSlideshow = [[Slideshow alloc] ...];
    3. [self addSubview:theSlideshow]; // Jetzt hält auch self die Slideshow
    4. slideshow = theSlideshow;
    „Meine Komplikation hatte eine Komplikation.“