数据库未关闭

Question

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

这个问题并不严重，这意味着我的程序不会崩溃，它会继续运行，但无论如何我希望尽可能不要出现错误。

我正在使用 AsyncTask 类来创建一些对象并将它们加载到一个数组中。我使用的数据库是从网上获得的文件，因此我没有 DBhelper 类。我在 AsyncTask 的 doInBackround 中打开数据库。在里面我有 try-catch 我做我所有的数据库相关的工作。

起初我在 try 语句中有 db 和 cursors 的变量，并且我还关闭了 db 和 cursors 在最后。我发现一个链接说应该使用 try-catch-final，所以我这样做了，我将变量声明移到 try-catch-final 语句的外部并管理关闭 db 和 cursor final 部分中的 em> 变量，但仍然没有帮助，错误仍然存​​在。

这里有一些错误，我不会全部 C/P：

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

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，在调试时我注意到错误仅在显示新 Activity 时显示，而不是在退出后立即显示来自doInBackground。

感谢您的帮助！

Answer 1

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

1. 在“if”条件内（if (SQLiteDatabase.openDatabase(DBpath, null, SQLiteDatabase.OPEN_READONLY) == null) return false;）

2. 在下一行 (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() 时失去了对对象的引用。