Finalizing a Cursor that has not been deactivated or closed non-fatal error

Question

i\'m getting a \"Finalizing a Cursor that has not been deactivated or closed\" error on this piece of code. The code is used to fill a listview.

Since it\'s a non

Answer 1

Just had the same problem and thought to let you know - just in case....

I accidentally called my fetch routine two times and hereby "lost" the resulting cursor of the first call. This caused the error.

Answer 2

startManagingCursor(cursor);

This has fixed my problem

Answer 3

Scott,

I ran into the same problem as you. Before you close your database, i.e. "db.close()," make sure that your cursors are closed first, i.e. "mCursor.close()"

Like so:

private void updateList()
{ 
    DBAdapter db = new DBAdapter(this);
    db.open();

    //load all waiting alarm
    mCursor=db.getTitles("state<2"); 
    setListAdapter(new MyCursorAdapter(this, mCursor)); 
    registerForContextMenu(getListView()); 

    // Let's close the cursor.
    mCursor.close();
    db.close(); 
} 

You mentioned that if you closed your cursor your list view stays empty. I recommend you pass the information over to a class and copy it over (allocate the memory) then close the cursor.

Answer 4

You should not be getting that message if you close the Cursor in onStop() or onDestroy(). Please try that again. Or, call startManagingCursor() after you get the Cursor from your query, and Android will close the Cursor on its own.

Answer 5

Close the cursor object wherever you are creating it.

When you create a cursor object and done with traversing through a SQLite table then close it after it has been used. This closing of cursor prevents exception in logcat.

You will not get any exception related to finalizing the cursor that left opened.

This fixed same issue in my Application.

Answer 6

I too have been having issues with the closing the cursor:

• Closing the cursor right after setting the list view's adapter causes the cursor to close before the data gets displayed.

• Can't use startManagingCursor to manage the cursor because it has been deprecated.

• The new cursorLoader replacement for startManagingCursor seems to be overkill.

• Moving the cursor's position as suggested did not work.

• Making the task an inner class of the activity and closing the cursor in the activity's onDestroy method works sometimes but not all the time.

• Making the task an inner class of the activity and closing the cursor in the activity's onStop method seems to be working.

I also found out that I can close the database and the sqlite open helper before closing the cursor. I can even close them right after setting the list view's adapter. The data will still display.