NSAutoreleasePool 在来自另一个线程的回调中的最佳实践

Question

我有一个 C++ 库，我想将它公开为一个 Objective-C 框架，因此它对 Objective-C 开发人员来说会更容易使用。在整理 C++ 库时，我遇到了一个在处理自动释放对象和线程方面的特殊问题。

这个库的一个特点是开发者可以注册一个“记录器”来接收通知消息作为来自库的回调。来自库的通知使用 C++ 类型并且是从另一个 (POSIX) 线程接收的，所以我创建了一个私有 C++ 包装类来处理这个问题：它接收回调，将 char* 参数转换为 NSString，并将其传递给到用户提供的 Objective-C 记录器实例。这一切都很好，看起来像这样：

// Is called from the C++ library from another posix thread
void ObjCLoggerWrapper::LogMessage(const char *message)
{
  NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init];
  // Pass string to the user-provided Objective-C instance called "Logger"
  [Logger logMessage:[[[NSString alloc] initWithUTF8String:message] autorelease]];
  [pool release];
}

作为用户回调的示例，我编写了这个简单的方法来收集用户类实例中 NSString 成员 m_text 中的所有日志记录（将在其他地方使用，但这无关紧要）。

-(void) logMessage: (NSString*)message
{
  @synchronized(self)
  {
    m_text = [m_text stringByAppendingFormat:@"%04d: %@\r\n", m_lineno++, message];
  }
}

到目前为止一切顺利。或者我是这么想的。但这是牛肉：

用户回调方法中所有自动释放的对象都将属于包装器的 NSAutoreleasePool，因此在回调完成时会被释放。

哎呀！这意味着我的 m_text 字符串，由 stringByAppendingFormat 消息隐式创建为自动释放对象，将在 logMessage 完成时释放并成为僵尸。下次访问时，代码会崩溃。当然，用户当然不会期望这一点。在意识到发生了什么之前，我自己不得不挠头几次。

所以我的问题是：当从另一个线程对用户代码进行回调时，我们应该如何处理自动释放的对象？

我看到了几种可能的选择。没有一个是完美的，谷歌也没有帮助（因此这个问题）。

1. 告诉用户“不要在你的回调代码中创建自动释放对象”。不好：此类对象通常是非自愿创建的，例如通过 stringByAppendingFormat 和大量其他框架方法。除了后来发生的难以调试的崩溃之外，没有任何警告。
2. 没有 NSAutoreleasePool。如果用户尝试创建自动释放的对象，缺少一个将导致警告。绝对不漂亮，但会以稳健的方式提醒用户注意问题。并且用户可以“只是”添加一个他自己的 NSAutoreleasePool 来解决这个问题。但再说一遍：不漂亮。
3. 没有 NSAutoreleasePool 并使用 performSelectorOnMainThread 在主线程上运行回调。任何新的自动释放对象都会在主线程的池中结束。我认为这是安全的，但欢迎 cmets - 例如，是否可以始终在主线程上执行回调？这种方法需要在包装器中进行更精细的编码，以避免线程死锁并等待结果，但到目前为止，这是我的首选。

明确一点：重写我自己的包装器是没有问题的。我的主要优先事项是为 Objective-C 框架的用户创建一个可以流畅无缝地工作的解决方案。谢谢！

Answer 1

这不是您的自动释放方法的问题。你的方法看起来不错。 问题是logMessage，因为所写的从根本上是有缺陷的。每当调用之间封闭的自动释放池耗尽时，它都会失败。主运行循环（模优化）在每次事件循环旋转时都会耗尽其自动释放池，因此在这种情况下，它会在那里失败。

几个 FWIW：

[Logger logMessage:[[[NSString alloc] initWithUTF8String:message] autorelease]];

可以写

[Logger logMessage:[NSString stringWithUTF8String:message]];

并且[pool drain] 比[pool release] 更受欢迎

Answer 2

如果不确切知道您的库是做什么的，就很难推荐一种方法。但是，调用 performSelectorOnMainThread 是一种相当安全的策略。鉴于您不希望日志消息需要立即执行任何操作，因此等待主线程执行您的回调应该没问题。