Como ya se ha comentado Glide es una librería potente para la carga de imágenes y performance dentro de un App.
Glide en su builder pattern, utiliza Contextos y cada uno de ellos de adapta a quien lo maneja.
Por ejemplo al utilizar el getContext()
de un Fragment
, Glide se suscribe a los eventos del ciclo de vida de dicho Fragment
y tan pronto como se disparan sus eventos del ciclo de vida, digamos que se detiene, cualquier solicitud pendiente se debe pausar; y cuando se destruye, se borran todas las solicitudes pendientes. Esto significa que la descarga de la imagen se detendrá a mitad de camino y no se usarán más recursos por ese Fragment
destruido. Por otro lado si se utiliza con getActivity()
, Glide se suscribe a los eventos del ciclo de vida de la Activity
y sucede lo mismo que anterior, pero solo cuando la Activity
se detiene o se destruye.
Bien, ahora digamos que quieres almacenar una gran cantidad de imagen en cache, como por ejemplo; en las redes sociales, almacenar un numero de imágenes de perfil de tus amigos (aquellas que se obtienen desde el servidor, y se guardan en cache para cuando se interrumpa la conexión a Internet), como esos datos quieres que estén seguros para cuando se interrumpa la conexión y seguir teniendo la misma persistencia. Dichas imágenes que por lo general son thumbnails y no consumen mucha memoria, deben ser cargadas con el getActivity().getApplicationContext()
esto, como no tiene un ciclo de vida tan corto como los Activity
y los Fragment
, hara que las imágenes se guarden en cache sin interrupciones a las descargas, asegurando que estarán ahí para usarse y que luego periódicamente se ira limpiando y enviada al Garbage Collector
.
Ahora bien, en tu caso, como tu Adapter
puede estar sujeto a varios contextos y los elementos son parte del ViewHolder
. En teoría no sabes cual contexto utilizar y una de las formas mas comunes es enviar el contexto en el constructor del Adapter
. Pero esto te puede traer problemas futuros.
La solucion es la siguiente:
Primero que todo arreglar tu ViewHolder
.
public class ViewHolder extends RecyclerView.ViewHolder {
public ImageView imageView;
public ViewHolder(View itemView) {
super(itemView);
imageView = itemView.findViewById(R.id.imagen);
}
}
y tu onBinViewHolder
deberia quedar asi:
@Override
public void onBindViewHolder(ViewHolder viewHolder, int pos) {
ImageItem item = fotosLista.get(pos);
Glide.with(viewHolder.itemView.getContext()).load(item)
.thumbnail(0.5f)
.crossFade()
.diskCacheStrategy(DiskCacheStrategy.ALL)
.into(viewHolder.imageView);
}
El problema que tienes en tu ViewHolder
, es que estas buscando una referencia cada vez que el onBind
se dispara. Realmente solo debes realizar tareas que cambien el contenido o las vistas de cierta forma en el onBind
. En este caso esta bien que el item se obtenga ahi, ya que por cada elemento, su item puede ser diferente. Sin embargo, obtener la referencia de una vista, agregar algún estilo, animacion, color, listeners, digase onClick
, onTouch
, etc... que sea el mismo para todos los elementos, deberia ir en la clase del ViewHolder
, porque no tiene sentido agregarle la misma propiedad o referencia a un elemento que nunca va a cambiar en el onBind
que siempre se va a disparar, lo recomendable es que se agregue dichos elementos en la clase del Holder
por buena practica.
En teoria, esto resolveria tu problema. Glide con thumbnails, pero que pasa si aun te sigue dando problemas. Puedes utilizar unos trucos que te ayudaran a evitar OOM en caso de que siga sucediendote:
- Limpiar las vistas cada vez que se reciclan: como Glide almacena internamente las imágenes en la cache, agregando su
DiskCacheStrategy
. Esto no hace que la vista al reciclarse, por igual se recicle su referencia dicha imagen en la cache, por lo que nunca esa referencia se borrara de la cache al menos que destruyas la aplicacion. Dicho esto, puedes utilizar un truco para limpiar la cache manualmente de la siguiente manera:
Creas el método, para obtener todos los ImageViews
del holder reciclado:
private void clearAllImageViews(ViewGroup viewGroup){
for (int i = 0; i < viewGroup.getChildCount(); i++){
View view = viewGroup.getChildAt(i);
if (view instanceof ViewGroup)
clearAllImageViews((ViewGroup)view);
else if (view instanceof ImageView){
if(((Activity)viewGroup.getContext()) != null && !((Activity)viewGroup.getContext()).isDestroyed())
Glide.with(viewGroup.getContext()).clear((ImageView)view);
}
}
}
Y sobreescribes el metodo de tu adapter que recicla la vista:
@Override
public void onViewRecycled(@NonNull MainViewHolder holder) {
super.onViewRecycled(holder);
clearAllImageViews((ViewGroup)holder.itemView);
}
La idea es buscar todas las ImageView
que hay en tu holder reciclado con el itemView y limpiarlos manualmente con Glide. Saludos.