笔记(一个关于形参的内存回收的问题)

今天踩了一个坑,被测试找到排查问题的过程一脸懵逼,由于问题稍显诡异而且搞出问题的做事方式比较符合我的思维习惯,必须记录下来,加深印象,严加警醒,防止以后记忆模糊了再犯同样的错误...
这里出问题的代码可以大概简略成下面的模型:

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对象在回调时又使用了这个形参,这个形参占用的内存空间也一直没有被回收掉,再加上调用方法的时候因为判空的原因代码没有执行进去, 所以每次调用方法执行的都是第一次传进去的那个值。
天坑啊!
其实这个形参没有被回收的原因,涉及到编译方面的知识,我也并不是十分明确,只是大概确定了问题点,但是目前是可以基本明确导致问题的写法了。
这种问题再叠加一些其他偶然的因素,有时候会隐藏的很深,并且表现出来奇奇怪怪,排查起来也不会那么顺,挺坑人的...
这一把记忆很深刻....再也不能这样写了..

    推荐阅读