루프에서 문자열을 작성/삭제하는 올바른 방법입니까?

Question

이 작업을 수행하는 방법이 궁금한 경우 그냥 누수가 없는지 확인하고 싶지만 문자열 내용 만 수정한다고 생각합니다.

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]; 

게리가

Answer 1

이 누출 현명한 잘 보인다. Xcode 3.2를 실행중인 경우 Build-> Build & Analyzer를 사용하여 Clang이 이런 종류의 것을 검사하도록 할 수 있습니다.

할당 한 항목을 릴리스하거나 새로 작성하거나 복사하거나 보유해야한다는 것을 기억하십시오.

@"/" 경로 구분 기호를 하드 코딩하는 대신 stringByAppendingPathComponent을 사용하는 것이 좋습니다. NSString에는 number of methods이 있습니다. 특별히 경로 작업에 사용됩니다. 코드 아무 문제가 전혀 없다

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

// do your thing 

[newPath release]; 

Answer 2

, 그것은 아무 문제가 없습니다.

그러나 필요도 적은 코드 및 메모리 관리를 수행 할 수 있습니다

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); 
    } 
} 

Answer 3

, 그것은 올바른 메모리 관리 : initWithFormat를 사용하고 해제하는 것이 더있을 수 있지만

NSString* fullPath = [rootPath stringByAppendingPathComponent:fileName];