后台处理导致 BAD ACCESS 崩溃？

Question

我在这里按照本指南进行后台处理：

http://evilrockhopper.com/2010/01/iphone-development-keeping-the-ui-responsive-and-a-background-thread-pattern/

并且唯一的1行进行后台处理的代码是：

sound = flite_text_to_wave([cleanString UTF8String], voice);

但由于某种原因，我的访问信号很差。

调试它表明它也在该行崩溃。这是我现在在该部分中的代码。请记住，其中大部分只是 sfoster 项目中的默认 Flite 内容，以前没有任何问题，当它全部在一起时，没有分成 3 个。

-(void)speakText:(NSString *)text //This is called by my app
{

    cleanString = [NSMutableString stringWithString:@""];
    if([text length] > 1)
    {
        int x = 0;
        while (x < [text length])
        {
            unichar ch = [text characterAtIndex:x];
            [cleanString appendFormat:@"%c", ch];
            x++;
        }
    }
    if(cleanString == nil)
    {   // string is empty
        cleanString = [NSMutableString stringWithString:@""];
    }
    //The next line i've put in from the link
    [self performSelectorInBackground:@selector(backgroundTextToSpeech) withObject:nil];

}

-(void)backgroundTextToSpeech {
    NSAutoreleasePool* pool = [[NSAutoreleasePool alloc] init];
//The following line is the one it crashes on
    sound = flite_text_to_wave([cleanString UTF8String], voice);
    [self performSelectorOnMainThread:@selector(backToForegroundTextToSpeech) withObject:nil waitUntilDone:YES];
    [pool release];
}


-(void)backToForegroundTextToSpeech {

    NSArray *filePaths = NSSearchPathForDirectoriesInDomains (NSDocumentDirectory, NSUserDomainMask, YES);
    NSString *recordingDirectory = [filePaths objectAtIndex: 0];
    // Pick a file name
    tempFilePath = [NSString stringWithFormat: @"%@/%s", recordingDirectory, "temp.wav"];
    // save wave to disk
    char *path; 
    path = (char*)[tempFilePath UTF8String];
    cst_wave_save_riff(sound, path);
    // Play the sound back.
    NSError *err;
    [audioPlayer stop];
    audioPlayer =  [[AVAudioPlayer alloc] initWithContentsOfURL:[NSURL fileURLWithPath:tempFilePath] error:&err];
    [audioPlayer setDelegate:self];
    [audioPlayer prepareToPlay];


}

任何想法我做错了什么/我可以采取什么不同的措施来阻止这发生了？

编辑：使用下面发布的一些代码更改调试器后的图片：

Answer 1

首先跳出来的是：

您的类需要保存 cleanString 的保留计数。

通常，这是通过保留（或复制，这通常更适合 NSString 和其他具体/不可变类型）属性来完成：

@interface MONSpeaker : NSObject
{
    NSString * cleanString;
}

@property (copy) NSString * cleanString;

@end

@implementation MONSpeaker

@synthesize cleanString;

/* ... */

- (void)dealloc
{
  [cleanString release], cleanString = nil;
  [super dealloc];
}

-(void)speakText:(NSString *)text // This is called by my app
{

    NSMutableString * str = [NSMutableString stringWithString:@""];
    if([text length] > 1)
    {
        int x = 0;
        while (x < [text length])
        {
            unichar ch = [text characterAtIndex:x];
            [str appendFormat:@"%c", ch];
            x++;
        }
    }
    if(str == nil) // why not check for nil at creation instead?
    { // string is empty
        str = [NSMutableString stringWithString:@""];
    }
    self.cleanString = str;
    // The next line i've put in from the link
    [self performSelectorInBackground:@selector(backgroundTextToSpeech) withObject:nil];

}

-(void)backgroundTextToSpeech {
    NSAutoreleasePool* pool = [[NSAutoreleasePool alloc] init];
// The following line is the one it crashes on
    sound = flite_text_to_wave([self.cleanString UTF8String], voice);
    [self performSelectorOnMainThread:@selector(backToForegroundTextToSpeech) withObject:nil waitUntilDone:YES];
    [pool release];
}

@end

好的 - 所以这不是 100% 线程安全的，但它是惯用的。

更能抵抗线程问题的变体会将字符串作为参数传递给 backgroundTextToSpeech:(NSString *)text。然后，backgroundTextToSpeech:(NSString *)text 将创建参数 text 的副本（当然，在销毁 pool 之前释放该副本）。

Answer 2

你不明白自动释放是如何工作的。您为 cleanString 变量分配一个自动释放的对象，然后在使用该值的后台开始一些处理。但是，当启动后台处理的方法将控制权返回到主运行循环时，存储在 cleanString 中的自动释放字符串将被释放，并且后台线程会陷入僵尸状态。

您可以使用 Grand Central Dispatch< 来简化代码/a>：

- (void) startProcessing {
    NSString *source = [NSString stringWithWhatever…];
    dispatch_queue_t targetQ = 
        dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0);
    dispatch_async(targetQ, ^{
        sound = flite_text_to_wave…;
        dispatch_async(dispatch_get_main_queue(), ^{
            [self speechProcessingDone];
        });
    });
}

优点是您不必维护自己的自动释放池，您不必添加额外的方法只是为了在后台执行一行并且该块将保留您的字符串，这样你就不会崩溃。

但你当然不应该在不知道发生了什么的情况下仅仅为了解决自动释放问题而使用 GCD。内存管理是基础，你必须100%确定你在做什么。

---

如果这超出了您的想象，请确保您了解 Cocoa 中的内存管理是如何工作的，例如通过阅读此 Objective-C斯科特·史蒂文森的教程。你必须这样做，没有其他办法。

然后回到代码，您将看到存储到 cleanString 中的可变字符串是自动释放的，这意味着它们将在您离开当前函数后不久被释放。在后台运行选择器后，退出当前函数，并且存储在 cleanString 中的字符串被释放。不久之后，后台线程到达以下行：

sound = flite_text_to_wave([cleanString UTF8String], voice);

但由于存储在 cleanString 中的对象已经被释放，因此会发生崩溃。解决方案很简单，就是保留该对象，直到使用完毕为止。您可以在运行后台线程之前保留它，并在后台线程结束时释放它。