笔记(一个关于形参的内存回收的问题)
今天踩了一个坑,被测试找到排查问题的过程一脸懵逼,由于问题稍显诡异而且搞出问题的做事方式比较符合我的思维习惯,必须记录下来,加深印象,严加警醒,防止以后记忆模糊了再犯同样的错误...
这里出问题的代码可以大概简略成下面的模型:
class Test {fun run() {
test(1)
test(2)
test(3)
}var a: A? = null;
private fun test(i: Int) {
Log.d("test", "param i = $i")//重点注意,i的值(1)
if (null == a) {
a = A();
a?.setCallback(object : A.Callback {
override fun onCallback() {
Log.d("test", "i = $i")//重点注意,i的值(2)
}
})
}
a?.doIt()
}class A {
fun doIt() {
mCallBack?.onCallback()
}lateinit var mCallBack: Callback
fun setCallback(callback: Callback) {
mCallBack = callback
}interface Callback {
fun onCallback()
}
}
}
按理来说,调用三次test(),传入三个不同的值,执行下去应该也是三个不同的结果才对。但是由于形参内存回收方面的问题,在这种逻辑下,打印出来的日志是这样的:
【笔记(一个关于形参的内存回收的问题)】
文章图片
日志
第一处和第二处两处位置引用同一个形参,打出来的值是不一样的...
第一处的值都是实时传进来的值,但是第二处的值,全部都是第一次传入的那个值。
我都懵逼了....
在短暂的懵逼之后大概捋了捋,基本能判断出是判空那里出的问题,由于有判空的逻辑存在,第二三次调用时代码是没有执行到第二次日志那里面的位置的,由此引发了大问题。
其实这里这样写的本意是为了节约运行内存,避免每次都去创建一个新的A对象,只创建一次,之后就反复使用。但是在这里的这个逻辑框架下,反而挖了一个大坑,把自己给埋进去了~
从出现的问题和代码的结构基本可以断定,第一次传进来的形参一直没有被回收,而且再后续的调用中被反复使用了,于是为了验证这个猜想,我把两次日志位置形参的内存地址打印了出来。
文章图片
形参内存地址
可以看到第二处位置形参的内存地址是一模一样的,只有第一处形参的内存地址发生了变动,这样基本就验证了猜想是对的。按常理,形参在调用方法执行完成之后内存应该会立刻被回收掉,但是这里没有,因为这里a是一个全局变量,且一直存在,而给a设置的匿名Callback对象也因为a的一直存在而没有被回收掉内存,这个Callback对象在回调时又使用了这个形参,这个形参占用的内存空间也一直没有被回收掉,再加上调用方法的时候因为判空的原因代码没有执行进去, 所以每次调用方法执行的都是第一次传进去的那个值。
天坑啊!
其实这个形参没有被回收的原因,涉及到编译方面的知识,我也并不是十分明确,只是大概确定了问题点,但是目前是可以基本明确导致问题的写法了。
这种问题再叠加一些其他偶然的因素,有时候会隐藏的很深,并且表现出来奇奇怪怪,排查起来也不会那么顺,挺坑人的...
这一把记忆很深刻....再也不能这样写了..
推荐阅读
- 一个人的旅行,三亚
- 一个小故事,我的思考。
- EffectiveObjective-C2.0|EffectiveObjective-C2.0 笔记 - 第二部分
- 一个人的碎碎念
- 七年之痒之后
- 我从来不做坏事
- 异地恋中,逐渐适应一个人到底意味着什么()
- 迷失的世界(二十七)
- live|live to inspire 一个普通上班族的流水账0723
- 遗憾是生活的常态,但孝顺这件事,我希望每一个人都不留遗憾