用于自定义类的 Objective-C 多重嵌套 initWith

Question

我是 Objective-C 的新手，我有一个问题。

我创建了一个自定义类并尝试为初始化创建重载：

- (id)init
{
    if (self = [super init]) {
        [self setIsCurrentCar:NO];
    }
    return self;
}

-(id) initWithID:(NSInteger)id {
    if(self = [self init]) {
        [self setID:id];
    }
    return self;
}

-(id) initWithID:(NSInteger)id CarYear:(NSString *)year {
    if(self = [self initWithID:id]) {
        [self setCarYear:year];
    }
    return self;
}

假设有一次，我调用了-(id) initWithIDCarYear 方法。

我想知道上面的代码在结构上是正确的。

• 在此代码中，self 设置了 3 次。有没有更好的 解决方案？
• 此代码是否存在内存泄漏？ （使用 ARC）
• 我是否必须始终检查if(self = ...) 或者它是 冗余代码？

谢谢

@编辑 下面的代码更好吗？

-(id) initWithID:(NSInteger)id CarYear:(NSString *)year {
    if (self = [super init]) {
        [self setIsCurrentCar:NO];
        [self setID:id];
        [self setCarYear:year];
    }
    return self;
}

Answer 1

虽然您的代码没问题，但我会以相反的顺序构造 init 调用，其中最详细的一个是指定的初始化程序，而更一般的调用会冒泡一些默认值：

-(id) initWithID:(NSInteger)id 
         CarYear:(NSString *)year 
{
    if(self = [super init]) {
        _year = year;
        _id = id;
    }
    return self;
}

-(id)initWithID:(NSInteger)id 
{
    return [self initWithID:id CarYear:@"unset"];
}

-(id)init 
{
    return [self initWithID:0];
}

如果调用更通用的初始化程序之一会产生非法状态，您可以改为抛出错误以禁止使用它。

让我们假设，汽车需要有一个 ID，但不是一年。使用initWithID 可以，但是使用init 会导致状态不一致，所以我们要强制不要使用它：

-(id)init 
{
    [NSException raise:NSInternalInconsistencyException 
                format:@"You must use -initWithID: or -initWithID:CarYear:", NSStringFromSelector(_cmd)];
    return nil;
}

---

• 在此代码中，self 设置为 3 次。有没有更好的解决方案？

见上

• 此代码是否存在内存泄漏？ （使用 ARC）

没有，一切都很好

• 是否必须始终检查 if(self = ...) 或者它是冗余代码？

正如我向您展示的那样：您可以在链中调用不同的 init 方法。只有该链中的最后一个需要执行该操作。

---

-(id) initWithID:(NSInteger)id CarYear:(NSString *)year {
    if (self = [super init]) {
        [self setIsCurrentCar:NO];
        [self setID:id];
        [self setCarYear:year];
    }
    return self;
}

您不应在 init 方法中对 self 使用 setter，see Apple's docs。

Answer 2

我想知道上面的代码在结构上是正确的。

是的。我看不出有什么问题。

在这段代码中，self 被设置了 3 次。有没有更好的解决方案？

这很正常。我不会费心去改变它。

这段代码有内存泄漏吗？ （使用 ARC）

没有。

我是否必须始终检查 if (self = ...) 或者它是多余的 代码？

您不必必须，但绝对应该。详情请见this question。

Answer 3

看起来您有一个强制初始化变量和两个实际上是可选的。

我建议为 ID 和 carYear 实现一个 init 方法和两个 @property()。这减少了初始化器的数量并更好地反映了类的使用约定。

Answer 4

据我所知...tried to create overloads for Initialization 此处不应使用此语句。

通常overload 表示同名多个参数，但在 obj-c 中我们不遵循这一点。在 obj-c 中，通过命名参数来伪造重载。

所以这里有 3 组不同的代码，每组都称为其他方法。而且您没有内存泄漏，因为您没有为同一个对象分配三次内存，而是在初始化它。