可以不在自定义 init 方法中调用 [super init] 吗？

Question

我有一个MKPolyline subblas，我想实现NSCoding，即

@interface RSRoutePolyline : MKPolyline <NSCoding>

I asked a question on the best way to encode the c-array and got an excellent answer。但是，MKPolyline 上没有定义 init 方法，即除了其类方法 polylineWithPoints:points 之外，没有其他方法可以为其提供数据。

这段代码我的评论可以吗？

- (void)encodeWithCoder:(NSCoder *)aCoder
{
    MKMapPoint *points = self.points;
    NSUInteger pointCount = self.pointCount;

    NSData *pointData = [NSData dataWithBytes:points length:pointCount * sizeof(MKMapPoint)];
    [aCoder encodeObject:pointData forKey:@"points"];
    [aCoder encodeInteger:pointCount forKey:@"pointCount"];
}

- (id)initWithCoder:(NSCoder *)aDecoder
{
    NSData* pointData = [aDecoder decodeObjectForKey:@"points"];
    NSUInteger pointCount = [aDecoder decodeIntegerForKey:@"pointCount"];

    // Edit here from @ughoavgfhw's comment
    MKMapPoint* points = (MKMapPoint*)[pointData bytes];

    // Is this line ok?
    self = (RSRoutePolyline*)[MKPolyline polylineWithPoints:points count:pointCount];

    return self;
}

Answer 1

你应该在 NSObject 的任何子类上调用一个 init 方法。由于 MKPolyline 是一个 NSObject，你应该初始化它。

但是 MKPolyline 没有方法，也没有 init。这是Objective C的意思是告诉你你不能继承它。

相反，正如 WDUK 建议的那样，定义您自己的类。它跟踪您的列表点，并管理 NSCoding 以根据需要保存和恢复它们。

 @interface RSPolyline: NSObject<NSCoding>

 - (id) initWithPoints: (NSArray*) points;
 - (id) initWithCoder:(NSCoder *)aDecoder;
 - (void) encodeWithCoder:(NSCoder *)aCoder;

 - (MKPolyline*) polyLine;

 @end

您的类可以根据请求生成折线，如果性能有问题，可能会缓存结果。

作为一项规则，不要先进行继承。当你想扩展和改进一门课时，首先要考虑作文。

Answer 2

不打电话给[super init] 很脏，这对我的良好编程理念来说并不是什么好兆头。如果不自己调用 super ，它就不是真正的子类；只是依赖于调用便利构造函数的副作用的组合的混蛋。这么说，我相信您所描述的方法可以正常工作，但它违背了良好的 Objective-C 编程及其约定。

我的建议是使用MKPolyLine 作为MKPolyLine 实例，并使用一个类别来添加您需要的额外花里胡哨。至于添加额外的实例变量等，您可以使用关联对象。这个概念的介绍可以是found here，这个SO问题解决了它们与类别的使用：How do I use objc_setAssociatedObject/objc_getAssociatedObject inside an object?

Answer 3

虽然通常允许在 init 方法中创建和返回不同的对象，但该行存在三个问题（如下所述）。相反，我建议重写points 和pointCount 属性，以便您可以返回存储在实例变量中的值，并在实例变量为空时调用超级实现。然后，您的初始化程序只需设置这些实例变量，以便使用它们。

- (MKMapPoint *)points {
    if(myPointsIvar == NULL) return [super points];
    else return myPointsIvar;
}
// similarly for pointCount

---

第一个问题是您正在创建一个新对象，但没有释放旧对象，这意味着您正在泄漏它。您应该将结果存储在不同的变量中，然后释放 self，然后返回结果（您不需要将其存储在 self 中）。

其次，polylineWithPoints:count: 返回一个自动释放的对象，但initWithCoder: 应该返回一个保留的对象。除非它上面有另一个保留，否则它可能会在您仍在使用它时被释放。

如果这些是唯一的问题，您可以像这样解决这两个问题：

MKPolyline *result = [MKPolyline polylineWithPoints:points count:pointCount];
[self release];
return [result retain];

但是，还有第三个问题不能这么容易解决。 polylineWithPoints:count: 不返回 RSRoutePolyline 对象，并且它返回的对象可能与您的子类的方法不兼容（例如，它可能不支持 NSCoding）。真的没有办法解决这个问题，所以你不能使用polylineWithPoints:count:。