Попытка открыть заново закрытую базу данных sqlited
У меня есть databaseHandler. Мне нужно подсчитать строки в таблице. Ошибка приложения с помощью этой ошибки: android attempt to reopen an already-closed object sqlitedatabase
.
Я просто использую этот код в действии:
db.getContactsCount();
Но приложение падает. Furthemore Я хочу сбросить таблицы (удалить строки таблицы). Я добавил метод ниже:
public void deleteTable() {
SQLiteDatabase db = this.getWritableDatabase();
db.delete("contacts", null, null);
}
Он работает хорошо, но я не могу использовать его:
@Override
public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
// Drop older table if existed
db.execSQL("DROP TABLE IF EXISTS " + TABLE_CONTACTS);
// Create tables again
onCreate(db);
}
Это обработчик базы данных:
public class DatabaseHandler extends SQLiteOpenHelper {
// All Static variables
// Database Version
private static final int DATABASE_VERSION = 1;
// Database Name
private static final String DATABASE_NAME = "contactsManager";
// Contacts table name
private static final String TABLE_CONTACTS = "contacts";
// Contacts Table Columns names
private static final String KEY_ID = "id";
private static final String KEY_NAME = "name";
private static final String KEY_PH_NO = "phone_number";
public DatabaseHandler(Context context) {
super(context, DATABASE_NAME, null, DATABASE_VERSION);
}
// Creating Tables
@Override
public void onCreate(SQLiteDatabase db) {
String CREATE_CONTACTS_TABLE = "CREATE TABLE " + TABLE_CONTACTS + "("
+ KEY_ID + " INTEGER PRIMARY KEY," + KEY_NAME + " TEXT,"
+ KEY_PH_NO + " TEXT" + ")";
db.execSQL(CREATE_CONTACTS_TABLE);
}
// Upgrading database
@Override
public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
// Drop older table if existed
db.execSQL("DROP TABLE IF EXISTS " + TABLE_CONTACTS);
// Create tables again
onCreate(db);
}
/**
* All CRUD(Create, Read, Update, Delete) Operations
*/
// Adding new contact
void addContact(Contact contact) {
SQLiteDatabase db = this.getWritableDatabase();
ContentValues values = new ContentValues();
values.put(KEY_NAME, contact.getName()); // Contact Name
values.put(KEY_PH_NO, contact.getPhoneNumber()); // Contact Phone
// Inserting Row
db.insert(TABLE_CONTACTS, null, values);
db.close(); // Closing database connection
}
// Getting single contact
Contact getContact(int id) {
SQLiteDatabase db = this.getReadableDatabase();
Cursor cursor = db.query(TABLE_CONTACTS, new String[] { KEY_ID,
KEY_NAME, KEY_PH_NO }, KEY_ID + "=?",
new String[] { String.valueOf(id) }, null, null, null, null);
if (cursor != null)
cursor.moveToFirst();
Contact contact = new Contact(Integer.parseInt(cursor.getString(0)),
cursor.getString(1), cursor.getString(2));
// return contact
return contact;
}
// Getting All Contacts
public List<Contact> getAllContacts() {
List<Contact> contactList = new ArrayList<Contact>();
// Select All Query
String selectQuery = "SELECT * FROM " + TABLE_CONTACTS;
SQLiteDatabase db = this.getWritableDatabase();
Cursor cursor = db.rawQuery(selectQuery, null);
// looping through all rows and adding to list
if (cursor.moveToFirst()) {
do {
Contact contact = new Contact();
contact.setID(Integer.parseInt(cursor.getString(0)));
contact.setName(cursor.getString(1));
contact.setPhoneNumber(cursor.getString(2));
// Adding contact to list
contactList.add(contact);
} while (cursor.moveToNext());
}
// return contact list
return contactList;
}
// Updating single contact
public int updateContact(Contact contact) {
SQLiteDatabase db = this.getWritableDatabase();
ContentValues values = new ContentValues();
values.put(KEY_NAME, contact.getName());
values.put(KEY_PH_NO, contact.getPhoneNumber());
// updating row
return db.update(TABLE_CONTACTS, values, KEY_ID + " = ?",
new String[] { String.valueOf(contact.getID()) });
}
// Deleting single contact
public void deleteContact(Contact contact) {
SQLiteDatabase db = this.getWritableDatabase();
db.delete(TABLE_CONTACTS, KEY_ID + " = ?",
new String[] { String.valueOf(contact.getID()) });
db.close();
}
// Getting contacts Count
public int getContactsCount() {
String countQuery = "SELECT * FROM " + TABLE_CONTACTS;
SQLiteDatabase db = this.getReadableDatabase();
Cursor cursor = db.rawQuery(countQuery, null);
cursor.close();
// return count
return cursor.getCount();
}
public void Upgrade (SQLiteDatabase db, int oldVersion, int newVersion) {
String CREATE_CONTACTS_TABLE = "CREATE TABLE " + TABLE_CONTACTS + "("
+ KEY_ID + " INTEGER PRIMARY KEY," + KEY_NAME + " TEXT,"
+ KEY_PH_NO + " TEXT" + ")";
db.execSQL(CREATE_CONTACTS_TABLE);
}
// Deleting single contact
public void deleteTable() {
SQLiteDatabase db = this.getWritableDatabase();
db.delete("contacts", null, null);
}
}
Ответы
Ответ 1
Это происходит из-за:
db.close();
в методах:
void addContact(Contact contact)
public void deleteContact(Contact contact)
Вы не должны закрывать соединение с базовой базой данных, если вы действительно не собираетесь работать с ней больше. Используйте SQLiteOpenHelper:close
, когда закончите работу.
Кроме того, вызовы getReadableDatabase()
и getWriteableDatabase()
возвращают один и тот же объект базы данных в 99% случаев, и они не повторно инициализируют подключение к базе данных, закрытое вручную вами.
Не обманывайте себя этими именами методов.
Ответ 2
Я хотел бы предложить вам, что не закрывайте объект базы данных до завершения работы с базой данных.
SQLiteDatabase db;
db = this.getWritableDatabase();
or
db = this.getReadableDatabase();
Выше оба метода дают нам объект базы данных с использованием вспомогательного класса и с помощью этого объекта мы можем выполнять различные действия в базе данных. Но, по вашему сценарию, я думаю, что вы закрыли объект базы данных до завершения операции с базой данных. Так закрыты после выполнения всей операции, используя
db.close();
Ответ 3
OnUpgrade - это когда вы вносили изменения в свою базу данных (например, добавляете таблицы), так что теперь есть новая версия вашей базы данных. Ваш:
private static final int DATABASE_VERSION = 1;
сейчас
private static final int DATABASE_VERSION = 2;
Поскольку версия базы данных теперь является более новой версией, вызывается метод OnUpgrade() и обновляет базу данных.
Ответ 4
это в основном происходит из-за cursor.close(); поскольку вы закрываете объект курсора до того, как вы вернете его счетчик.
Ответ 5
Я должен был бы определить int count = cursor.getCount();
перед закрытием базы данных. Кажется, return cursor.getCount();
после закрытия базы данных это бесполезное усилие.
Наконец, удалите db.close();
в конце удаления и удаления методов.
Вместо этого добавление этого кода вместо обработчика помогло:
public void closeDB() {
SQLiteDatabase db = this.getReadableDatabase();
if (db != null && db.isOpen())
db.close();
}
Манипулирование функцией close
внутри деятельности в конце использования базы данных было лучше. Благодаря @Drew.
Я все еще не могу понять использование onUpgrade
внутри обработчика???