Potentielle Speicherlöcher

  • Potentielle Speicherlöcher

    Guten Morgen.

    Ich habe letztens den Analyze Befehl ausgeführt und bin dann aus allen Wolken gefallen. Die App funktioniert ansich ja ohne Probleme nur möchte ich trotzdem die ganzen Warnungen (oder was immer das ist) loswerden, falls das möglich ist.

    Vielleicht habt ihr ja ein paar Tipps für mich.

    Hier ist so ein Fall:

    Quellcode

    1. -(NSString*) getWord: (NSString*) word
    2. {
    3. NSMutableString *wordToConvert = [[NSMutableString alloc] initWithString:word];
    5. NSArray *arraySonderZeichen = [NSArray arrayWithObjects: @"ae", @"oe", @"ue", @"Ae" ,@"Oe", @"Ue", nil];
    7. for (int i=0 ; i< ([arraySonderZeichen count]) ; i++)
    8. {
    9. NSRange substr = [wordToConvert rangeOfString: [arraySonderZeichen objectAtIndex:i]];
    10. if (substr.location != NSNotFound)
    11. {
    12. if (substr.location == 0)
    13. {
    14. switch (i) { .....}
    16. }
    17. else
    18. {
    19. NSString *substr2 = [wordToConvert substringWithRange:NSMakeRange((substr.location-1), 1)];
    20. if ((substr2 != @"a") || (substr2 != @"i")|| (substr2 != @"o")|| (substr2 != @"u"))
    21. {
    22. switch (i) {
    23. case 0:
    24. [wordToConvert replaceCharactersInRange:substr withString:@"ä"];
    25. break;
    26. case 1:
    27. [wordToConvert replaceCharactersInRange:substr withString:@"ö"];
    28. break;
    29. case 2:
    30. [wordToConvert replaceCharactersInRange:substr withString:@"ü"];
    31. break;
    32. case 3:
    33. [wordToConvert replaceCharactersInRange:substr withString:@"Ä"];
    34. break;
    35. case 4:
    36. [wordToConvert replaceCharactersInRange:substr withString:@"Ö"];
    37. break;
    38. case 5:
    39. [wordToConvert replaceCharactersInRange:substr withString:@"Ü"];
    40. break;
    41. default:
    42. break;
    43. }
    44. }
    45. }
    46. }
    47. }
    48. return wordToConvert;
    Alles anzeigen



    Und zwar löst hier die Initialisierung(NSMutableString *wordToConvert = [[NSMutableString alloc] initWithString:word];) das Leck aus. Ist mir ja irgendwie klar nur da es mein Rückgabewert ist kann Ichs ja nicht releasen und da ich es bearbeiten muss kann ich keinen Zeiger verwenden.
    Das ganze in ähnlicher Form existiert ein paar mal in meinem Projekt.

    Bitte keine zu komplizierten Antwort bin Newbe (schreibt man das so?).

    mfg
    joe

  • ist das ganze nicht etwas kompliziert?

    warum verwendest du nicht einfach die replace-funktion von NSMutableString und rufst die 6 mal auf. fertig.

    zum eigentlichen problem: du sollt alloc-init vermeiden und convenience allocator verwenden. in diesem fall "stringWithString".

    oder eben wie gesagt ein autorelease senden.

    btw: der methondenname ist auch nicht korrekt so!

  • Hallo zusammen.

    Danke für eure Antworten. Autorelease hat das Loch gestopft. Danke vielmals. Und gritsch danke für den Tipp mit dem replace (Warum einfach wenn es auch kompliziert geht :) ) und der Rückgabewert muss natürlich NSMutableString sein. Habs gerade geändert.

    Mfg
    Joe