Закрытие курсора вызывает StaleDataException на фоновом фрагменте с помощью CursorAdapter
Это продолжение моего вопроса здесь: Как закрыть курсор, используемый в цикле for
Ответы разрешили предупреждение "Курсор завершен без предварительного закрытия", но оно вызвало исключение StaleDataException в очень конкретной ситуации.
Если список был прокручен, этот курсор закрыт...
Cursor cursor = null;
cursor = (Cursor) getListView().getItemAtPosition(n);
//do something
if(cursor != null) {
cursor.close();
}
и фрагмент фоновый я получаю следующую ошибку:
09-15 21:16:58.240: E/test(21621): Exception
FATAL EXCEPTION: main
android.database.StaleDataException: Attempted to access a cursor after it has been closed.
at android.database.BulkCursorToCursorAdaptor.throwIfCursorIsClosed(BulkCursorToCursorAdaptor.java:64)
at android.database.BulkCursorToCursorAdaptor.getCount(BulkCursorToCursorAdaptor.java:70)
at android.database.AbstractCursor.moveToPosition(AbstractCursor.java:196)
at android.database.CursorWrapper.moveToPosition(CursorWrapper.java:162)
at android.support.v4.widget.CursorAdapter.getItemId(CursorAdapter.java:225)
at android.widget.AbsListView.onSaveInstanceState(AbsListView.java:1782)
at android.view.View.dispatchSaveInstanceState(View.java:11950)
at android.view.ViewGroup.dispatchFreezeSelfOnly(ViewGroup.java:2685)
at android.widget.AdapterView.dispatchSaveInstanceState(AdapterView.java:782)
at android.view.ViewGroup.dispatchSaveInstanceState(ViewGroup.java:2671)
at android.view.View.saveHierarchyState(View.java:11933)
at android.support.v4.app.FragmentManagerImpl.saveFragmentViewState(FragmentManager.java:1608)
at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1004)
at android.support.v4.app.FragmentManagerImpl.removeFragment(FragmentManager.java:1212)
at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:639)
at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1478)
at android.support.v4.app.FragmentManagerImpl$1.run(FragmentManager.java:446)
at android.os.Handler.handleCallback(Handler.java:615)
at android.os.Handler.dispatchMessage(Handler.java:92)
at android.os.Looper.loop(Looper.java:137)
at android.app.ActivityThread.main(ActivityThread.java:4793)
Похоже, что этот вопрос относится к аналогичной проблеме: объединение курсоров во время onLoadFinished() вызывает StaleDataException после поворота, но это относится к объединению курсоров и предлагает использовать swapCursor, я не уверен, как бы я применил это в этой ситуации.
Мои мысли таковы getListView().getItemAtPosition(n)
должен возвращать ссылку на курсор, а не новый курсор, и поэтому, когда действие является фоновым и пытается получить доступ к теперь закрытому курсору, сохраняя при этом состояние фрагмента, происходит сбой. Как упоминалось ранее, он падает только при прокрутке списка, я не уверен, почему это должно повлиять на него.
Как правильно закрыть курсор, не вызывая сбой?
РЕДАКТИРОВАТЬ код в ответ на комментарий с просьбой посмотреть, как был загружен курсор:
String[] desired_columns = {
MediaStore.Audio.Media._ID, //this column is needed, even though it won't be displayed so the cursor can populate the listview
MediaStore.Audio.Media.ARTIST,
MediaStore.Audio.Media.ALBUM,
MediaStore.Audio.Media.TITLE,
MediaStore.Audio.Media.DURATION
};
String selectionStatement = MediaStore.Audio.Media.IS_MUSIC + " != ? AND " + MediaStore.Audio.Media.DURATION + " > ?";
String[] selectionArguments = new String[2];
selectionArguments[0] = "0";
selectionArguments[1] = "7000";
Cursor myCursor = getActivity().getContentResolver().query( MediaStore.Audio.Media.EXTERNAL_CONTENT_URI,
desired_columns,
selectionStatement, //selection criteria
selectionArguments, //selection arguments like with SQL PDO use ? in criteria and put user input here to avoid sql injection
MediaStore.Audio.AudioColumns.ARTIST + " ASC"); //sort order of results
//moveToFirst() returns false if the cursor is empty
if (myCursor != null && myCursor.moveToFirst()) {
customCursorAdapter myCursorAdapter = new customCursorAdapter(getActivity(), myCursor, 0);
setListAdapter(myCursorAdapter);
getListView().setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
getListView().setSelector(R.drawable.list_selection_colouring);
}
2 ответа
Я нашел решение, которое избегает устаревшего manageQuery, а также не использует CursorLoaders.
Решением было закрыть курсор внутри onDestroy(). Я пытался закрыть его в OnPause, но это не помогло. Глядя на жизненный цикл фрагмента http://developer.android.com/guide/components/fragments.html onDestroy - это почти последний вызванный метод, поэтому состояние фрагментов уже будет сохранено, что делает безопасным закрытие курсора.
Вот код, который я использовал:
@Override
public void onDestroy() {
super.onDestroy();
if (cursor != null) {
cursor.close();
}
}
Попробуй с этим
Cursor cursor = null;
try {
cursor = (Cursor) getListView().getItemAtPosition(n);
//do something
} finally {
if (cursor != null) {
cursor.close();
}
}