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

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

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

联系用户

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

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

分析

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


1987809-c1ed0b6edcf74b3c.png
请在这里填写图片描述

找到PHP接口说明,


1987809-c9f260f9f95b774a.png
请在这里填写图片描述

每次调用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包裹让其尽早释放;

猜你喜欢

转载自blog.csdn.net/weixin_34082695/article/details/87390981