Android SQLite DB Когда закрывать

Я работаю с базой данных SQLite на android. Мой менеджер баз данных является одноэлементным и прямо сейчас открывает соединение с базой данных при ее инициализации. Безопасно оставлять базу данных открытой все время, чтобы, когда кто-то вызывает мой класс для работы с базой данных, она уже открыта? Или мне следует открывать и закрывать базу данных до и после каждого доступа. Есть ли вред просто оставлять его открытым все время?

Спасибо!

Я бы держал его открытым все время и закрывал каким-нибудь методом жизненного цикла, например onStopили onDestroy. Таким образом, вы можете легко проверить , если база данных уже используется по телефону isDbLockedByCurrentThreadили isDbLockedByOtherThreadsна одном SQLiteDatabaseобъекте каждый раз , прежде чем использовать его. это предотвратит множественные манипуляции с базой данных и спасет ваше приложение от потенциального сбоя

поэтому в вашем синглтоне у вас может быть такой метод для получения вашего единственного SQLiteOpenHelperобъекта:

private SQLiteDatabase db;
private MyDBOpenHelper mySingletonHelperField;
public MyDBOpenHelper getDbHelper() {
    db = mySingletonHelperField.getDatabase();//returns the already created database object in my MyDBOpenHelper class(which extends `SQLiteOpenHelper`)
    while(db.isDbLockedByCurrentThread() || db.isDbLockedByOtherThreads()) {
        //db is locked, keep looping
    }
    return mySingletonHelperField;
}

поэтому всякий раз, когда вы хотите использовать свой открытый вспомогательный объект, вызывайте этот метод получения (убедитесь, что он многопоточен)

другой метод в вашем синглтоне может быть (вызывается КАЖДЫЙ РАЗ, прежде чем вы попытаетесь вызвать метод получения выше):

public void setDbHelper(MyDBOpenHelper mySingletonHelperField) {
    if(null == this.mySingletonHelperField) {
        this.mySingletonHelperField = mySingletonHelperField;
        this.mySingletonHelperField.setDb(this.mySingletonHelperField.getWritableDatabase());//creates and sets the database object in the MyDBOpenHelper class
    }
}

вы также можете закрыть базу данных в синглтоне:

public void finalize() throws Throwable {
    if(null != mySingletonHelperField)
        mySingletonHelperField.close();
    if(null != db)
        db.close();
    super.finalize();
}

Если пользователи вашего приложения могут очень быстро создавать множество взаимодействий с базой данных, вам следует использовать что-то вроде того, что я продемонстрировал выше. но если взаимодействие с базой данных минимально, я бы не стал беспокоиться об этом и просто каждый раз создавал и закрывал базу данных.

На данный момент нет необходимости проверять, заблокирована ли база данных другим потоком. Хотя вы используете одноэлементный SQLiteOpenHelper в каждом потоке, вы в безопасности. Из isDbLockedByCurrentThreadдокументации:

Название этого метода происходит от того времени, когда активное соединение с базой данных означало, что поток удерживал фактическую блокировку базы данных. В настоящее время больше не существует настоящей «блокировки базы данных», хотя потоки могут блокироваться, если они не могут получить соединение с базой данных для выполнения определенной операции.

isDbLockedByOtherThreads устарело с API уровня 16.

По вопросам:

Мой менеджер баз данных является одноэлементным и прямо сейчас открывает соединение с базой данных при ее инициализации.

Мы должны разделить «открытие БД», «открытие соединения». SQLiteOpenHelper.getWritableDatabase () предоставляет открытую БД. Но нам не нужно контролировать соединения, поскольку это делается внутри компании.

Безопасно оставлять базу данных открытой все время, чтобы, когда кто-то вызывает мой класс для работы с базой данных, она уже открыта?

Да, это так. Соединения не зависают, если транзакции правильно закрыты. Обратите внимание, что ваша БД также будет автоматически закрыта, если GC завершит ее.

Или мне следует открывать и закрывать базу данных до и после каждого доступа.

Закрытие экземпляра SQLiteDatabase не дает ничего особенного, кроме закрытия подключений, но это плохо для разработчика, если в данный момент есть какие-то подключения. Кроме того, после SQLiteDatabase.close () SQLiteOpenHelper.getWritableDatabase () вернет новый экземпляр.

Есть ли вред просто оставлять его открытым все время?

Нет, нет. Также обратите внимание, что закрытие БД в несвязанный момент и поток, например, в Activity.onStop (), может закрыть активные соединения и оставить данные в несогласованном состоянии.

В Android 8.1 есть SQLiteOpenHelper.setIdleConnectionTimeout(long)метод, который:

Устанавливает максимальное количество миллисекунд, в течение которого соединение SQLite может находиться в режиме ожидания, прежде чем оно будет закрыто и удалено из пула.

https://developer.android.com/reference/android/database/sqlite/SQLiteOpenHelper.html#setIdleConnectionTimeout(long)

С точки зрения производительности оптимальный способ - сохранить единственный экземпляр SQLiteOpenHelper на уровне приложения. Открытие базы данных может быть дорогостоящим и является блокирующей операцией, поэтому ее не следует выполнять в основном потоке и / или в методах жизненного цикла активности.

Метод setIdleConnectionTimeout () (представленный в Android 8.1) можно использовать для освобождения оперативной памяти, когда база данных не используется. Если установлен тайм-аут простоя, соединение (я) с базой данных будет закрыто после периода бездействия, т. Е. Когда к базе данных не было доступа. Соединения будут повторно открываться прозрачно для приложения при выполнении нового запроса.

Кроме того, приложение может вызывать releaseMemory (), когда оно переходит в фоновый режим или обнаруживает нехватку памяти, например, в onTrimMemory ().

Вы также можете использовать ContentProvider. Он сделает это за вас.

Создайте свой собственный контекст приложения, затем откройте и закройте базу данных оттуда. Этот объект также имеет метод OnTerminate (), который можно использовать для закрытия соединения. Я еще не пробовал, но кажется, что это лучший подход.

@binnyb: Мне не нравится использовать finalize () для закрытия соединения. Может работать, но из того, что я понимаю, писать код в методе Java finalize () - плохая идея.