在 Obj-C 初始化器期间管理引用计数的惯用方法是什么？

Question

我正在学习 Objective-C。在我的第一个重要项目中，我遇到了一个问题，即与默认初始化程序相比，如何最好地处理传递给初始化程序的资源。我的类有一个保留的资源，engine，它可以在创建后手动设置，或者在显式初始化期间设置，或者在默认初始化期间设置：

- (id)init {
    if ((self = [super init])) {
        id e = [[XorShiftEngine alloc] init];
        [self setEngine: e];
        [e release];
    }
    return self;
}

- (id)initWithEngine:(NSObject <RandEngine> *)e {
    if ((self = [super init]))
        [self setEngine: e];
    return self;
}

- (id)setEngine:(NSObject <RandEngine> *)newEngine {
    [newEngine retain];
    [engine release];
    engine = newEngine;
    // Some other stuff which needs to happen on changing the engine.
    return engine;
}

特别是默认初始化程序，对我来说看起来非常丑陋，交错代码与 self 相关，然后与成员相关，然后再次与 self 相关，然后再次与成员相关，并且命名该对象只是为了以后能够释放它。它还违反了指定的初始值设定项习惯用法。

有没有更惯用的方法来做到这一点，而不使用自动释放池？

Answer 1

您是在谈论 -init 中的 e 吗？我没有看到那里的问题......你创建一个对象，你使用它，你释放它。这是创建和使用对象的常见且正确的模式。不过，如果您对此不满意，您可以让 -init 将 nil 传递给 -initWithEngine: 并让 -initWithEngine: 创建如果没有提供引擎。

人们通常不会谈论“默认初始化程序”，而是谈论“指定初始化程序”。指定的初始化程序是所有其他初始化程序都调用的初始化程序——它通常是允许最多自定义的初始化程序。在本例中，-initWithEngine: 是您指定的初始值设定项。

Answer 2

- init
{
    self = [super init];
    if (self != nil) {
        // initialization here
    }
    return self;
}

如果您有“覆盖初始化程序”，它们应该遵循相同的模式：

- initWithBob:(Bob*)aBob
{
   self = [self init];
   if ( self != nil ) {
       ... deal with aBob here ...
   }
   return self;
}

这也是一个很好的例子，说明了为什么在一个类上避免使用多个初始化程序是明智的。跟踪可能会很痛苦，并且会使子类化变得更加乏味且容易出错。最好有一个初始值设定项和多个子类，或者有一个初始值设定项并允许在事后配置实例。

即，使用可以根据需要在 init 之后调用的 @property(retain) Bob* bob;，而不是 initWithBob:。将初始化器限制为所需状态是一个很好的经验法则。

---

假设您正在声明一个 Car 类，我会这样做：

@interface Car:NSObject
+ car;
+ carWithEngine:(Engine*)anEngine;
- initWithEngine:(Engine*)anEngine;
@end

实现应该是显而易见的； car 创建默认引擎并调用 alloc/initWithEngine:/autorelease。

这为您提供了一个初始值设定项，使子类化变得非常简单明显。

Answer 3

- (id)init {
    id e = [[XorShiftEngine alloc] init];
    self = [self initWithEngine: e];
    [e release];
    return self;
}

// designated initializer
- (id)initWithEngine:(NSObject <RandEngine> *)e {
    self = [super init];
    if (self != nil) {
        engine = [e retain];
        // engine initialization stuff
    }
    return self;
}

- (id)setEngine:(NSObject <RandEngine> *)newEngine {
    [newEngine retain];
    [engine release];
    engine = newEngine;
    // Some other stuff which needs to happen on changing the engine.
    return engine;
}

你基本上做了正确的事。我唯一要改变的是直接分配变量并保留它，而不是在 init 方法中调用 setter。

Apple 建议不要在 init 和 dealloc 方法中调用属性 setter/getter（我假设您已经声明了保留属性并且只是覆盖了 setter）。

如果您有引擎自定义代码，需要在第一次在初始化程序中设置引擎以及每次更改引擎时发生，那么您应该将其重构为一个单独的方法。