CAAnimation 中可能意想不到的内存泄漏问题

先说我遇到的此问题的场景

示例

如图,现在这种客户端跳网页,上方会有个加载进度条样式已经很普遍了.市面上几乎所有的app都是这个样式了.

可就在我某次无聊把玩项目打发时间时,偶然发现此ViewController会概率发生pop到上级页面后不释放问题,没有走dealloc方法 -_-!!!

内存泄漏可是项目大忌啊,好怕怕 *_*.发现了问题就得改啊.

经过多次测试把玩发现终于找到重现规律:
1. 就是在跳转H5页面后
2. 在webView还未完全加载完成,上方动画条progress还未走到100%时
3. 返回到上级页面后,该ViewController就会出现内存泄漏的问题

我的bug修复之旅:

起初我以为是webView的问题,折腾了很长时间发现无果.结果是webView最终释放了,但是控制器仍然没有释放.(当然webView使用不当,也会造成内存泄漏,这不是本次的讨论重点)

于是,我把思路转向第二个必现条件progress动画上.可是动画那一块的代码看去看来也不觉得有问题啊.

最后我猜可能是block使用不当,也许是哪一块出现了循环引用导致的.最后结果是,大把的时光飞去,那个背锅的block代码块并没有被我找到.

就在我一筹莫展之际,突然灵光一闪,我之所以怀疑block的更本原因是循环引用.那此控制器不被释放定然也是被什么给强引用了一下,pop到上级页面的时候,它没有释放,导致了当前页面也没有释放.于是我又把目光投向了第二步

只有动画未完成时,才会内存泄漏,那就应该是animation对象强行持有了当前控制器.再次查看代码:

    animation.duration = duration;
    animation.autoreverses = NO;
    animation.toValue = [NSValue valueWithCGRect:frame];
    animation.removedOnCompletion = NO;
    animation.fillMode = kCAFillModeForwards;
    animation.delegate = self;
    [self.progressView.layer addAnimation:animation forKey:@"progress"];

这里唯一一处引用了当前控制器的代码animation.delegate = self;于是我点开delegate进入看了下

@property(nullable, strong) id <CAAnimationDelegate> delegate;

额!!! 官方给出此属性定义的是strong类型,我们平时在开发时,为了防止内存泄漏,都定义的是weak或者assign.但是这里给出的是strong.也难怪我第一次看到这段代码的时候直接就忽略掉它了.坑呀

找到了原因,问题就好解决了.我在控制器生命周期中某个适当的环节添加了这句代码

[self.progressView.layer removeAllAnimations];

这样就ok了,记得适当时机移除掉动画.animation释放了,self自然就不再被持有了.

但是出于好奇为何animation的delegate用的是strong来定义,之前一直没注意.最后查阅了相关资料才有所了解

首先动画是异步的，在动画的过程中，它的 delegate 随时都有可能被释放掉，如果不是个强引用的话，比如用户点了返回之类的。另外一方面，一般来说你并不会持有一个 CAAnimation 的强引用（跟 UITableView 不一样）。文档里也说这是内存管理规则中的一个例外。

自己遇到的问题,拿出来总结一下.

最后编辑于：2018.01.11 18:12:03