同僚とメモリリークの話になり自分の理解不足を痛感したので調査してみました。
前提 動作確認した環境は下記。
Android Studio 4.0.1
Pixel 3a XL(Android Emulator)
Android 10
また、今回はActivityがメモリリークしているか確認します。
内部クラスのstatic参照
BroadcastReceiver解除忘れ
インナークラスで親をプロパティ保持
メモリリークを検知する方法
アプリ起動
画面を回転する
Android SutdioのProfilerから強制的にGCを走らせる
Activityのfinalizeメソッドが呼ばれるか確認
正常系 まずは空のActivityの時にfinalizeメソッドは呼ばれるのか確認します。
ソースコード
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 class MainActivity : AppCompatActivity () { override fun onCreate (savedInstanceState: Bundle ?) super .onCreate(savedInstanceState) setContentView(R.layout.activity_main) Log.d(TAG, "onCreate" ) } override fun onDestroy () Log.d(TAG, "onDestroy" ) super .onDestroy() } protected fun finalize () Log.d(TAG, "finalize" ) } }
上記のようなコードを書きました。
結果
1 2 3 4 D/MainActivity: onCreate D/MainActivity: onDestroy D/MainActivity: onCreate D/MainActivity: finalize
当たり前ですが画面回転を行ってからGCが走ると画面回転前のMainActivityインスタンスは解放されます。
内部クラスのstatic参照 非staticな内部クラスを親がstaticプロパティとして保持するパターンです。
ソースコード
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 class MainActivity : AppCompatActivity () { companion object { private var innerClass: SomeInnerClass? = null } override fun onCreate (savedInstanceState: Bundle ?) super .onCreate(savedInstanceState) setContentView(R.layout.activity_main) Log.d(TAG, "onCreate" ) if (innerClass == null ) { innerClass = SomeInnerClass() } } inner class SomeInnerClass () }
結果
1 2 3 D/MainActivity: onCreate D/MainActivity: onDestroy D/MainActivity: onCreate
強制的にGCした時にfinalizeメソッドが呼ばれていないのでメモリリークを起こしています。
BroadcastReceiver解除忘れ Androidでよくある実装として、各種リスナーやレシーバーをActivityに登録して使うやり方があります。
ソースコード
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 class MainActivity : AppCompatActivity () { private var TAG = MainActivity::class .simpleName private var localBroadcastReceiver: BroadcastReceiver? = null override fun onCreate (savedInstanceState: Bundle ?) super .onCreate(savedInstanceState) setContentView(R.layout.activity_main) Log.d(TAG, "onCreate" ) } override fun onStart () super .onStart() registerBroadCastReceiver() } override fun onStop () super .onStop() } private fun registerBroadCastReceiver () localBroadcastReceiver = object : BroadcastReceiver() { override fun onReceive (context: Context , intent: Intent ) } } registerReceiver( localBroadcastReceiver, IntentFilter("android.net.conn.CONNECTIVITY_CHANGE" ) ) } }
結果
1 2 3 D/MainActivity: onCreate D/MainActivity: onDestroy D/MainActivity: onCreate
画面回転後にGCを走らせてもfinalizeメソッドが呼ばれませんでした。
インナークラスで親をプロパティ保持 ソースコード
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 class MainActivity : AppCompatActivity () { private var TAG = MainActivity::class .simpleName private lateinit var innerClass: InnerClass override fun onCreate (savedInstanceState: Bundle ?) super .onCreate(savedInstanceState) setContentView(R.layout.activity_main) Log.d(TAG, "onCreate" ) innerClass = InnerClass() innerClass.activity = this } override fun onDestroy () Log.d(TAG, "onDestroy" ) super .onDestroy() } protected fun finalize () Log.d(TAG, "finalize" ) } inner class InnerClass { lateinit var activity: Activity } }
結果
1 2 3 4 D/MainActivity: onCreate D/MainActivity: onDestroy D/MainActivity: onCreate D/MainActivity: finalize
MainActivityとInnerClass間で循環参照になっているのにGCで問題なく解放されてますね・・・謎
所感 公式のGCに関する情報 を確認する限りAndroid 10でも変更が入ってたりするのでそのあたりの変更により以前に比べてメモリリークしにくくなっているのかなぁと推測します。
参考サイト https://www.geeksforgeeks.org/memory-leaks-in-android/ https://qiita.com/amay077/items/3df253f66724c56faaff https://tomokey.blogspot.com/2011/05/android.html
