Objective-C自定义属性getter无限递归



我有一个Objective-C类,它实现了数据树的节点。它的属性对公共是只读的,而类的私有扩展(此处未显示)实现了属性的setter,因此管理器类可以创建树。

// Interface
@interface DataSet : NSObject {
    NSString        *name;
    NSString        *data;
@private
    DataSet         *parent;
    NSMutableArray  *children;
}
@property (nonatomic, readonly, copy) NSString *name;
@property (nonatomic, readonly, copy) NSString *data;

我想为其中一个属性实现一个自定义getter,如果该属性为nil,它将在树上遍历,直到找到具有该属性的非nil值的祖先节点。

我的问题是在不引起getter调用本身的无限递归的情况下实现getter。

// Implementation
@interface DataSet ()
@property (nonatomic, retain) DataSet           *parent;
@property (nonatomic, retain) NSMutableArray    *children;
@end
@implementation DataSet
@synthesize name;
// do not @synthesize data
@synthesize parent, children;
// custom getter walks up tree to find first non-nil 'data' property
- (NSString*) data {
    NSString *result = nil;
    DataSet *set = self;
    while (set != nil && result == nil) {
        result = [set data];    // <=== INFINITE RECURSION HERE
        set = set.parent;
    }
    return result;
}

我搜索了这个论坛和其他论坛,但没有找到任何我想在这里做什么的例子。有人有什么建议吗?

此外,getter中的最后一行应该是吗

return [result copy];

嗯,我想你想要这样的东西:

-(NSString *) data {
    // Determine result from current instance data.
    NSString *result = ....; 
    // If nothing, ask parent instance of this instance.
    if (result == nil) {
        result = [parent data];
    }
    // Might still be nil if parent returns nothing.
    return result;
}

嗯,实际上,当你有一个包含一些文本数据的data变量时,它可以这样做:

-(NSString *) data {
    // If data is nil, ask parent instance for a value, otherwise return a copy.
    return data == nil ? [parent data] : [data copy];
}

因此,数据集的每个实例都不需要有一个循环。他们所做的只是与直系亲属核实。这样,如果你有一个a->B->C->D的数据图,并执行[D data];,D将检查自己,然后作为C检查自己,再作为B检查自己,并询问a。你将返回第一个成功的非零值。

只需直接访问ivar:

// custom getter walks up tree to find first non-nil 'data' property
- (NSString*) data {
    NSString *result = nil;
    DataSet *set = self;
    while (set != nil && result == nil) {
        result = set->data;
        set = set->parent;
    }
    return [result copy];
}

这样可以避免调用属性访问器,从而避免递归。

是的,最后一行应该是return [result copy];,因为您声明您的属性具有copy属性。如果没有用copy属性声明它,那么就不会返回副本。

解决了它。

我可以在对象的父级上递归调用自定义getter(但不能在对象本身上调用)。这不是问题所在。

关键是当我释放对象时,不要在dealloc中使用自定义getter,否则一些"数据"对象将被多次返回(从而释放)。

// custom getter - if data is nil on this object, 
// find the first non-nil value in its list of ancestors
- (NSString*) data {
    NSString *result = data;
    if (result == nil && parent != nil)
        result = [parent data];
    return result;
}
- (void) dealloc {
    NSInteger count = [children count];
    for (NSInteger index = count - 1; index >= 0; --index) {
        DataSet *child = [children objectAtIndex:index];
        [children removeLastObject];
        [child release];
    }
    [children release];
    // DON'T call custom getter if data is nil
    // or we'll get one of its ancestors' data object, release it,
    // then later release it again when releasing the ancestor
    if (data != nil)
        [self.data release];
    [self.name release];
    [super dealloc];
}

最新更新