1

While adding Room Database, it is suggested to use Singleton Design Pattern

Note: You should follow the singleton design pattern when instantiating an AppDatabase object, as each RoomDatabase instance is fairly expensive, and you rarely need access to multiple instances.

So, adding Room Database, following Google example which is written in Java, in will be as below

private var INSTANCE: AppDatabase? = null

fun getInstance(context: Context): AppDatabase? {
    if (INSTANCE == null){
        synchronized(AppDatabase::class){
            INSTANCE = Room.databaseBuilder(context.applicationContext,
                    AppDatabase::class.java, "app_database")
                    .build()
        }
    }
    return INSTANCE
}

When I call getInstance, compiler suggests that getInstance can be null. So my question is there any case that getInstance be null and do I have to check if it's null. If not, then how should I instantiate AppDatabase so that getInstance return AppDatabase not AppDatabase? and it fits documentation recommendation?

1 Answer 1

1

Use something like this to make sure it is not null

object DatabaseSource {
    private lateinit var INSTANCE: AppDatabase

    fun getInstance(context: Context): AppDatabase {
        if (!::INSTANCE.isInitialized) {
            synchronized(AppDatabase::class) {
                if (!::INSTANCE.isInitialized) {
                    INSTANCE = Room.databaseBuilder(context.applicationContext,
                                                    AppDatabase::class.java,
                                                    "app_database").build()
                }
            }
        }
        return INSTANCE
    }
}

Call it with:

val db = DatabaseSource.getInstance(context)

And this will never be null so you no longer have that issue.

I added double locking to be safer for thread safety on the getInstance() call.

But really you should be using dependency injection with singletons to avoid this passed in dependency of context and this manual creation/locking. You'll have to have the context available everywhere which is a bad pattern.

This also acts as an answer to your other post which is apparently a duplicate of this one.

9
  • That indeed was what I am looking for. But how would I pass the context here? Since I am creating INSTANCE in my companion object, it's not a good practice to have context inside static fields. Looking google sample, I see that they use also which works perfectly. What do you say?
    – musooff
    Nov 20, 2018 at 2:16
  • Are you using dependency injection? because this seems like it should be injected as a singleton, and the context should be made available for injection as well. Nov 20, 2018 at 2:38
  • 1
    The comment about not using lateinit there is not universally a valid comment. It is fine in any context where you KNOW the lifecycle gaurantees it will be instantiated before use. Here the only way to access it is by a method that guarantees it is valid. So you are fine. The other comment is not relevant without this context. Nov 20, 2018 at 3:07
  • 1
    I saw your proposed edit of volatile, but since this is only modified under lock, and we are checking if assigned which is I believe a bit field that would not be partially set. You can add it if you want, but not sure it is really what you want and will just add more contention on accessing it. Nov 20, 2018 at 3:11
  • 1
    Let's not add things we see in other posts without reason. Nov 20, 2018 at 3:12

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.