Quel est le cycle de vie de l'adaptateur RecyclerView ?

Question

Je demande des images au présentateur dans l'adaptateur :

  @Override
  public void onBindViewHolder(SiteAdapter.ViewHolder holder, int position)
  {
    Site site = sites.get(position);
    holder.siteName.setText(site.getName());
    requestHolderLogo(holder, site.getLinks().getLogoUrl());
  }

  private void requestHolderLogo(final ViewHolder holder, final String logoUrl)
  {
    compositeSubscription.add(
      presenter.bitmap(logoUrl)
        .subscribe(
          bitmap -> {
            holder.siteLogo.setImageBitmap(bitmap);
            holder.siteLogo.setVisibility(View.VISIBLE);
          },
          error -> {
            holder.siteName.setVisibility(View.VISIBLE);
          })
    );
  }

Je devrais me désabonner quand ViewHolder est réutilisé. C'est facile.

Mais comment arrêter tous les abonnements lorsque la vue est détruite ? Je devrais aussi probablement annuler la référence au présentateur pour éviter les fuites de mémoire.

Answer 1

Je pense que le meilleur moyen de le faire serait de :

1. Gardez un subscription référence dans la SiteAdapter.ViewHolder
2. unsubscribe le site subscription objet dans onBindViewHolder (il est appelé lorsque le ViewHolder est réutilisé)
3. Gardez le CompositeSubscription dans votre adapter
4. Utilisez le onDetachedFromRecyclerView de votre Adapter à unsubscribe le site compositeSubscription

Comme ça :

public class SiteAdapter extends RecyclerView.Adapter<SiteAdapter.ViewHolder> {

    private CompositeSubscription compositeSubscription = new CompositeSubscription();

    // other needed SiteAdapter methods

    @Override
    public void onBindViewHolder(SiteAdapter.ViewHolder holder, int position) {
        if (holder.subscription != null && !holder.subscription.isUnsubscribed()) {
            compositeSubscription.remove(holder.subscription);
            // this will unsubscribe the subscription as well
        }
        Site site = sites.get(position);
        holder.siteName.setText(site.getName());
        requestHolderLogo(holder, site.getLinks().getLogoUrl());
    }

    private void requestHolderLogo(final SiteAdapter.ViewHolder holder, final String logoUrl) {
        holder.subscription = presenter.bitmap(logoUrl)
                .subscribe(
                        bitmap -> {
                            holder.siteLogo.setImageBitmap(bitmap);
                            holder.siteLogo.setVisibility(View.VISIBLE);
                        },
                        error -> {
                            holder.siteName.setVisibility(View.VISIBLE);
                        });
        compositeSubscription.add(holder.subscription);
    }

    @Override
    public void onDetachedFromRecyclerView(RecyclerView recyclerView) {
        compositeSubscription.unsubscribe();
    }

    public static class ViewHolder extends RecyclerView.ViewHolder {

        public Subscription subscription;

        // some holder-related stuff

        public ViewHolder(View itemView) {
            super(itemView);
            // init holder
        }
    }
}

Answer 2

Pour les autres qui ont le même problème : viewDetachedFromWindow dans l'adaptateur n'est appelé que lorsque l'adaptateur est mis à null dans le onPause (Activity, Fragment) ou le onDetachFromWindow (Activity, Fragment).

recyclerview.setAdapter(null)

Ensuite, vous obtenez viewDetachedFromWindow(...) où vous pouvez libérer vos états internes et vos abonnements. Je configurerais vos abonnements dans un bind, assurez-vous avant chaque appel bind que vous relâchez les anciens abonnements car une vue peut être recyclée.

Une autre possibilité est de gonfler une vue personnalisée au lieu d'un simple layout dans votre factory. Vous pouvez alors effectuer le nettoyage dans la vue personnalisée onDetachFromWindow(). Vous obtenez le onDetachedFromWindow également sans mettre l'adaptateur à null.

Answer 3

On peut appeler public void onViewRecycled(@NonNull VH holder)