【问题标题】：Android sqlite log reports and db.close()Android sqlite 日志报告和 db.close()
【发布时间】：2011-01-11 17:10:45
【问题描述】：

日志报告数据库或游标未关闭。我基本上有一个带有自定义表面视图的活动，并使用处理程序向活动报告。当我收到消息时，我会显示一个警报对话框并更新数据库。

private Handler handler = new Handler() {
    public void handleMessage(Message msg) {
        switch(msg.what) {
        case 1:
            dh.open();
            dh.updateEvent("id", "name", "someone");
            dh.close();
            successAlert.show();
            break;
        case 2:
            failAlert.show();
            break;
        }
    }
};

以前我没有“dh.close()”，这就是日志报告数据库/光标未关闭的时候。但是自从我添加了它之后，它需要很长时间才能完成。收到消息后，系统似乎挂起。我做错了什么还是通常需要这么长时间。我也尝试过使用带有 finally 的 try 块来关闭数据库。

编辑：

public class DatabaseHelper {

private Database dbHelper;
private SQLiteDatabase db;
private Context context;

public DatabaseHelper(Context context) {
    this.context = context;
    //database = new Database(context);
}

public void open() {
    dbHelper = new Database(context);
    db = dbHelper.getWritableDatabase();
}

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

public void updateEvent(int id, String name, int other) {
    ContentValues cv = new ContentValues();
    cv.put("id", id);
    cv.put("name", name);
    cv.put("other", other);
    db.update("stateTable", cv, "id=" + id, null);
}

public boolean checkState(int id) {
    db = dbHelper.getReadableDatabase();
    Cursor cursor = db.query("stateTable", null, null, null, null, null, null);
    cursor.moveToPosition(id - 1);
    int i = cursor.getInt(2);
    android.util.Log.d("semajhan", ": " + i);
    if (i == 1) {
        return true;
    } else {
        return false;
    }
}

}

扩展 SQLiteOpenHelper：

public class Database extends SQLiteOpenHelper {

private static final String DATABASE_NAME = "events.db";
private static final int DATABASE_VERSION = 1;
private static final String TABLE_NAME = "stateTable";
private static final String ID = "id";
private static final String NAME = "name";
private static final String OTHER = "other";
private static final String DATABASE_CREATE = "CREATE TABLE stateTable (id INT, name TEXT, other INT)";
private static final String DATABASE_UPGRADE = "DROP TABLE IF EXISTS table";

public Database(Context context) {
    super(context, DATABASE_NAME, null, DATABASE_VERSION);
    // TODO Auto-generated constructor stub
}

@Override
public void onCreate(SQLiteDatabase db) {
    // TODO Auto-generated method stub
    db.execSQL(DATABASE_CREATE);
    // added initial values
}

@Override
public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
    // TODO Auto-generated method stub
    db.execSQL(DATABASE_UPGRADE);
    onCreate(db);
}

}

01-11 13:57:41.239: ERROR/ActivityManager(61): ANR in com.semajhan.soodles  (com.semajhan.soodles/.Level1)
01-11 13:57:41.239: ERROR/ActivityManager(61): Reason: keyDispatchingTimedOut
01-11 13:57:41.239: ERROR/ActivityManager(61): Load: 1.64 / 0.56 / 0.26
01-11 13:57:41.239: ERROR/ActivityManager(61): CPU usage from 35716ms to -1ms ago:
01-11 13:57:41.239: ERROR/ActivityManager(61):   44% 862/com.semajhan.soodles: 37% user + 7.2% kernel / faults: 853 minor
01-11 13:57:41.239: ERROR/ActivityManager(61):   29% 61/system_server: 27% user + 1.9% kernel / faults: 142 minor
01-11 13:57:41.239: ERROR/ActivityManager(61):   0.2% 731/com.android.quicksearchbox: 0% user + 0.2% kernel / faults: 30 minor
01-11 13:57:41.239: ERROR/ActivityManager(61):   0.2% 707/com.android.launcher: 0.2% user + 0% kernel / faults: 30 minor
01-11 13:57:41.239: ERROR/ActivityManager(61):   0.2% 801/com.svox.pico: 0.1% user + 0.1% kernel / faults: 363 minor
01-11 13:57:41.239: ERROR/ActivityManager(61):   0% 117/com.android.systemui: 0% user + 0% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):   0% 41/adbd: 0% user + 0% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61): 99% TOTAL: 86% user + 13% kernel + 0% irq
01-11 13:57:41.239: ERROR/ActivityManager(61): CPU usage from 1969ms to 2620ms later:
01-11 13:57:41.239: ERROR/ActivityManager(61):   54% 61/system_server: 48% user + 6% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):     40% 69/SurfaceFlinger: 40% user + 0% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):     10% 92/InputDispatcher: 7.5% user + 3% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):     1.5% 62/HeapWorker: 1.5% user + 0% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):   44% 862/com.semajhan.soodles: 32% user + 12% kernel / faults: 2 minor
01-11 13:57:41.239: ERROR/ActivityManager(61):     24% 874/Thread-13: 24% user + 0% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):     23% 862/studios.soodles: 4.6% user + 18% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):     1.5% 867/Compiler: 0% user + 1.5% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):   0.8% 731/com.android.quicksearchbox: 0% user + 0.8% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61):     0.8% 732/HeapWorker: 0% user + 0.8% kernel
01-11 13:57:41.239: ERROR/ActivityManager(61): 100% TOTAL: 76% user + 23% kernel

【问题讨论】：

处理程序代码看起来不错，但是我们看不到您的 dh 是如何编码的，所以很难说是什么导致了延迟。
稍后将发布更多代码。
好的，数据库编码在哪里？除非它是 adk 的一部分，而且我在任何地方都看不到它......
Database 是 SQLiteOpenHelper 扩展类。

标签： java android sqlite

【解决方案1】：

对SQLiteOpenHelper 实例的getReadableDatabase 或getWritableDatabase 的第一次调用需要真的很长时间才能完成。每次需要查询数据库时，不应创建新的 Database 对象（您的 SQLiteOpenHelper 实例）。尝试在DatabaseHelper 中使用相同的Database 实例。

当使用SQLiteOpenHelper 时，您不想关闭SQLiteDatabase 对象以获取SQLiteOpenHelper，因为它是共享的；即getWritableDatabase 总是返回相同的SQLiteDatabase 对象。

请注意，您的 checkState 方法会泄漏游标。为了帮助防止游标泄漏，我总是在获得游标后使用try-finally。例如：

    db = dbHelper.getReadableDatabase();
    Cursor cursor = db.query("stateTable", null, null, null, null, null, null);
    try {
        cursor.moveToPosition(id - 1);
        int i = cursor.getInt(2);
        android.util.Log.d("semajhan", ": " + i);
        if (i == 1) {
                return true;
        } else {
                return false;
        }
    } finally {
        cursor.close();
    }

【讨论】：

我想我理解您试图向我解释的内容。在我的 DatabaseHelper 类中，我每次都创建一个新的 SQLiteOpenHelper 实例，您暗示不要这样做。我似乎无法弄清楚如何在我的 DatabaseHelper 类中创建一次 SQLiteOpenHelper 实例并从那里使用它。
@semajhan：为DatabaseHelper 类型的对象添加一个私有成员变量到您的活动中。在您调用 DatabaseHelper#open 的活动的 onCreate 方法覆盖内。在您调用 DatabaseHelper#close 的活动的 onDestroy 方法覆盖内。
@semajhan：实际上，我看到是您的Handler 需要访问DatabaseHelper 实例。你在为服务编程吗？如果是这样，请将 DatabaseHelper 类型的对象的私有成员变量添加到服务中。在您调用 DatabaseHelper#open 的服务的 onCreate 方法覆盖内。在您调用 DatabaseHelper#close 的服务的 onDestroy 方法覆盖内。
@semajhan：我还应该说，在这种情况下，只创建一个SQLiteOpenHelper 实例是通过只创建一个DatabaseHelper 实例来完成的。我上面的两个cmets是如何只使用一个DatabaseHelper实例的。
@semajhan：如果您有 50 个活动需要访问单个数据库，那么我建议编写一个内容提供程序。

【解决方案2】：

我想知道关于游标未关闭的潜在问题是否与this answer 中的相同：活动需要有一个onDestroy 方法来关闭DatabaseHelper。（我现在只能说你没有显示 onDestroy 方法，但这显然只是你代码的一部分。）我还写了关于这个问题的博客here。

您还写道：“但是自从我添加了 [dh.close()] 后，需要很长时间才能完成。”您是否正在对数据库进行大量写入？也许您需要使用事务来定期刷新这些写入。如果您只是从数据库中读取，那么我不知道为什么 close 调用需要很长时间才能完成。但是没有看到更多你的代码，这些只是猜测。

~~编辑：ccheneson 关于在调用dbHelper.getReadableDatabase() 之前调用db.close() 的建议值得关注。但是，您需要检查 db 是否为空。这会将checkState 更改为：~~

public boolean checkState(int id) {
    if (db != null) {
        db.close();
    }
    db = dbHelper.getReadableDatabase();
    // ... etc ....

如果事实证明这是真正的问题，请接受 ccheneson 的回答，而不是我的回答。

（上面的编辑不正确；我应该知道的更好！请参阅下文。）

编辑 1：我不知道你为什么要用这种方法重新打开数据库。假设你的DatabaseHelper.open 方法被调用，你有一个可读的和可写的数据库句柄。 checkState 不需要重新打开它来执行读取。

编辑 2：但是，SQLiteOpenHelper.getReadableDatabase 几乎总是返回从 SQLiteOpenHelper.getWriteableDatabase 返回的相同数据库句柄。您不需要显式关闭该数据库句柄； SQLiteOpenHelper 将为您关闭它。您确实需要确保在销毁您的活动时调用SQLiteOpenHelper.close（正如我在上面写的那样。）

【讨论】：

我现在正在查看您的博客和链接。我只写一行 3 列：一个 INT、TEXT 和另一个 INT。对 SQL 很陌生，但恕我直言，只写 1 行 3 列应该非常快。
所以基本上当我创建一个 SQLiteOpenHelper 实例时，我已经“打开”了数据库进行读写？对于阅读，我不需要显式调用 db.getReadableDatabase()？
@semajhan：不完全是。在您调用db.getWriteableDatabase 或db.getReadableDatabase 之前，数据库不会打开。但是调用db.getWriteableDatabase 允许您读取和写入数据库。因此，我认为您不需要在您已经调用 db.getWriteableDatabase 之后再调用 db.getReadableDatabase。
啊，有道理。因此，如果我已经调用了 getWritableDatabase()，我将永远不需要调用 getReadableDatabase()。但是假设我在 getWritableDatabase() 之后调用 getReadableDatabase()，这是否意味着我打开数据库两次并可能导致泄漏？
@semajhan：不，它不会导致泄漏。 SQLiteOpenHelper 保留对它提供给您的数据库句柄的引用。如果你重新打开数据库进行读取或写入，它只会简单地返回那个句柄（假设没有错误：你有适当的权限，磁盘未满等）。事实上，如果你查看源代码，你可以看到调用getReadableDatabase 通常会给你一个已经可写的数据库！（见android.git.kernel.org/?p=platform/frameworks/…）

【解决方案3】：

public void close() {
    dbHelper.close(); // to change to db.close(); ?
}

我想在这里你会想要关闭数据库句柄 db 而不是 dbHelper

还有你checkState方法：

public boolean checkState(int id) {
    db = dbHelper.getReadableDatabase();
    Cursor cursor = db.query("stateTable", null, null, null, null, null, null);
    cursor.moveToPosition(id - 1);
    int i = cursor.getInt(2);
    android.util.Log.d("semajhan", ": " + i);
    if (i == 1) {
        return true;
    } else {
        return false;
    }
}

你会想像这样关闭光标：

int i = cursor.getInt(2);
cursor.close();
android.util.Log.d("semajhan", ": " + i);

【讨论】：

问题是，我已经完成了所有这些。在我的旧代码中，我有 db close 而不是 dbHelper 的句柄。在我的 checkState 方法中，我还调用了 cursor.close()。但这导致了泄漏。一旦我从关闭 db 切换到 dbHelper，泄漏就消失了，但我的日志中出现了一些奇怪的错误。只是一堆数字，然后给我一个百分比：user=72% kernel=28% 之类的。
另外，当您调用db = dbHelper.getReadableDatabase(); 时，请先尝试关闭数据库：db.close(); db = dbHelper.getReadableDatabase();。在那之后，没有其他想法:)
先关闭数据库会导致强制关闭 =[
查看DDMS下的LogCat，看看有没有stack trace可以帮助我们调试
找到了 DDMS，但在 LogCat 中我没有看到任何与“堆栈跟踪”相关的内容。