使用多线程遍历或操作集合对象的时候,务必要注意内存的及时释放,否则一不小心就触发内存峰值而闪退了。
最近新版本发布后,有较多用户反馈闪退;经过部分走访用户初步得出结论是内存闪退,而且和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内存了
结论
使用数组遍历时,切记多添加autoreleasepool
[NSArray enumerateObjectsWithOptions:usingBlock:] 防止踩坑系统的各种array对于非自己alloc,new的autorelease对象,要及时@autoreleasepool包裹让其尽早释放;