stringByAppendingString 导致系统内存不足的应用程序

Question

当我在“大”（37000 行）文本文件上运行以下代码时，为什么我的系统告诉我应用程序的内存不足？

-(void) writeToFile: (NSString*)filePath withSeparator:(NSString*) fieldSep{
NSString* completeFile = [[[NSString alloc] initWithString:@""] autorelease];
for(int i=0;i<[self numberOfRows];i++){
    printf("im at line number... %i of %i\n",i,[self numberOfRows]);
    for(int j=0;j<[self numberOfColumns];j++){
        completeFile = [completeFile stringByAppendingString:[self objectInRow:i column:j]];

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

}

出于调试原因，我添加了 printf，前 4000 行似乎立即发生，然后慢慢变慢...我的文件包含超过 37000 行类似于这些：

1893-11-6   136 194 165

Answer 1

当您使用工厂方法分配对象时，这些对象会被添加到自动释放池中。仅当您的事件循环运行时，在您的 IBAction 返回之后，自动释放池才会被耗尽。

这里的诀窍是将循环的内容放在它自己的自动释放池中。

但让我们先解决最大的问题。您应该在此处使用一个 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 %i\n",i,[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%@",
              filePath, [error localizedFailureReason]);
    }
}

不过，这会带来另一个问题。看到[self objectInRow:i column:j]了吗？它仍然（可能）是一个自动释放的对象。这不会得到清理。

根据数据的大小，我们可能使您的代码运行时不会崩溃，但这是何时崩溃而不是如果的问题。

为了解决这个问题，我们需要引入自动释放池。让我们每行和每列做一个。这可能看起来有点过分（事实上，在这种情况下，因为我们已经在外部循环中消除了 autoreleasepool 的使用），但 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 %i\n",i,[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%@",
              filePath, [error localizedFailureReason]);
    }
}

不过，最后一点。您在这里的错误检查是不安全的。 成功时这样传入的错误指针会发生什么情况没有定义。

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

相反，你想要这个：

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

那么，这应该做你想做的。

Answer 2

问题是每次调用stringByAppendingString: 都会创建一个新的自动释放NSString 对象。但是，由于该方法在循环中继续，因此 autorelease 没有机会释放这些对象。

您可以通过在内部循环周围添加一个自动释放池来解决它，如下所示：

for(int i=0;i<[self numberOfRows];i++){
    printf("im at line number... %i of %i\n",i,[self numberOfRows]);
    @autoreleasepool {
        for(int j=0;j<[self numberOfColumns];j++){
            completeFile = [completeFile stringByAppendingString:[self objectInRow:i column:j]];

            if(j<[self numberOfColumns]-1){
                //separator for all columns except last one
                completeFile = [completeFile stringByAppendingString:fieldSep];
            }
        }
    }
        completeFile = [completeFile stringByAppendingString:@"\n"];
}

Answer 3

您应该使用NSMutableString 附加这些字符串。