记一次PHP与自动释放池的内存释放的坑

使用多线程遍历或操作集合对象的时候，务必要注意内存的及时释放，否则一不小心就触发内存峰值而闪退了。

最近新版本发布后，有较多用户反馈闪退；经过部分走访用户初步得出结论是内存闪退，而且和PHPLibrary相关修改有关；

联系用户

联系了部分用户，捞取到较多日志；确定原因基本是内存闪退，发生闪退时浏览器占用了1.3G左右的内存，超过了内存限制而被jetsam杀死；这个不太合理，我们基本不会这样疯狂申请这么大内存而不释放的。

最后发现用户都有一个共性，相册图片都很多。而恰好我们在新版本有个较大的修改为了适配iOS12而全面废弃了ALAssetLibrary，所以怀疑是这里。

分析

给用户打包集成了QAPM后，终于找到问题祸首了，原来是PHP的localIdentifier接口导致的

请在这里填写图片描述

找到PHP接口说明，

请在这里填写图片描述

每次调用localIdentifier会触发一次deep copy字符串，从而分配了内存；但是一个字符串也不可能这么容易导致内存飙涨呢？

定位

原因是我们在数组遍历中，频繁触发了localIdentifier接口的调用，根据ARC说明，不是new,alloc,等创建的对象，不是属于我们管理的，因此该接口返回的是一个autorelease的对象，其释放是由autoreleasepool统一释放的。
而我们的代码是如下调用的:

[albums enumerateObjectsWithOptions:NSEnumerationReverse usingBlock:^(PHAsset*  _Nonnull result, NSUInteger idx, BOOL * _Nonnull stop) {
                if (result) {
                    @synchronized (self.lockFileList) {
                        NSArray *arr=[_fileList copy];
                        for (int i = 0; i < arr.count; i++) {
                            MttPictureFile *file = arr[i];
                            if ([file.assetURL.absoluteString isEqual:result.localIdentifier]) {
                                *stop = YES;
                                return;
                            }
                        }
                    }

                    for (int i = 0; i < self.videoFiles.count; i++) {
                        MttAlbumVideoFile *file = self.videoFiles[i];
                        if ([file.assetURL.absoluteString isEqual:result.localIdentifier]) {
                            *stop = YES;
                            return;
                        }
                    }
                    
                    [self addPictureFileWithAsset:result];
                }
                ///...其余代码
}];

存在2个问题，

1.数组元素copy没有及时释放，这个还好，不会影响太大，但是遍历的时候访问了localIdentifier，这个会导致极端的情况下，触发上万个localIdentifier创建

2.另一个更严重的问题，经过反汇编调试，发现
[PHPFetchResult enumerateObjectsWithOptions:usingBlock:] 这里踩坑了，这里不带autoreleasepool而导致内存一直增加直到PHP遍历执行完毕后的autoreleasepool统一回调了才统一释放。

以上2点会导致，当用户有1万张图时，假设_fileList为5000,即用户一次性新增了5千张图，而原先有5000张图，那么这次回到就会触发5000 * 5000 * 2次localIdentifier的创建，这些遍历均会等到GCD block执行完毕才统一释放；这里就轻松的能占用几百MB到上千MB内存了

结论

1. 使用数组遍历时，切记多添加autoreleasepool
   [NSArray enumerateObjectsWithOptions:usingBlock:] 防止踩坑系统的各种array

2. 对于非自己alloc,new的autorelease对象，要及时@autoreleasepool包裹让其尽早释放；