Datos fuera de sincronización entre un CursorLoader personalizado y un CursorAdapter respaldando un ListView

Fondo:

Tengo un CursorLoader encargo que trabaja directamente con la base de datos de SQLite en vez de utilizar un ContentProvider . Este cargador funciona con un ListFragment respaldado por un CursorAdapter . Hasta aquí todo bien.

Para simplificar las cosas, vamos a suponer que hay un botón Eliminar en la interfaz de usuario. Cuando el usuario hace clic en esto, onContentChanged() una fila del DB, y también llamo onContentChanged() en mi cargador. Además, en la devolución de llamada notifyDatasetChanged() , llamo notifyDatasetChanged() en mi adaptador para actualizar la interfaz de usuario.

• Actualizar la barra de progreso de AsyncTaskLoader?
• ¿Cómo funciona la cancelación de un AsyncTaskLoader?
• AsyncTaskLoader personalizado con caché
• AsyncTaskLoader no se ejecuta
• Explique AsyncTaskLoader

Problema:

Cuando los comandos de eliminación ocurren en sucesión rápida, lo que significa que onContentChanged() se llama en sucesión rápida, bindView() termina trabajando con datos obsoletos . Lo que esto significa es que se ha eliminado una fila, pero ListView sigue intentando mostrar esa fila. Esto conduce a las excepciones Cursor.

¿Qué estoy haciendo mal?

Código:

Este es un CursorLoader personalizado (basado en este consejo de la Sra. Diane Hackborn)

 /** * An implementation of CursorLoader that works directly with SQLite database * cursors, and does not require a ContentProvider. * */ public class VideoSqliteCursorLoader extends CursorLoader { /* * This field is private in the parent class. Hence, redefining it here. */ ForceLoadContentObserver mObserver; public VideoSqliteCursorLoader(Context context) { super(context); mObserver = new ForceLoadContentObserver(); } public VideoSqliteCursorLoader(Context context, Uri uri, String[] projection, String selection, String[] selectionArgs, String sortOrder) { super(context, uri, projection, selection, selectionArgs, sortOrder); mObserver = new ForceLoadContentObserver(); } /* * Main logic to load data in the background. Parent class uses a * ContentProvider to do this. We use DbManager instead. * * (non-Javadoc) * * @see android.support.v4.content.CursorLoader#loadInBackground() */ @Override public Cursor loadInBackground() { Cursor cursor = AppGlobals.INSTANCE.getDbManager().getAllCameras(); if (cursor != null) { // Ensure the cursor window is filled int count = cursor.getCount(); registerObserver(cursor, mObserver); } return cursor; } /* * This mirrors the registerContentObserver method from the parent class. We * cannot use that method directly since it is not visible here. * * Hence we just copy over the implementation from the parent class and * rename the method. */ void registerObserver(Cursor cursor, ContentObserver observer) { cursor.registerContentObserver(mObserver); } } 

Un fragmento de mi clase ListFragment que muestra las LoaderManager llamada LoaderManager ; Así como un método refresh() que llamo cuando el usuario añade / elimina un registro.

 @Override public void onActivityCreated(Bundle savedInstanceState) { super.onActivityCreated(savedInstanceState); mListView = getListView(); /* * Initialize the Loader */ mLoader = getLoaderManager().initLoader(LOADER_ID, null, this); } @Override public Loader<Cursor> onCreateLoader(int id, Bundle args) { return new VideoSqliteCursorLoader(getActivity()); } @Override public void onLoadFinished(Loader<Cursor> loader, Cursor data) { mAdapter.swapCursor(data); mAdapter.notifyDataSetChanged(); } @Override public void onLoaderReset(Loader<Cursor> loader) { mAdapter.swapCursor(null); } public void refresh() { mLoader.onContentChanged(); } 

Mi CursorAdapter es solo uno regular con newView() siendo over-ridden para devolver el diseño de fila recién inflado XML y bindView() usando el Cursor para enlazar columnas a View s en el diseño de fila.

EDITAR 1

Después de cavar en esto un poco, creo que la cuestión fundamental aquí es la forma en que CursorAdapter maneja el Cursor subyacente. Estoy tratando de entender cómo funciona.

Tome el siguiente escenario para una mejor comprensión.

1. Supongamos que CursorLoader ha terminado de cargar y devuelve un Cursor que ahora tiene 5 filas.
2. El Adapter comienza a mostrar estas filas. Mueve el Cursor a la siguiente posición y llama a getView()
3. En este punto, incluso mientras la vista de lista está en proceso de ser procesada, una fila (por ejemplo, con _id = 2) se elimina de la base de datos.
4. Aquí es donde el problema es – El CursorAdapter ha movido el Cursor a una posición que corresponde a una fila eliminada. El método bindView() todavía intenta acceder a las columnas de esta fila utilizando este Cursor , que no es válido y obtenemos excepciones.

Pregunta:

• ¿Es correcto este entendimiento? Estoy particularmente interesado en el punto 4 anterior en el que estoy haciendo la suposición de que cuando se elimina una fila, el Cursor no se actualiza a menos que yo pida que sea.
• Suponiendo que esto es correcto, ¿cómo puedo pedir a mi CursorAdapter para descartar / abortar su representación de la ListView incluso como está en progreso y pedirle que utilice el nuevo Cursor (devuelto a través de Loader#onContentChanged() y Adapter#notifyDatasetChanged() ?

PS Pregunta a los moderadores: ¿Debería cambiar esta edición a otra pregunta?

EDIT 2

Basado en la sugerencia de varias respuestas, parece que hubo un error fundamental en mi comprensión de cómo Loader s trabajo. Resulta que:

1. El Fragment o el Adapter no deben estar funcionando directamente en el Loader .
2. El Loader debe supervisar todos los cambios en los datos y sólo debe dar al Adapter el nuevo Cursor en onLoadFinished() siempre que cambie los datos.

Armado con esta comprensión, intenté los cambios siguientes. – Ninguna operación en el Loader . El método de actualización no hace nada ahora.

Además, para depurar lo que está pasando dentro de Loader y el ContentObserver , me ocurrió lo siguiente:

 public class VideoSqliteCursorLoader extends CursorLoader { private static final String LOG_TAG = "CursorLoader"; //protected Cursor mCursor; public final class CustomForceLoadContentObserver extends ContentObserver { private final String LOG_TAG = "ContentObserver"; public CustomForceLoadContentObserver() { super(new Handler()); } @Override public boolean deliverSelfNotifications() { return true; } @Override public void onChange(boolean selfChange) { Utils.logDebug(LOG_TAG, "onChange called; selfChange = "+selfChange); onContentChanged(); } } /* * This field is private in the parent class. Hence, redefining it here. */ CustomForceLoadContentObserver mObserver; public VideoSqliteCursorLoader(Context context) { super(context); mObserver = new CustomForceLoadContentObserver(); } /* * Main logic to load data in the background. Parent class uses a * ContentProvider to do this. We use DbManager instead. * * (non-Javadoc) * * @see android.support.v4.content.CursorLoader#loadInBackground() */ @Override public Cursor loadInBackground() { Utils.logDebug(LOG_TAG, "loadInBackground called"); Cursor cursor = AppGlobals.INSTANCE.getDbManager().getAllCameras(); //mCursor = AppGlobals.INSTANCE.getDbManager().getAllCameras(); if (cursor != null) { // Ensure the cursor window is filled int count = cursor.getCount(); Utils.logDebug(LOG_TAG, "Count = " + count); registerObserver(cursor, mObserver); } return cursor; } /* * This mirrors the registerContentObserver method from the parent class. We * cannot use that method directly since it is not visible here. * * Hence we just copy over the implementation from the parent class and * rename the method. */ void registerObserver(Cursor cursor, ContentObserver observer) { cursor.registerContentObserver(mObserver); } /* * A bunch of methods being overridden just for debugging purpose. * We simply include a logging statement and call through to super implementation * */ @Override public void forceLoad() { Utils.logDebug(LOG_TAG, "forceLoad called"); super.forceLoad(); } @Override protected void onForceLoad() { Utils.logDebug(LOG_TAG, "onForceLoad called"); super.onForceLoad(); } @Override public void onContentChanged() { Utils.logDebug(LOG_TAG, "onContentChanged called"); super.onContentChanged(); } } 

Y aquí hay fragmentos de mi Fragment y LoaderCallback

 @Override public void onActivityCreated(Bundle savedInstanceState) { super.onActivityCreated(savedInstanceState); mListView = getListView(); /* * Initialize the Loader */ getLoaderManager().initLoader(LOADER_ID, null, this); } @Override public Loader<Cursor> onCreateLoader(int id, Bundle args) { return new VideoSqliteCursorLoader(getActivity()); } @Override public void onLoadFinished(Loader<Cursor> loader, Cursor data) { Utils.logDebug(LOG_TAG, "onLoadFinished()"); mAdapter.swapCursor(data); } @Override public void onLoaderReset(Loader<Cursor> loader) { mAdapter.swapCursor(null); } public void refresh() { Utils.logDebug(LOG_TAG, "CamerasListFragment.refresh() called"); //mLoader.onContentChanged(); } 

Ahora, siempre que hay un cambio en el DB (fila agregada / suprimida), el método onChange() del ContentObserver debe ser llamado – correcto? No veo que esto suceda. Mi ListView nunca muestra ningún cambio. La única vez que veo algún cambio es si llamo explícitamente onContentChanged() al Loader .

¿Qué pasa aquí?

EDIT 3

Ok, así que volví a escribir mi Loader para extender directamente desde AsyncTaskLoader . Aún no veo mis cambios de base de datos actualizados, ni el método onContentChanged() de mi Loader se llama cuando inserto / elimino una fila en el DB 🙁

Sólo para aclarar algunas cosas:

1. He utilizado el código para CursorLoader y acaba de modificar una sola línea que devuelve el Cursor . Aquí, he reemplazado la llamada a ContentProvider con mi código DbManager (que a su vez utiliza DatabaseHelper para realizar una consulta y devolver el Cursor ).

   Cursor cursor = AppGlobals.INSTANCE.getDbManager().getAllCameras();

2. Mis inserciones / actualizaciones / eliminaciones en la base de datos suceden desde cualquier lugar y no a través de Loader . En la mayoría de los casos las operaciones de DB están ocurriendo en un Service fondo, y en un par de casos, de una Activity . Utilizo directamente mi clase de DbManager para realizar estas operaciones.

Lo que todavía no obtengo es – ¿ quién le dice a mi Loader que una fila ha sido añadida / eliminada / modificada? En otras palabras, ¿dónde se ForceLoadContentObserver#onChange() ? En mi Cargador, inscribo a mi observador en el Cursor :

 void registerContentObserver(Cursor cursor, ContentObserver observer) { cursor.registerContentObserver(mObserver); } 

Esto implicaría que la responsabilidad está en el Cursor para notificar mObserver cuando ha cambiado. Pero, a continuación, AFAIK, un 'Cursor' no es un objeto "en vivo" que actualiza los datos que está apuntando a como y cuando los datos se modifican en el DB.

Esta es la última iteración de mi cargador:

 import android.content.Context; import android.database.ContentObserver; import android.database.Cursor; import android.support.v4.content.AsyncTaskLoader; public class VideoSqliteCursorLoader extends AsyncTaskLoader<Cursor> { private static final String LOG_TAG = "CursorLoader"; final ForceLoadContentObserver mObserver; Cursor mCursor; /* Runs on a worker thread */ @Override public Cursor loadInBackground() { Utils.logDebug(LOG_TAG , "loadInBackground()"); Cursor cursor = AppGlobals.INSTANCE.getDbManager().getAllCameras(); if (cursor != null) { // Ensure the cursor window is filled int count = cursor.getCount(); Utils.logDebug(LOG_TAG , "Cursor count = "+count); registerContentObserver(cursor, mObserver); } return cursor; } void registerContentObserver(Cursor cursor, ContentObserver observer) { cursor.registerContentObserver(mObserver); } /* Runs on the UI thread */ @Override public void deliverResult(Cursor cursor) { Utils.logDebug(LOG_TAG, "deliverResult()"); if (isReset()) { // An async query came in while the loader is stopped if (cursor != null) { cursor.close(); } return; } Cursor oldCursor = mCursor; mCursor = cursor; if (isStarted()) { super.deliverResult(cursor); } if (oldCursor != null && oldCursor != cursor && !oldCursor.isClosed()) { oldCursor.close(); } } /** * Creates an empty CursorLoader. */ public VideoSqliteCursorLoader(Context context) { super(context); mObserver = new ForceLoadContentObserver(); } @Override protected void onStartLoading() { Utils.logDebug(LOG_TAG, "onStartLoading()"); if (mCursor != null) { deliverResult(mCursor); } if (takeContentChanged() || mCursor == null) { forceLoad(); } } /** * Must be called from the UI thread */ @Override protected void onStopLoading() { Utils.logDebug(LOG_TAG, "onStopLoading()"); // Attempt to cancel the current load task if possible. cancelLoad(); } @Override public void onCanceled(Cursor cursor) { Utils.logDebug(LOG_TAG, "onCanceled()"); if (cursor != null && !cursor.isClosed()) { cursor.close(); } } @Override protected void onReset() { Utils.logDebug(LOG_TAG, "onReset()"); super.onReset(); // Ensure the loader is stopped onStopLoading(); if (mCursor != null && !mCursor.isClosed()) { mCursor.close(); } mCursor = null; } @Override public void onContentChanged() { Utils.logDebug(LOG_TAG, "onContentChanged()"); super.onContentChanged(); } } 

 synchronized(this) { // this is ListFragment or ListView. notifyDatasetChanged(); } 

 @Override public void onResume() { super.onResume(); // Always call the superclass method first if (some_condition) { getSupportLoaderManager().getLoader(LOADER_ID).onContentChanged(); } }