如何防止静态字段中的上下文类引起的内存泄漏

Question

帮助！我有泄漏（至少我是这么认为的……）

下面的instance 值是我从静态伴随对象到此类的“端口”。我有一些关于以某种方式使用 applicationContext 的想法（我的 TODO：评论）......因为我收到了这个内存泄漏警告......：

为了避免这个问题，同时在每次调用这些函数时不需要上下文参数（我想以某种方式将上下文存储在类中，并在它死亡时将其处理掉），有什么好的练习解决问题？

...

class Repository private constructor(
        var context: Context
)
{
    /**
     * Repository class, class for all communication between database and external
     * sources
     * */

    private val TAG = javaClass.simpleName

    val appDatabase: AppDatabase = AppDatabase.getInstance(context.applicationContext)

    //TODO: Test if better with applicationContext?

    companion object {
        private val TAG = this::class.java.simpleName

        @Volatile private var instance:Repository ?=null

        fun getInstance(context: Context)= instance
                ?: synchronized(this){
                    instance
                            ?:Repository(context).also { instance = it }
                }
 

        fun liveDataUserResult(staffId: StaffType=StaffType.DRIVER) =
            instance?.liveDataUserResult(staffId = staffId)

        fun liveDataDepartmentResult(staffId: StaffType=StaffType.DRIVER) =
            instance?.liveDataDepartmentResult(staffId = staffId)

...

        fun liveDataGalleryItemSiteNoInUserPath() = instance?.liveDataGalleryItemSiteNoInUserPath()

...

    }

    fun liveDataGalleryItemSiteNoInUserPath() =
        appDatabase.galleryItemDao().liveDataGalleryItemSiteNoInUserPath(allText = context.getString(R.string.repository_all_text))

}
 

Answer 1

以上所有归功于 ADM 和 @Henry Twist 的出色答案（我已接受），但我将在此处发布我的结果：

class Repository private constructor(
         context: Context
)
{
    /**
     * Repository class, class for all communication between database and external
     * sources
     * */

    private val TAG = javaClass.simpleName

    val applicationContext: Context = context.applicationContext

    val appDatabase: AppDatabase = AppDatabase.getInstance(applicationContext)

    companion object {
        private val TAG = this::class.java.simpleName

        @Volatile private var instance:Repository ?=null

        fun getInstance(context: Context)= instance
                ?: synchronized(this){
                    instance
                            ?:Repository(context).also { instance = it }
                }


        fun liveDataUserResult(staffId: StaffType=StaffType.DRIVER) =
            instance?.liveDataUserResult(staffId = staffId)
        fun liveDataDepartmentResult(staffId: StaffType=StaffType.DRIVER) =
            instance?.liveDataDepartmentResult(staffId = staffId)

...
              
        fun liveDataGalleryItemSiteNoInUserPath() = instance?.liveDataGalleryItemSiteNoInUserPath()

...

    }

    fun liveDataGalleryItemSiteNoInUserPath() =
        appDatabase.galleryItemDao().liveDataGalleryItemSiteNoInUserPath(allText = applicationContext.getString(R.string.repository_all_text))
    
}

当然，Dagger2 或 Dagger-HILT 会更优雅。

Answer 2

不要在Repository 中传递Context，除非直接传递AppDatabase 实例。

AppDatabase 在整个应用程序中用作Singleton 因此，您只需在应用程序类或其他一些Singleton 中启动它一次，然后在任何地方使用相同的Object。

如果您使用任何依赖注入框架，如 Dagger2 或 Dagger-Hilt，创建/注入依赖将很容易。

您可以删除构造函数字段的 var 以使其工作，然后它将不会全局保存引用。感谢@Henry Twist。

class Repository private constructor(context: Context) {
    private val TAG = javaClass.simpleName
    var appDatabase: AppDatabase = AppDatabase.getInstance(context.applicationContext)
    companion object {
        @Volatile
        private var instance: Repository? = null
        @JvmStatic
        fun getInstance(context: Context) = instance
            ?: synchronized(this) {
                instance
                    ?: Repository(context).also { instance = it }
            }
    }
}