数据库未关闭

Posted 2023-04-19

【中文标题】数据库未关闭

【英文标题】：Database not closed

【发布时间】：2012-02-18 20:38:01

【问题描述】：

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

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

我正在使用 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。

感谢您的帮助！

【参考方案1】：

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

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

在下一行 (db = SQLiteDatabase.openDatabase(DBpath, null, SQLiteDatabase.NO_LOCALIZED_COLLATORS);)

你只关闭第二个（db）。

【参考方案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() 时失去了对对象的引用。

【讨论】：

虽然你说的!db.isOpen()看起来不对，但是第二部分检查db != null会成功，所以这不是错误的原因。

嗯，是的，你是对的。所以无论如何都应该尝试关闭数据库。那么 firebolt7 一定是对的 - 你失去了到一个 db 实例的链接，我错过了那部分:)

感谢您的调查，但错误与 firebolt7 建议的一样。这个 if 子句打开了 db，我以前有它，以测试 db 是否存在，我从没想过关闭它，因为我认为 android 会自行完成，因为我没有将此连接放入变量中。我的错。还有对db.isOpen的否定，你是对的。