尽管所有内容都已关闭，但SQLite连接泄漏了。

Question

尽管所有内容都已关闭，但SQLite连接泄漏了。

41

我发现很多东西像是 close the connection 和 close the cursor，但我都已经做了这些事情。仍然存在SQLite连接泄漏的问题，我收到了如下的警告：

A SQLiteConnection object for database was leaked!

我有一个数据库管理器，我在我的活动中使用以下代码调用：

DatabaseManager dbm = new DatabaseManager(this);

下面是我的数据库管理器类的代码：

public class DatabaseManager {

    private static final int DATABASE_VERSION = 9;
    private static final String DATABASE_NAME = "MyApp";
    private Context context = null;
    private DatabaseHelper dbHelper = null;
    private SQLiteDatabase db = null;


    public static class DatabaseHelper extends SQLiteOpenHelper {

         public DatabaseHelper(Context context) {
             super(context, DATABASE_NAME, null, DATABASE_VERSION);
         }

         @Override
         public void onCreate(SQLiteDatabase db) {

                   //create database tables
         }

         @Override
         public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
                      //destroy and recreate them
         }

     }

     public DatabaseManager(Context ctx) {
         this.context = ctx;
     }

    private DatabaseManager open() throws SQLException {
        dbHelper = new DatabaseHelper(context);
        db = dbHelper.getWritableDatabase();

        if (!db.isReadOnly()) {
            db.execSQL("PRAGMA foreign_keys = ON;");
        }

        return this;
    }

    private void close() {
        dbHelper.close();
    }
}

当我调用数据库方法时，我会执行以下操作：

public Object getData() {

    open();

            //... database operations take place ...

    close();

    return data;
}

但如我所说，我仍然收到了SQLite连接泄漏的警告。

我做错了什么？

- flp

1

我认为你也应该调用db.close()。 - user1525382

4

为什么DatabaseHelper要是静态的？这可能会导致资源泄漏，因为在没有对DatabaseHelper的引用时，它仍然持有到数据库的连接。 - MalaKa

1

我已经移除了static，但不幸的是它仍然泄漏。 - flp

close(); 行是否总是在成功执行 open(); 行后执行？您是否记录异常或只是忽略它们？ - iCantSeeSharp

你应该在finally块中关闭资源，例如将db.close()放在finally块中。 - Igor Čordaš

显示剩余3条评论

3个回答

3

上面接受答案的完整例子如下：它可能会对某些人有所帮助。

辅助类:

public class DatabaseHelper extends SQLiteOpenHelper {

private static final String DATABASE_NAME = "sample.db";
private static final int DATABASE_VERSION = 1;

private static DatabaseHelper mInstance;

private DatabaseHelper(@Nullable Context context) {
    super(context, DATABASE_NAME, null, DATABASE_VERSION);
}

public static synchronized DatabaseHelper getInstance(Context context) {

    if (mInstance == null) {
        mInstance = new DatabaseHelper(context.getApplicationContext());
    }
    return mInstance;
}

@Override
public void onCreate(SQLiteDatabase db) {

    // create table stuff


}

@Override
public void onUpgrade(SQLiteDatabase db, int i, int i1) {

 // drop table stuff

    onCreate(db);
 }
}

活动：

SQLiteDatabase database = DatabaseHelper.getInstance(getApplicationContext()).getWritableDatabase();

Cursor cursor = database.query("query");

if (cursor != null) {
   while (cursor.moveToNext()) {
    // stuff
     }
   cursor.close();
   database.close();
 }

- Silambarasan

你能帮忙解决这个问题吗？ - MrinmoyMk

1

private void method() {
        Cursor cursor = query();
        if (flag == false) {  // WRONG: return before close()
            return;
        }
        cursor.close();
   }

良好的实践应该像这样：

Good practice should be like this:

    private void method() {
        Cursor cursor = null;
        try {
            cursor = query();
        } finally {
            if (cursor != null)
                cursor.close();  // RIGHT: ensure resource is always recovered
        }
    }

- Avinash Shinde

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- sjas · Accepted Answer

加粗的字体在引用中对应于您代码中的这部分：

private DatabaseManager open() throws SQLException {
    dbHelper = new DatabaseHelper(context);
    db = dbHelper.getWritableDatabase();

来源: http://www.androiddesignpatterns.com/2012/05/correctly-managing-your-sqlite-database.html

方法一: 使用抽象工厂来实例化SQLiteOpenHelper

将数据库帮助程序声明为静态实例变量，并使用抽象工厂模式来保证单例属性。下面的示例代码应该会让您对如何正确设计DatabaseHelper类有一个很好的了解。

静态工厂getInstance方法确保在任何给定时间只会存在一个DatabaseHelper。如果mInstance对象尚未初始化，则将创建一个对象。如果已经创建了一个对象，则仅返回它。

不应使用new DatabaseHelper(context)来初始化您的辅助对象。
相反，始终使用DatabaseHelper.getInstance(context)，因为它保证整个应用程序生命周期中只存在一个数据库帮助程序。

public static class DatabaseHelper extends SQLiteOpenHelper { 

  private static DatabaseHelper mInstance = null;

  private static final String DATABASE_NAME = "database_name";
  private static final String DATABASE_TABLE = "table_name";
  private static final int DATABASE_VERSION = 1;

  public static DatabaseHelper getInstance(Context ctx) {

    // Use the application context, which will ensure that you 
    // don't accidentally leak an Activity's context.
    // See this article for more information: http://bit.ly/6LRzfx
    if (mInstance == null) {
      mInstance = new DatabaseHelper(ctx.getApplicationContext());
    }
    return mInstance;
  }

  /**
   * Constructor should be private to prevent direct instantiation.
   * make call to static factory method "getInstance()" instead.
   */
  private DatabaseHelper(Context ctx) {
    super(ctx, DATABASE_NAME, null, DATABASE_VERSION);
  }
}