Question

我正在iOS上开发基于联系人的应用程序。

基本上我使用NSMutableArray进行人员管理。

这是我的代码。

此代码有时会发生崩溃。

如果没有崩溃，那么我可以在调试窗口上显示 allPeople 计数。

当我遇到崩溃时，我只能看到 allPeople 的内存地址。

似乎 allPeople 已经被释放。

我的错误是什么？

谢谢。

@property (nonatomic, strong) NSMutableArray *allPeople_;

    - (void) C {


     ABAddressBookRequestAccessWithCompletion(addressbookRef, ^(bool granted, CFErrorRef error) {
                if (!granted) {
                    return;
                }                              

 dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_BACKGROUND, 0), ^{
            [self A];
            [self B];
        });
    }
- (void)A {

@autoreleasepool {

        NSMutableArray *people = [NSMutableArray arrayWithArray:_allPeople_];
        //do something

        [arrayLock lock];
        _allPeople_ = people;
        [arrayLock unlock];
    }
}

- (void)B {

        //Here is crash location.            
        NSMutableArray *people = [NSMutableArray arrayWithArray:_allPeople_];
        //do something

        [arrayLock lock];
        _allPeople_ = people;
        [arrayLock unlock];
}

这是崩溃日志。

Fatal Exception: NSRangeException
*** -[__NSArrayM getObjects:range:]: range {0, 3009} extends beyond bounds for empty array

Fatal Exception: NSRangeException
0  CoreFoundation                 0x18d57afe0 __exceptionPreprocess
1  libobjc.A.dylib                0x18bfdc538 objc_exception_throw
2  CoreFoundation                 0x18d45d13c -[__NSArrayM getObjects:range:]
3  CoreFoundation                 0x18d45cc80 -[NSArray initWithArray:range:copyItems:]
4  CoreFoundation                 0x18d45cb74 +[NSArray arrayWithArray:]
5  App                           0x1001f92b4 -[AddressCollector C]

Answer 1

尝试复制数组：_allPeople_ = [people copy];

我不确定是否需要自动释放池，内存由操作系统自动管理。

您可以致电

，而不是锁定

@synchronized(_allPeople_) {

}

Answer 2

好吧，我觉得你已经发布了答案： [__NSArrayM getObjects：range：]：range {0,3009}超出了空数组的边界

您正在尝试访问范围远远超出它的空数组:)。很可能已经通过ARC管理了内存。

建议：

如果您在主线程上使用此代码，请不要使用锁定，解锁只要。这没用。
如果你使用多个线程（你没有显示它与你的代码，也没有评论）使用原子的属性。你呢不需要锁定，解锁（你有非原子的）
仅使用autoreleasepool 当你创造许多短命的物体时。（例如很多 NSString的字符串）。请记住，您需要使用多线程保护数据。
对于后台处理，将更新移动到主线程是有益的，如下所示： dispatch_async(dispatch_get_main_queue(), ^{ self._allPeople_ = processedAllPeople; //update UI });

您不需要同步，因为所有更新都将在同一个线程上。 UI只能在主线程上更新。
制作数据副本。你需要深层复制。

如何管理NSMutableArray内存

2 个答案: