Est-ce la bonne façon de créer/détruire une chaîne en boucle?

Question

Juste curieux si c'est le moyen de le faire, je veux juste m'assurer qu'il ne fuit pas, même si je pense que je ne fais que modifier le contenu de la chaîne.

NSMutableString *newPath = [[NSMutableString alloc] init]; 

for(fileName in [manager enumeratorAtPath:rootPath]){ 
    if ([[fileName pathExtension] isEqual:@"exr"]) { 
     [fileArray addObject:fileName]; 

     // THIS BIT 
     [newPath setString:rootPath]; 
     [newPath appendString:@"/"]; 
     [newPath appendString:fileName]; 
     // HERE 
     attrDir = [manager attributesOfItemAtPath:newPath error:&myError]; 

     fileSize = [attrDir objectForKey: @"NSFileSize"]; 
     NSLog(@"File: /%@ Size: %@", fileName, fileSize); 
    } 
} 
[newPath release]; 

gary

Answer 1

Cela semble bien sage fuite. Si vous utilisez Xcode 3.2, vous pouvez Construire-> Build & Analyzer pour que Clang vérifie ce genre de chose. Rappelez-vous que vous n'avez qu'à publier les éléments que vous avez alloués, nouveaux, copiés ou conservés.

Envisagez d'utiliser stringByAppendingPathComponent, plutôt que de coder en dur le séparateur de chemin @"/". NSString a un number of methods comme ceci spécifiquement pour travailler avec des chemins.

NSString* fullPath = [rootPath stringByAppendingPathComponent:fileName]; 

Answer 2

Il n'y a rien de mal à cela, mais il pourrait être préférable d'utiliser initWithFormat et libérer:

NSString *newPath = [[NSString alloc] initWithFormat:@"%@/%@",rootPath,fileName]; 

// do your thing 

[newPath release]; 

Answer 3

Il n'y a absolument rien de mal avec votre code, il est la gestion de la mémoire correcte.

Mais il peut être fait avec encore moins de code et gestion de la mémoire nécessaire:

for(fileName in [manager enumeratorAtPath:rootPath]){ 
    if ([[fileName pathExtension] isEqualToString:@"exr"]) { 
    [fileArray addObject:fileName]; 

    NSString* newPath = [rootPath stringByAppendingPathComponent:fileName]; 
    attrDir = [manager attributesOfItemAtPath:newPath error:&myError]; 

    fileSize = [attrDir objectForKey: @"NSFileSize"]; 
    NSLog(@"File: /%@ Size: %@", fileName, fileSize); 
    } 
}