Objective C - 自定义视图和实现 init 方法？

Question

我有一个自定义视图，我希望能够在代码内和笔尖中初始化。

编写 initWithFrame 和 initWithCoder 方法的正确方法是什么？ 它们都共享一个用于某些初始化的代码块。

Answer 1

在这种情况下，正确的做法是创建另一个方法，其中包含 -initWithFrame: 和 -initWithCoder: 通用的代码，然后从 -initWithFrame: 和 -initWithCoder: 调用该方法。 code>-initWithFrame: 和 -initWithCoder:：

- (void)commonInit
{
    // do any initialization that's common to both -initWithFrame:
    // and -initWithCoder: in this method
}

- (id)initWithFrame:(CGRect)aRect
{
    if ((self = [super initWithFrame:aRect])) {
        [self commonInit];
    }
    return self;
}

- (id)initWithCoder:(NSCoder*)coder
{
    if ((self = [super initWithCoder:coder])) {
        [self commonInit];
    }
    return self;
}

请注意 Justin 的回答中概述的问题，特别是任何子类不得覆盖 -commonInit。我在这里使用这个名称是因为它具有说明性的价值，但您可能会想要一个与您的类联系更紧密并且不太可能被意外覆盖的名称。如果您正在创建一个专门构建的 UIView 子类，而该子类本身不太可能被子类化，那么使用上面的通用初始化方法就完全没问题了。如果您正在编写一个框架供其他人使用，或者如果您不理解该问题但想做最安全的事情，请改用静态函数。

Answer 2

解决方案并不像最初看起来那么简单。初始化存在一些危险 - 下面将详细介绍这些危险。出于这些原因，我通常在 objc 程序中采用以下两种方法之一：

对于平凡的情况，重复不是一个坏策略：

- (id)initOne
{
    self = [super init];
    if (nil != self) { monIntIvar = SomeDefaultValue; }
    return self;
}

- (id)initTwo
{
    self = [super init];
    if (nil != self) { monIntIvar = SomeDefaultValue; }
    return self;
}

对于不平凡的情况，我建议采用以下一般形式的静态初始化函数：

// MONView.h

@interface MONView : UIView
{
    MONIvar * ivar;
}

@end

// MONView.m

static inline bool InitMONView(MONIvar** ivar) {
    *ivar = [MONIvar new];
    return nil != *ivar;
}

@implementation MONView

- (id)initWithFrame:(CGRect)frame
{
    self = [super initWithFrame:frame];
    if (nil != self) {
        if (!InitMONView(&ivar)) {
            [self release];
            return nil;
        }
    }
    return self;
}

- (id)initWithCoder:(NSCoder *)coder
{
    self = [super initWithCoder:coder];
    if (nil != self) {
        if (!InitMONView(&ivar)) {
            [self release];
            return nil;
        }
    }
    return self;
}

// …

@end

Objective-C++ ：

如果您使用的是 objc++，那么您可以简单地为您的 c++ ivars 实现合适的默认构造函数，并省略大部分初始化和 dealloc 脚手架（假设您已正确启用编译器标志）。

更新：

我将解释为什么这是初始化对象的安全方法，并概述为什么其他答案的典型实现是危险的一些原因。

在初始化期间调用实例方法的常见初始化程序的典型问题是它们滥用继承图，通常会引入复杂性和错误。

建议：在部分构造的对象上调用重写的实例方法（例如在初始化和释放期间）是不安全的，应避免。访问器特别糟糕。在其他语言中，这是程序员的错误（例如UB）。查看有关该主题的 objc 文档（参考：“实现初始化程序”）。我认为这是必须的，但我仍然知道有人坚持认为实例方法和访问器在部分构造的状态下更好，因为它“通常适合他们”。

要求：尊重继承图。从基础向上初始化。从上到下破坏。总是。

建议：保持所有初始化一致。如果你的基地从 init 返回一些东西，你应该假设一切都很好。不要为您的客户端和子类引入脆弱的初始化舞蹈（它可能会作为错误返回）。您需要知道您是否保留了有效的实例。此外，当您从指定的初始化器返回对象时，子类将（正确地）假设您的基类已正确初始化。您可以通过将基类的 ivars 设为私有来降低出现这种情况的可能性。一旦从 init 返回，客户端/子类就会假设它们派生的对象是可用的并且已正确初始化。随着类图的增长，情况变得非常复杂，错误开始蔓延。

建议：检查 init.c 中是否有错误。还保持错误处理和检测的一致性。返回 nil 是确定初始化期间是否出现错误的明显约定。及早发现。

好的，但是共享实例方法怎么样？

exmaple 从另一篇文章借用并更改：（

@implementation MONDragon

- (void)commonInit
{
    ivar = [MONIvar new];
}

- (id)initWithFrame:(CGRect)aRect
{
        if ((self = [super initWithFrame:aRect])) {
                [self commonInit];
        }
        return self;
}

- (id)initWithCoder:(NSCoder*)coder
{
        if ((self = [super initWithCoder:coder])) {
                [self commonInit];
        }
        return self;
}

// …

顺便说一句，该示例中没有错误处理）

Caleb：我在上面的代码中看到的最大“危险”是有人可能创建相关类的子类，覆盖 -commonInit，并可能两次初始化该对象。

具体来说，子类 -[MONDragon commonInit] 将被调用两次（泄漏资源，因为它们将被创建两次）并且基类的初始化程序和错误处理将不会执行。

Caleb：如果这是一个真正的风险……

这两种影响都可能等同于一个不可靠的程序。使用传统的初始化可以轻松避免该问题。

Caleb：……处理它的最简单方法是保持 -commonInit 私有和/或将其记录为不可覆盖的内容

因为运行时在消息传递时不区分可见性，这种方法很危险，因为任何子类都可以轻松声明相同的私有初始化方法（见下文）。

将方法记录为不应重写的方法会增加子类化者的负担，并引入可以通过使用其他方法轻松避免的复杂性和问题。它也很容易出错，因为编译器不会标记它。

如果坚持使用实例方法，您保留的约定，例如 -[MONDragon ConstructionMONDragon] 和 -[MONKomodo ConstructionMONKomodo] 可以显着减少大多数情况下的错误案例。初始化程序可能仅对类实现的 TU 可见，因此编译器可以标记一些潜在的错误。

旁注：一个常见的对象构造函数，例如：（

- (void)commonInit
{
    [super commonInit];
    // init this instance here
}

我也见过）甚至更糟，因为它限制初始化，删除上下文（例如参数），并且最终仍然会导致人们在指定的初始化器和指定的初始化器之间跨类混合初始化代码-commonInit。

通过这一切，由于普遍的误解和愚蠢的错误/疏忽而浪费了大量时间来调试上述所有问题，我得出的结论是，当您需要为类实现公共初始化时，静态函数是最容易理解和维护的。类应该使它们的客户端免受危险，这是“通过实例方法的通用初始化程序”多次失败的问题。

它不是基于指定方法的 OP 中的选项，但作为一般注意事项：您通常可以使用便利构造函数更轻松地合并公共初始化。这对于在处理类簇、可能返回特化的类以及可能选择从多个内部初始化器中进行选择的实现时最小化复杂性特别有用。

Answer 3

如果他们共享代码，只需让他们调用第三个初始化方法即可。

例如，initWithFrame 可能看起来像这样：

- (id)initWithFrame:(CGRect)frame {
    if ((self = [super initWithFrame:frame])) {
        [self doMyInitStuff];
    }

    return self;
}

请注意，如果您使用的是 OS X（而不是 iOS），则框架将为 NSRect 而不是 CGRect< /代码>。

如果您需要进行错误检查，请让您的初始化方法返回如下错误状态：

- (id)initWithFrame:(CGRect)frame {
    if ((self = [super initWithFrame:frame])) {
        if (![self doMyInitStuff]) {
            [self release];
            self = nil;
        }
    }

    return self;
}

假设 doMyInitStuff 方法在出错时返回 NO。

另外，如果您还没有看过，有一些关于初始化的文档可能对您有用（尽管它没有直接解决这个问题）：

Cocoa 编码指南：框架开发人员的提示和技术