База данных SQLite никогда не закрывается в моем приложении для Android - PullRequest
0 голосов
/ 09 марта 2012

У меня проблема с моим приложением для Android. Я использую код ниже для открытия базы данных SQLite в AsyncTask. Все работает нормально, но когда я пытаюсь закрыть базу данных методом onStop() или onDestroy, она никогда не закрывается.

Код для создания и открытия базы данных:

открытый класс SQLiteDB расширяет SQLiteOpenHelper {

private final Context context;  
private SQLiteDatabase sqliteDatabase = null;

public SQLiteDB(Context context, String DBName) {
    super(context, DBConstant.DB_NAME, null, context.getResources().getInteger(ppredota.android.navigation.view.activities.R.string.database_version));
    this.context = context;
}

public void createDB() throws IOException{


    if(existDB()){
        this.getReadableDatabase();
        this.close();
    }
    else {
        this.getWritableDatabase();
        try {
            copyDB();
            this.close();   
        }
        catch (Exception e) {
            throw new Error("Chyba pri kopirovani databaze");
        }
    }
}
private boolean existDB() {

    SQLiteDatabase checkDatabase = null;

    try{
        String fullPath = DBConstant.DB_PATH + DBConstant.DB_NAME;
        checkDatabase = SQLiteDatabase.openDatabase(fullPath, null, SQLiteDatabase.OPEN_READWRITE);
    }
    catch (SQLiteException sqle) {
        Log.i("existDB()", "Databaze nelze otevrit, neexistuje");
    }
    if(checkDatabase == null){
        Log.i("existDB", "Databaze jeste neexistuje...");
        return false;
    }
    else{
        Log.i("existDB", "Databaze uz existuje...");
        checkDatabase.close();
        return true;
    }

}

private void copyDB() throws IOException {

    InputStream inDBStream = context.getAssets().open(DBConstant.DB_NAME);
    String newDBPath = DBConstant.DB_PATH + DBConstant.DB_NAME;

    OutputStream outDBStream = new FileOutputStream(newDBPath);
    Log.i("copyDB", "Otevren outputstream s cestou k nove databazi");

    byte[] buffer = new byte[1024];
    int length;
    while ((length = inDBStream.read(buffer))>0){
        outDBStream.write(buffer, 0, length);
    }

    outDBStream.flush();
    outDBStream.close();
    inDBStream.close();
}

public void openDB() throws SQLException {
    String fullPath = DBConstant.DB_PATH + DBConstant.DB_NAME;
    if(sqliteDatabase!=null){
        if(sqliteDatabase.isOpen()){
            Log.i("openDB()", "Databaze je jiz otevrena");
        }
        else{
            sqliteDatabase = SQLiteDatabase.openDatabase(fullPath, null, SQLiteDatabase.OPEN_READONLY);
            Log.i("openDB()", "Databaze" + sqliteDatabase.getPath() + "otevrena");
        }
    }
    else{
        sqliteDatabase = SQLiteDatabase.openDatabase(fullPath, null, SQLiteDatabase.OPEN_READONLY);
        if(sqliteDatabase.isOpen()){
            Log.i("openDB()", "Databaze otevrena");
        }

    }
}
@Override
public void close() {

    if(sqliteDatabase!=null){
        sqliteDatabase.close();
        Log.i("close()", "Databaze zavrena");
    }
    super.close();

}

public SQLiteDatabase getSQLiteDatabase() {
    if(sqliteDatabase==null){
        Log.i("getSQLiteDatabase()","Problem, vraci sqliteDatabase = null");
    }
    else{
        Log.i("getSQLiteDatabase()","instance sqliteDatabase vracena bez problemu");
    }

    return sqliteDatabase;

}

Класс AssyncTask:

открытый класс OpenDatabaseTask расширяет AsyncTask {

private Context context;
private SQLiteDB sqliteDB;

public OpenDatabaseTask(Context context,SQLiteDB sqliteDB) {
    this.context = context;
    this.sqliteDB = sqliteDB;
}
@Override
protected Void doInBackground(Void... params) {
    publishProgress();

    try {
        sqliteDB.createDB();
    } catch (IOException e) {
        e.printStackTrace();
    }
    sqliteDB.openDB();

    return null;
}
@Override
protected void onProgressUpdate(Void...unused){
    Log.i(OpenDatabaseTask.class.toString(), "Spusteno vlakno");
}

}

и Активность (только важная часть):

    private SQLiteDB sqliteDB;
    private SQLiteData sqliteData;
    private OpenDatabaseTask openDatabaseTask;

 @Override
 public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.navigatemenu);

        sqliteDB = new SQLiteDB(getApplicationContext(), sourceDatabaseName);
        openDatabaseTask = new OpenDatabaseTask(getApplicationContext(), sqliteDB);
        openDatabaseTask.execute();

    protected void onDestroy(){
        super.onDestroy();
        Log.i("onDestroy()", "Pokus o zavreni databaze");
            //here is the problem, database never closed
        sqliteDB.close();
    }
    protected void onStop(){
        super.onStop();
        Log.i("onStop()", "Pokus o zavreni databaze");
            //here is the problem, database never closed
        sqliteDB.close();
    }
    protected void onPause(){
        super.onPause();
    }
    protected void onResume(){
        super.onResume();
        //Log.i("onResume()", "Pokus o otevreni databaze");
    }
}

Поэтому, когда я пытаюсь использовать метод close() для закрытия базы данных, sqliteDatabase всегда равен нулю, а база данных никогда не закрывается. Таким образом, база данных все еще открыта, и после вызова onDestroy возникает исключение.

Спасибо за ваше время и извините за мой английский, я чешский :)

Ответы [ 2 ]

0 голосов
/ 26 мая 2015

У меня есть приложение с интенсивным взаимодействием с БД.Я не пользователь помощник.Моя БД открывается много раз, в потоке пользовательского интерфейса и в фоновом потоке, но никогда не закрывается.Пока проблем не было, не знаю, правильно ли это делать ...

0 голосов
/ 09 марта 2012

Просто предположение (при условии, что вы хотите закрыть при уничтожении)

закрыть базу данных перед вызовом супер

protected void onDestroy(){
    // close befor super is called
    sqliteDB.close();

    super.onDestroy();
    Log.i("onDestroy()", "Pokus o zavreni databaze");
    // sqliteDB.close(); // super.onDestroy may already has destroyed the DB
}

Примечание:

Предполагая, чтобаза данных существует и открыта только тогда, когда видимость видна, вы должны открыть базу данных в onResume () и закрыть ее в onPause ().

Если база данных должна быть открыта при загрузке кода в память, создайте еев onCreate и закройте его в onDestroy

В вашем примере вы открываете его в onCreate и закрываете его в onStop. Проблема: когда активность становится видимой во второй раз, когда БД закрывается.Подробнее см. документация по активности Android и посмотрите на жизненный цикл приложения при нажатии кнопки.

...