NSTimer lässt Anwendung einfrieren

  • NSTimer lässt Anwendung einfrieren

    Hi,
    ich versuche gerade alle fünf Sekunden Daten via TCP von einem Server zu holen. Das Holen der Daten funktioniert erfreulicherweise einwandfrei. Dummerweise lässt der Timer aber meine Anwendung einfrieren.

    Quellcode

    1. - (void)awakeFromNib
    2. {
    3. // Timer starten: alle 5 Sekunden Daten holen
    4. trackTimer = [[NSTimer scheduledTimerWithTimeInterval:5.0 target:self selector:@selector(trackTimerFired:) userInfo:nil repeats:YES] retain];
    6. // Aber bitte jetzt schonmal
    7. [self trackTimerFired:nil];
    8. }
    10. - (void)trackTimerFired:(NSTimer*)timer
    11. {
    12. Socket *controlSocket = [Socket socket];
    13. NSMutableData *response = [NSMutableData data];
    14. NSString *caption = [NSString string];
    16. // Künstler holen
    17. [controlSocket connectToHostName:@"192.168.178.22" port:1308];
    18. [controlSocket writeString:@"get current artist"];
    19. [controlSocket readData:response];
    20. caption = [caption stringByAppendingString:[[NSString alloc] initWithData:response encoding:NSUTF8StringEncoding]];
    21. // zurücksetzen
    22. [response init];
    23. // Tracknamen holen
    24. [controlSocket writeString:@"get current trackname"];
    25. [controlSocket readData:response];
    26. caption = [caption stringByAppendingString:@"\n"];
    27. caption = [caption stringByAppendingString:[[NSString alloc] initWithData:response encoding:NSUTF8StringEncoding]];
    29. // TextField setzen
    30. [currentTrack setStringValue:[caption copy]];
    32. // Alles weg damit
    33. [response release];
    34. [caption release];
    35. [controlSocket release];
    36. }
    Alles anzeigen


    Beim Aufruf der Timer-Methode (egal ob das jetzt manuell geschieht oder erst beim Firen des Timers) bleibt das Programm stehen und das Debuggerfenster öffnet sich. Ich hab gerade keine schlüssige Erklärung warum.
    Hat jemand da eine Idee?

    Chris

  • Irgendwo wird ein Log stehen mit so etwas wie EXC_BAD_ACCESS. ;)

    @Calgan:
    Bitte lies dir menen Artikel zur Speicherverwaltung durch:
    macentwicklerwelt.net/doku.php?id=wiki:speicherverwaltung

    oder
    <Werbeeinblendung>
    Rodewig/Negm-Awad, Objective-C und Cocoa, S. 132-176
    </Werbeeinblendung>
    Es hat noch nie etwas gefunzt. To tear down the Wall would be a Werror!
    25.06.2016: [Swift] gehört zu meinen *Favorite Tags* auf SO. In welcher Bedeutung von "favorite"?

  • RE: NSTimer lässt Anwendung einfrieren

    Also, ich schreib dir die Fehler hier rein:

    Quellcode

    1. - (void)awakeFromNib
    2. {
    3. trackTimer = [[NSTimer scheduledTimerWithTimeInterval:5.0 target:self selector:@selector(trackTimerFired:) userInfo:nil repeats:YES] retain];
    5. [self trackTimerFired:nil];
    6. }
    8. - (void)trackTimerFired:(NSTimer*)timer
    9. {
    10. Socket *controlSocket = [Socket socket];
    11. NSMutableData *response = [NSMutableData data];
    12. NSString *caption = [NSString string]; //ganz weg die zeile, weil du unten den pointer caption eh neu setzt
    14. [controlSocket connectToHostName:@"192.168.178.22" port:1308];
    15. [controlSocket writeString:@"get current artist"];
    16. [controlSocket readData:response];
    17. caption = [caption stringByAppendingString:[[NSString alloc] initWithData:response encoding:NSUTF8StringEncoding]]; //Autorelease hinter "alloc/init"
    19. [response init]; //weg damit, das wurde schoin initialisiert, niemals mehr als ein init
    21. [controlSocket writeString:@"get current trackname"];
    22. [controlSocket readData:response];
    23. caption = [caption stringByAppendingString:@"\n"];
    24. caption = [caption stringByAppendingString:[[NSString alloc] initWithData:response encoding:NSUTF8StringEncoding]]; //autorelease hinter "alloc/init"
    27. [currentTrack setStringValue:[caption copy]]; //autorelease hinter copy, copy bring immer ein object zurück mit retaincount 1, kein autoreleastes
    30. [response release];
    31. [caption release]; //caption ist autoreleased, nicht releasen
    32. [controlSocket release]; //controlsocket das gleiche wie caption
    33. }
    Alles anzeigen


    ich hätte den code so geschrieben, so ist es für mich übersichtlicher:

    Quellcode

    1. - (void)awakeFromNib
    2. {
    3. trackTimer = [[NSTimer scheduledTimerWithTimeInterval:5.0 target:self selector:@selector(trackTimerFired:) userInfo:nil repeats:YES] retain];
    5. [self trackTimerFired:nil];
    6. }
    8. - (void)trackTimerFired:(NSTimer*)timer
    9. {
    10. Socket *controlSocket = [Socket socket];
    12. [controlSocket connectToHostName:@"192.168.178.22" port:1308];
    14. [controlSocket writeString:@"get current artist"];
    15. NSMutableData *currentArtist = [NSMutableData data];
    16. [controlSocket readData:currentArtist];
    19. [controlSocket writeString:@"get current trackname"];
    20. NSMutableData *currentTrackname = [NSMutableData data];
    21. [controlSocket readData: currentTrackname];
    23. NSString *caption = [[[NSString alloc] initWithData:currentArtist encoding:NSUTF8StringEncoding] autorelease];
    24. caption = [caption stringByAppendingString:@"\n"];
    25. caption = [caption stringByAppendingString:[[[NSString alloc] initWithData:currentTrackNameencoding:NSUTF8StringEncoding] autorelease]];
    28. [currentTrackField setStringValue:caption];
    29. }
    Alles anzeigen

  • Hi,
    danke für die Erläuterungen. Ich habe mich wohl ein wenig zu sehr an alte Gewohnheiten gehalten, alle Objekte bei Nicht-Mehr-Verwendung freizugeben.

    Den Artikel werde ich mir auf jeden Fall durchlesen, ich denke, dass sich dann einige Unklarheiten beseitigt sein sollten.

    Der Code sieht wirklich übersichtlicher aus, danke. War wohl gestern Abend ein wenig spät. *g*
    Habe das response init rein gemacht, weil ich das Data-Objekt wieder leeren wollte, um neue Daten reinzuschreiben und bei NSData kein clear oder so gefunden. Und bei den übrigen Sachen wollte ich nicht, dass mir Objekte den Speicher zu müllen. Aber scheinbar braucht man sich um Speichermanagement nicht wirklich zu kümmern.

    Danke für eure Hilfe,
    Chris

  • Original von CalganX
    Habe das response init rein gemacht, weil ich das Data-Objekt wieder leeren wollte, um neue Daten reinzuschreiben und bei NSData kein clear oder so gefunden.

    Zum leeren eines NSMutableData Objekt fallen mir da spontan zwei Möglichkeiten ein:

    Quellcode

    1. [theMutableData setData:[NSData data]];
    3. // oder
    5. [theMutableData setLength:0];

    Original von CalganX
    Aber scheinbar braucht man sich um Speichermanagement nicht wirklich zu kümmern.

    Oh doch, man muss sich um das Speichermanagement kümmern. Nur muss man es richtig tun.

    Michael