objective-c – stringByAppendingString导致系统内存耗尽应用程

这里的技巧是将循环的内容放在自己的autorelasepool中.

但我们先解决最大的问题.你应该在这里使用一个NSMutableString类,这将大大减少你需要创建的对象的数量.

我们将completeFile切换为NSMutableString,使用工厂方法构造,然后附加到它：

-(void) writeToFile: (NSString*)filePath withSeparator:(NSString*) fieldSep{
    NSMutableString* completeFile = [NSMutableString string];
    for(int i=0;i<[self numberOfRows];i++){
        printf("im at line number... %i of %in",[self numberOfRows]);
        for(int j=0;j<[self numberOfColumns];j++){
            [completeFile appendString:[self objectInRow:i column:j]];

            if(j<[self numberOfColumns]-1){
                //separator for all columns except last one
                completeFile appendString:fieldSep];
            }
        }
            [completeFile appendString:@"n"];
    }
    NSError *error = nil;
    [completeFile writeToFile:filePath atomically:NO
                     encoding:NSStringEncodingConversionAllowLossy error:&error];
    if(error){
        NSLog(@"Error writing file at %@n%@",[error localizedFailureReason]);
    }
}

但这留下了另一个问题.看到[self objectInRow：i column：j]？它(可能)仍然是一个自动释放的对象.那不会被清理干净.

我们可能已经使您的代码运行而不会崩溃,具体取决于数据的大小,但这是一个什么时候崩溃的问题.

要解决这个问题,我们需要引入自动释放池.我们每行和每列做一个.这可能看起来过多(事实上,在这种情况下,因为我们已经消除了外循环中的autoreleasepool使用),但自动释放池相当便宜.如果您正在对大量数据进行循环,那么这只是一种很好的做法.

您可以使用@autorelease块替换每个for块,例如：

for(int i=0;i<[self numberOfRows];i++){

附：

for(int i=0;i<[self numberOfRows];i++) @autoreleasepool {

这给了我们这段代码：

-(void) writeToFile: (NSString*)filePath withSeparator:(NSString*) fieldSep{
    NSMutableString* completeFile = [NSMutableString string];
    for(int i=0;i<[self numberOfRows];i++) @autoreleasepool {
        printf("im at line number... %i of %in",[self numberOfRows]);
        for(int j=0;j<[self numberOfColumns];j++) @autoreleasepool {
            [completeFile appendString:[self objectInRow:i column:j]];

            if(j<[self numberOfColumns]-1){
                //separator for all columns except last one
                completeFile appendString:fieldSep];
            }
        }
            [completeFile appendString:@"n"];
    }
    NSError *error = nil;
    [completeFile writeToFile:filePath atomically:NO
                     encoding:NSStringEncodingConversionAllowLossy error:&error];
    if(error){
        NSLog(@"Error writing file at %@n%@",[error localizedFailureReason]);
    }
}

不过最后一点.您在此处的错误检查不安全.未定义在成功时传入的错误指针会发生什么变化.

[completeFile writeToFile:filePath atomically:NO
                     encoding:NSStringEncodingConversionAllowLossy error:&error];
    if(error){
        NSLog(@"Error writing file at %@n%@",[error localizedFailureReason]);
    }

相反,你想要这个：

BOOL ok = [completeFile writeToFile:filePath atomically:NO
                     encoding:NSStringEncodingConversionAllowLossy error:&error];
    if(!ok){
        NSLog(@"Error writing file at %@n%@",[error localizedFailureReason]);
    }

那么,这应该做你想要的.