Tengo un problema con el cual no he dado la solución y me ha vuelto loco. He buscado en stackoverflow español e ingles pero no doy con la solución.
Tengo una app donde se agregan todos las imágenes disponibles de una carpeta en especifico en un recyclerview donde se le permite al usuario eliminar imágenes una a una o a través de la selección múltiple. El problema se me presenta cuando se intenta eliminar una a una sin usar la selección múltiple. Esto ocurre de manera aleatoria en diferentes dispositivos, por lo cual se me es algo complicado de reproducir.
Las imágenes se agregan en un ArrayList
llamado ListaImagenes
y para eliminar la imagen en la posición seleccionada o correspondiente itero sobre la lista de esta manera:
File fileToDelete = new File(adaptador.getSimpleItem(position));
for (Iterator<String> it = adptador.getImageList().iterator(); it.hasNext();) {
String imgPos = it.next();
if (imgPos.equals(adaptador.getSimpleItem(position))) {
it.remove();
adaptador.removeItem(position);
if (fileToDelete.exists()) {
if (fileToDelete.delete()) {
//Muestro un anuncio al usuario notificando que la imagen fué eliminada
}
}
break;
}
}
La función removeItem
y getSimpleItem
en el adaptador son las siguiente:
public void removeItem(int position) {
notifyItemRemoved(position);
notifyItemRangeChanged(position, ListaImagenes.size());
}
public String getSimpleItem(int position) {
if (position < ListaImagenes.size()) {
return ListaImagenes.get(position);
}else{
return "";
}
}
Solución Gracias a @Andrespengineer por su respuesta.
La solución que me ha funcionado hasta el momento, ya que no se ha vuelto a presentar más este error, es la siguiente:
Antes de acceder a la posición de un elemento hago la siguiente verificación:
public String getSimpleItem(int position) {
if (position < ListaImagenes.size() && position != RecyclerView.NO_POSITION) {
//Resto de el código
}
}
Y además de eso. Ahora he cambiado el método getAdapterPosition()
por getBindingAdapterPosition()
ya que el método anterior está obsoleto.
This method is deprecated. This method is confusing when adapters nest other adapters. If you are calling this in the context of an Adapter, you probably want to call getBindingAdapterPosition() or if you want the position as RecyclerView sees it, you should call getAbsoluteAdapterPosition().
Pueden obtener mas información en este enlace
position
cuando definesfileToDelete
? Como indicas que lo ejecutas desde diferentes lugares, habría que ver qué le llega en cada caso. Lo otro: sería de utilidad incluir un fragmento máaaaas largo de la traza de error. Lo que pusiste en el título da pistas pero ver la traza ayudaría más.getAdapterPosition()
de recyclerview a través de una interfaz.listener.onDeleteItemClicked(getAdapterPosition());
El log no contiene mucha información me apunta directamente a la funcióngetSimpleItem
if (position < ListaImagenes.size())