7

所以基本上我两次查询数据库。我不明白这个错误真正来自哪里,因为我没有在任何地方关闭数据库。返回错误的代码是这样运行的。我已经检查过了,我刚刚看到了一个像我这样的案例。

BeaconHandler pullAllDB = new BeaconHandler(this);
    try {
        List<Beacon> beaconsShown = pullAllDB.getAllBeacons();
        for (final Beacon bn : beaconsShown) {
            try {
                int messageCount = pullAllDB.getMessageCount();
                Log.d("Message", messageCount + " Messages Found");
                if (messageCount > 0) {
                    //Do Something
                } else {
                    // Do Nothing
                }
            } 
            catch (Exception e) {
                e.getStackTrace();
                Log.e("Message", e.getMessage());
            }
        }
    }

以及执行查询的代码...

public int getBeaconsCount() {

    String countQuery = "SELECT * FROM " + TABLE_BASIC_BEACON;
    SQLiteDatabase db = this.getReadableDatabase();
    Cursor cursor = db.rawQuery(countQuery, null);
    cursor.close();

    // return count
    return cursor.getCount();

}

public int getMessageCount() {

    String mcountQuery = "SELECT * FROM " + MESSAGE_BEACON;
    SQLiteDatabase mdb = this.getReadableDatabase();
    Cursor mcursor = mdb.rawQuery(mcountQuery, null);
    mcursor.close();

    // return count
    return mcursor.getCount();

}
4

2 回答 2

21

如果您遇到错误,您应该发布一个 logcat。它有助于查看哪条线路导致您的问题。

来自 Android 文档。

关闭()

关闭光标,释放其所有资源并使其完全无效。

您在关闭后调用mcursor.getCount()它可能会导致错误

也许尝试这样的事情。

int count = mcursor.getCount();
mcursor.close();

// return count
return count ;
于 2013-09-20T00:02:45.200 回答
1

我在这里假设这pullAllDB是您的数据库对象,其中包含执行查询的代码。在这种情况下,在行之前List<Beacon> beaconsShown = pullAllDB.getAllBeacons();,您应该在完成运行查询之后执行pullAllDB.open();和执行类似的操作。pullAllDB.close();

所以总而言之,你的功能看起来像..

try {
    //open the database class here
    pullAllDB.open();

    List<Beacon> beaconsShown = pullAllDB.getAllBeacons();
    for (final Beacon bn : beaconsShown) {
        try {
            int messageCount = pullAllDB.getMessageCount();
            Log.d("Message", messageCount + " Messages Found");
            if (messageCount > 0) {
                //Do Something
            } else {
                // Do Nothing
            }
        } 
        catch (Exception e) {
            e.getStackTrace();
            Log.e("Message", e.getMessage());
        }

    //close the database here
    pullAllDB.close();
    }
}
于 2013-09-20T00:13:43.443 回答