Cursor schließen?

so.r.u · 2010-03-08T12:56:18+0100

Hallo!

Ich habe eine Datenbankanfrage in meinem Programm, bekommen aber ständig IllegalStateExceptions wegen einem Cursor. Dabei schließe ich nach jeder Anfrage diesen Cursor:

Code:

 public WayPoint getLatestWayPoint(int route_id) {
        Cursor c = null;
        WayPoint w = null;
        try {
            c = db.query(WAYPOINTS, new String[] { "max(timestamp)" },
                    "route_id = ?", new String[] { route_id + "" }, null, null,
                    " timestamp ASC");

            (...)

            w = new WayPoint(id, route_id, timestamp, latitude, longitude, comment);
        } catch (SQLException e) {
            Log.e(TAG, "Database error fetching latest waypoint for route with id = " + route_id); }
        if ( c != null ) {
            c.close(); 
        }
        return w;
    }

Der c.close-Fall wird auch jedes Mal durchlaufen, nur danach ist c immer noch einSQLiteCursor, sollte es dann nicht null sein? Woran könnte das denn liegen? Ach ja das Ergebnis der Abfrage ist richtig.

SeraphimSerapis · 2010-03-08T13:13:27+0100

ich glaube c.close() entfernt nicht die referenz auf den db.query(..), sondern deaktiviert ihn nur dauerhaft.

so.r.u · 2010-03-08T13:36:01+0100

Aber was heißt deaktiviert? Laut API macht close:

Closes the Cursor, releasing all of its resources and making it completely invalid.

Deactivate:

Deactivates the Cursor, making all calls on it fail until requery() is called.

Was muss man dann aufrufen, damit der Cursor "finalized" ist, d.h. dass alle Ressourcen freigegeben werden?

SeraphimSerapis · 2010-03-08T15:46:29+0100

Steht doch da - close reicht - am besten innerhalb einer finally.

so.r.u · 2010-03-08T15:53:51+0100

Aber close wird ja aufgerufen, scheinbar is aber noch nicht alles freigegeben, sonst würds ja nicht zu einer IllegalStateException kommen, oder?

so.r.u · 2010-04-06T18:41:30+0200

Hab endlich den Fehler gefunden, ich hab den Cursor zweimal benutzt, d.h. ungefähr sowas gemacht

...

Code:

c = db.query(WAYPOINTS, new String[] { "max(timestamp)" },
                    "route_id = ?", new String[] { route_id + "" }, null, null,
                    " timestamp ASC");
if (c.moveToFirst()) {
     int timeCol = c.getColumnIndex("max(timestamp)");
     last_time = c.getString(timeCol);
}
c = db.query(WAYPOINTS, null, "route_id = ? and timestamp = ?",
                    new String[] { route_id + "", last_time + "" }, null, null,
                    " timestamp ASC");

...

D.h. einem Cursor zwei Queries zugewiesen, wenn man vor dem zweiten query() c.close() aufruft (und am Ende nochmal) dann klappts!
Oder am besten zwei Cursor erstellen, jeweils einen für ein query verwenden, und wie SeraphimSerapis vorgeschlagen hat das in ein finally verpacken:

Code:

Cursor c = null;
Cursor c2 = null;

try {
    String last_time = "";
    c = db.query(WAYPOINTS, new String[] { "max(timestamp)" },
                    "route_id = ?", new String[] { route_id + "" }, null, null,
                    " timestamp ASC");
    if (c.moveToFirst()) {
        int timeCol = c.getColumnIndex("max(timestamp)");
        last_time = c.getString(timeCol);
     }
     c2 = db.query(WAYPOINTS, null, "route_id = ? and timestamp = ?",
                    new String[] { route_id + "", last_time + "" }, null, null,
                    " timestamp ASC");
     if (c2.moveToFirst()) {
     //do something
} catch (SQLException e) {
     Log.e(TAG, "e.getMessage()); }
finally{
     if ( c != null ) { c.close(); }
     if ( c2 != null ) { c2.close(); }
}

Quick results: Click search for more Results…

Cursor schließen?

so.r.u

Neues Mitglied

SeraphimSerapis

Enthusiast

so.r.u

Neues Mitglied

SeraphimSerapis

Enthusiast

so.r.u

Neues Mitglied

so.r.u

Neues Mitglied

Ähnliche Themen