观察用于插入/删除的 NSMutableArray

类具有 NSMutableArray 类型的属性(和实例 var) ,其中包含合成访问器(通过 @property)。如果使用以下方法观察该数组:

[myObj addObserver:self forKeyPath:@"theArray" options:0 context:NULL];

然后像这样在数组中插入一个对象:

[myObj.theArray addObject:NSString.string];

观察到的 ValueForKeyPath... 通知是 没有发送的:

[[myObj mutableArrayValueForKey:@"theArray"] addObject:NSString.string];

这是因为 mutableArrayValueForKey返回一个代理对象,该对象负责通知观察者。

但是合成的访问器不应该自动返回这样一个代理对象吗?解决这个问题的正确方法是什么——我是否应该编写一个只调用 [super mutableArrayValueForKey...]的自定义访问器?

35414 次浏览

You need to wrap your addObject: call in willChangeValueForKey: and didChangeValueForKey: calls. As far as I know, there's no way for the NSMutableArray you're modifiying to know about any observers watching its owner.

I would not use willChangeValueForKey and didChangeValueForKey in this situation. For one, they're meant to indicate the value at that path has changed, not that values in a to-many relationship are changing. You would want to use willChange:valuesAtIndexes:forKey: instead, if you did it this way. Even so, using manual KVO notifications like this is bad encapsulation. A better way of doing it is defining a method addSomeObject: in the class that actually owns the array, which would include the manual KVO notifications. This way, outside methods that are adding objects to the array don't need to worry about handling the array owner's KVO as well, which wouldn't be very intuitive and could lead to unneeded code and possibly bugs if you start adding objects to the array from several places.

In this example I would actually continue to use mutableArrayValueForKey:. I'm not positive with mutable arrays, but I believe from reading the documentation that this method actually replaces the entire array with a new object, so if performance is a concern you'll also want to implement insertObject:in<Key>AtIndex: and removeObjectFrom<Key>AtIndex: in the class that owns the array.

But shouldn't the synthesized accessors automatically return such a proxy object?

No.

What's the proper way to work around this--should I write a custom accessor that just invokes [super mutableArrayValueForKey...]?

No. Implement the array accessors. When you call these, KVO will post the appropriate notifications automatically. So all you have to do is:

[myObject insertObject:newObject inTheArrayAtIndex:[myObject countOfTheArray]];

and the Right Thing will happen automatically.

For convenience, you can write an addTheArrayObject: accessor. This accessor would call one of the real array accessors described above:

- (void) addTheArrayObject:(NSObject *) newObject {
[self insertObject:newObject inTheArrayAtIndex:[self countOfTheArray]];
}

(You can and should fill in the proper class for the objects in the array, in place of NSObject.)

Then, instead of [myObject insertObject:…], you write [myObject addTheArrayObject:newObject].

Sadly, add<Key>Object: and its counterpart remove<Key>Object: are, last I checked, only recognized by KVO for set (as in NSSet) properties, not array properties, so you don't get free KVO notifications with them unless you implement them on top of accessors it does recognize. I filed a bug about this: x-radar://problem/6407437

I have a list of all the accessor selector formats on my blog.

Your own answer to your own question is almost right. Don't vend theArray externally. Instead, declare a different property, theMutableArray, corresponding to no instance variable, and write this accessor:

- (NSMutableArray*) theMutableArray {
return [self mutableArrayValueForKey:@"theArray"];
}

The result is that other objects can use thisObject.theMutableArray to make changes to the array, and these changes trigger KVO.

The other answers pointing out that efficiency is increased if you also implement insertObject:inTheArrayAtIndex: and removeObjectFromTheArrayAtIndex: are still correct. But there is no need for other objects to have to know about these or call them directly.

one solution is to use an NSArray and create it from scratch by inserting and removing, like

- (void)addSomeObject:(id)object {
self.myArray = [self.myArray arrayByAddingObject:object];
}


- (void)removeSomeObject:(id)object {
NSMutableArray * ma = [self.myArray mutableCopy];
[ma removeObject:object];
self.myArray = ma;
}

than you get the KVO and can compare old and new array

NOTE: self.myArray should not be nil, else arrayByAddingObject: results in nil too

Depending on the case, this might be the solution, and as NSArray is only storing pointers, this isn't much an overhead, unless you work with large arrays and frequent operations

when you just want to observe the count changed, you may use an aggregate key path:

[myObj addObserver:self forKeyPath:@"theArray.@count" options:0 context:NULL];

but be aware that any reordering in theArray will not fire.

If you don't need the setter you can also use the simpler form below, which has similar performance (same growth rate in my tests) and less boilerplate.

// Interface
@property (nonatomic, strong, readonly) NSMutableArray *items;


// Implementation
@synthesize items = _items;


- (NSMutableArray *)items
{
return [self mutableArrayValueForKey:@"items"];
}


// Somewhere else
[myObject.items insertObject:@"test"]; // Will result in KVO notifications for key "items"

This works because if the array accessors are not implemented and there is no setter for the key, mutableArrayValueForKey: will look for an instance variable with the name _<key> or <key>. If it finds one, the proxy will forward all messages to this object.

See these Apple docs, section "Accessor Search Pattern for Ordered Collections", #3.