Question

我查看了有关此问题的其他问题，并尝试了他们的解决方案，但不幸的是，它没有帮助我。

问题不是关键性的，这意味着我的程序不会崩溃，它会继续运行，但无论如何我希望在可能的情况下不会出错。

我正在使用 AsyncTask 类来创建一些对象并将它们加载到一个数组中。我使用的数据库是从网络获取的文件，因此我没有DBhelper类。我在AsyncTask的doInBackround中打开数据库。在其中我尝试捕获我所有与数据库相关的工作。

起初我在try语句中有 db 和 cursors 的变量，而且还关闭了 db 和游标在最后。我找到了一个链接，说应该使用try-catch-final，所以我这样做了，我将try-catch-final语句的变量声明移出，并管理 db 和游标的关闭 final 部分中的变量，但仍然无效，错误仍在此处。

以下是一些错误，我不会C / P全部：

01-25 17：22：45.142：E / Database（333）：从未在数据库'/data/data/stet.cityapp/app_databases/baza.db上显式调用close（）

01-25 17：22：45.142：E / Database（333）：android.database.sqlite.DatabaseObjectNotClosedException：应用程序未关闭此处打开的游标或数据库对象

01-25 17：22：45.142：E / Database（333）：at stetocina.cityapp.SplashScreen $ LoadDB.doInBackground（SplashScreen.java:1）

01-25 17：22：45.142：E / Database（333）：at java.util.concurrent.FutureTask $ Sync.innerRun（FutureTask.java:306）

01-25 17：22：45.142：E / Database（333）：at java.util.concurrent.ThreadPoolExecutor $ Worker.run（ThreadPoolExecutor.java:581）

代码概要：

protected Boolean doInBackground(String... params) {
SQLiteDatabase db = null;
Cursor poisCursor = null, tmpCursor = null;
String whereString;
try {               
    String DBpath = getDir("databases", 0).getAbsolutePath() + File.separator + "baza.db";
    if (SQLiteDatabase.openDatabase(DBpath, null, SQLiteDatabase.OPEN_READONLY) == null) return false;
    db =  SQLiteDatabase.openDatabase(DBpath, null, SQLiteDatabase.NO_LOCALIZED_COLLATORS);

    ...

    result = true;
} catch (Exception e) {
    result = false;
    return result;
} finally {
    if (!poisCursor.isClosed() || poisCursor != null) {
        poisCursor.deactivate();
        poisCursor.close();
        poisCursor = null;
    }
    if (!db.isOpen() || db != null) {
        db.close();
        db = null;
    }
}
return result;

}

我是否在AsyncTask中遇到了一些问题？在AsyncTaks的 onPostExecute 方法中，如果一切顺利且结果为真，我打开一个新的Activity，在调试时我发现错误仅在显示新活动时显示，而不是在退出后立即显示来自doInBackground。

感谢您的帮助！

Answer 1

数据库是不是被打开了两次？

在'if'条件内（如果（SQLiteDatabase.openDatabase（DBpath，null，SQLiteDatabase.OPEN_READONLY）== null）返回false;）
在下一行（db = SQLiteDatabase.openDatabase（DBpath，null，SQLiteDatabase.NO_LOCALIZED_COLLATORS）;）

并且您只关闭第二个（db）。

Answer 2

仔细看:)这个

if (!db.isOpen() || db != null) {
    db.close();
    db = null;
}

只有在未打开时才会尝试关闭数据库。此外，我认为

有问题

|| x != null

这两个条款的一部分。

在Java中，“双”逻辑运算符是条件。对于||这意味着如果第一个参数是 true 那么第二个参数根本就不会被评估，因为无论如何逻辑OR都已经为真。如果不是 - 则检查您的链接是否为空。那么，你正在做的是，你检查光标是否关闭。如果关闭，你检查链接是否为null - 如果成功调用任何方法，这显然是真的，所以不使用第二次检查。

我认为正确的条款将是

if ( poisCursor != null && !poisCursor.isClosed()) // prevent NPE when calling .isClosed() in case poisCursor is null

和

if (db != null && db.isOpen()) // same thing + note the absence of negation

（您可以在此处详细了解条件运算符：http://docs.oracle.com/javase/tutorial/java/nutsandbolts/op2.html）

编辑： @ firebolt7是对的，你在if子句中调用open（）时失去了对你所获得的对象的引用。

数据库未关闭

2 个答案: