web-dev-qa-db-ja.com

既に閉じられているオブジェクトsqlitedatabaseを再度開こうとする

私はdatabaseHandlerを持っています。テーブルの行を数える必要があります。このエラーによりアプリがクラッシュします:Android attempt to reopen an already-closed object sqlitedatabase
私は単にこのコードをアクティビティで使用します:

db.getContactsCount();

しかし、アプリがクラッシュします。さらに、テーブルをリセットしたい(テーブルの行を削除したい)。以下のメソッドを追加しました:

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);
    }
}
17
Namikaze Minato

それは次の理由で起こります:

db.close();

メソッドで:

void addContact(Contact contact)

public void deleteContact(Contact contact)

reallyでデータベースを操作する予定がない場合を除いて、基になるデータベースへの接続を閉じないでください。作業が終了したら、_SQLiteOpenHelper:close_を使用します。

さらに、getReadableDatabase()getWriteableDatabase()を呼び出すと、同じデータベースオブジェクトが99%の時間で返され、手動で閉じたデータベース接続をしないでください再初期化します。

これらのメソッド名に騙されないでください。

26
Drew

データベース作業を完了する前にデータベースオブジェクトを閉じないでください。

SQLiteDatabase db;
db = this.getWritableDatabase();

or

db = this.getReadableDatabase();

上記の両方のメソッドは、ヘルパークラスを使用してデータベースオブジェクトを提供し、このオブジェクトを使用して、データベースに対してさまざまなアクションを実行できます。しかし、あなたのシナリオでは、データベース操作を完了する前にデータベースオブジェクトを閉じたと思います。を使用してすべての操作を行った後、閉じた

db.close();
7
Kailas Bhakade

OnUpgradeはデータベースに変更を加えたとき(つまり、テーブルを追加したとき)なので、データベースの新しいバージョンが存在します。君の:

private static final int DATABASE_VERSION = 1;

今でしょ

private static final int DATABASE_VERSION = 2;

データベースのバージョンが新しいバージョンになっているため、OnUpgrade()メソッドが呼び出され、データベースが更新されます。

基本的に、cursor.close();が原因で発生します。カウントを返す前にカーソルオブジェクトを閉じているためです。

3
Ruchit Jain

データベースを閉じる前にint count = cursor.getCount();を定義しておく必要がありました。データベースを閉じた後のreturn cursor.getCount();は無駄な作業のようです。
deleteメソッドとremoveメソッドの最後に、最後にdb.close();を削除する必要がありました。
代わりにハンドラーにこのコードを追加すると、多くのことが役立ちました:

public void closeDB() {
    SQLiteDatabase db = this.getReadableDatabase();
    if (db != null && db.isOpen())
        db.close();
}

データベース使用の最後にアクティビティ内でclose関数を操作する方が優れていました。 @Drewに感謝します。
まだハンドラー内でのonUpgradeの使用法を理解できません???

2
Namikaze Minato