发现 SQLite 数据库泄漏

Question

我正在创建一个应用程序。我收到此错误：

11-08 13:46:24.665: 错误/数据库 (443): java.lang.IllegalStateException： /data/data/com.testproj/databases/Testdb SQLiteDatabase 创建并 从未关闭

我似乎找不到原因，因为它有时会向我显示错误，有时不会。这是我的代码：

public class SQLiteAssistant extends SQLiteOpenHelper {
    public SQLiteAssistant(Context context){
            super(context, DB_NAME, null, DB_VERSION_NUMBER);
            this.myContext = context;
    }

    public void openDataBase() throws SQLException{
        String myPath = DB_PATH + DB_NAME;
        myDataBase = SQLiteDatabase.openDatabase(myPath, null, SQLiteDatabase.OPEN_READWRITE);
    }

    public void closeDataBase() {
        if(this.myDataBase != null) {
            if(this.myDataBase.isOpen())
                this.myDataBase.close();
            }
        }   
    }
}

在另一个类中，我有这些查询：

public class Db{  

    private static SQLiteAssistant sqlite;

    public static String getSomeString(Context ctx) {

        sqlite = new SQLiteAssistant(ctx);
        sqlite.openDataBase();

        Cursor cursor = sqlite.myDataBase.rawQuery("SELECT someColumn from SomeTable",null);

        if (cursor != null) {
            if (cursor.getCount()==1) {
                 if(cursor.moveToFirst()) {
                     String testString = cursor.getString(cursor.getColumnIndex("someColumn")); 
                     cursor.close();
                     sqlite.closeDataBase();
                     sqlite.close();
                     return testString
                 }
            }
        }

        sqlite.closeDataBase();
        sqlite.close();

        return null;
     }
}

我的问题是当我启动一个新活动时，我在其中得到一个 AsyncTask。此任务从 Web 服务获取数据并访问数据库以获取 String。这是AsyncTask：

protected class BackTask extends AsyncTask<Context, String, String> {
     @Override
     protected String doInBackground(Context... params) {
         try{
            //get requeste data from the database
            //access the web service

            return result;

         } catch (Exception e) { 
                   return null;
         }
         return null;
     }
}

如果我让活动顺其自然，一切都会顺利。如果我不这样做并快速按后退按钮，则会收到错误消息。关于如何解决这个问题有什么建议吗？

Answer 1

我不确定您是否正确使用 SQLiteOpenHelper...您不需要该 myDataBase 字段，其想法是它为您管理数据库连接。不要以这种方式子类化...除非您在 onCreate() 等中执行未在此处发布的操作，看起来您可以直接使用 SQLiteOpenHelper ，即：

SQLiteOpenHelper sqlite = new SQLiteOpenHelper(ctx, DB_PATH+DB_NAME, null,
    DB_VERSION_NUMBER);

假设结束活动也应该停止您的后台任务，我建议致电 AsyncTask.cancel(true) 来自您的 Activity.onPause()。确保通过 onCancelled() 清除数据库。

如果您的后台任务是唯一读取数据库的任务，那么让它拥有 SQLiteOpenHelper 实例。静态数据很容易陷入麻烦，所以恕我直言，最好避免它。我会做这样的事情：

protected class BackTask extends AsyncTask<String, Integer, String>
{
    private SQLiteOpenHelper sqlite;

    public void BackTask(Context ctx) {
        sqlite = new SQLiteOpenHelper(ctx, DB_PATH+DB_NAME, null,
                                      DB_VERSION_NUMBER);
    }
    @Override
    protected String doInBackground(String... params) 
    {
         try {
                //get requeste data from the database
                //access the web service
                return result;

              } catch (Exception e) { 
         }
         return null;
    }

    @Override
    protected void onCancelled() {
         sqlite.close();
    }

    @Override
    protected void onPostExecute(String result)
         sqlite.close();
         // Update UI here
    }
}

Answer 2

我认为这部分：

 cursor.close();
                sqlite.closeDataBase();
                        sqlite.close();

必须在finally close中，比如

Try{ 
    //Do something
   }
   catch(){
     //Catch exception
   }
   finally{
   //Close cursor or/and eventually close database if you don't need it in the future
   }

也不要忘记在onDestroy方法中关闭数据库。

onCreate(Bundle b){ 
//create database instance
}
onDestroy{
//close db
}