1. 15

    Court

    Pour améliorer les performances,

    1. Supprimer les allocations à partir onBindViewHolder
    2. Réutilisation LayoutInflater, au lieu d'obtenir un nouveau à chaque fois.
    3. Minimiser le travail répétitif dans onBindViewHolder mise en œuvre
    4. Spinner Adaptateur devrait également recycler les points de vue

    Fond

    Lors de l'utilisation d'un adaptateur pour le défilement, la chose la plus importante à faire est sûr, c'est que nous NE PAS allouer de nouveaux objets (ou réduire autant que possible).

    Tout le but d'un RecyclerView avec un Adaptateur est de faire en sorte que nous Recyclons nos objets, de sorte que le travail nécessaire au cours de défilement est minime.

    Depuis l'allocation de mémoire est très "cher", afin d'améliorer les performances de défilement, la première chose à vérifier est les allocations au cours de la onBindViewHolder. Toutes les allocations si tout doit être fait dans le onCreateViewHolder.

    Une fois que toutes les affectations sont effacées, si nous avons encore à la traîne, il est temps pour certains micro améliorations. Ceux-ci incluent l'amélioration de la qualité du code, la réutilisation de la logique de résultats, etc.

    Que faire?

    1) Supprimer les allocations à partir onBindViewHolder

    Dans le code suivant:

    final PopupMenu popup = new PopupMenu(getContext(), viewHolder.deleteMenu);
MenuInflater inflater = popup.getMenuInflater();
inflater.inflate(R.menu.delete_menu, popup.getMenu());
viewHolder.deleteMenu.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
popup.show();
}
});
popup.setOnMenuItemClickListener(new PopupMenu.OnMenuItemClickListener() {
@Override
public boolean onMenuItemClick(MenuItem item) {
if (item.getItemId() == R.id.action_delete) {
removeData(holder.getAdapterPosition());
((CartActivity) activity).checkIfEmpty();
}
return true;
}
});

    Vous avez actuellement des 3 allocations directes (nouveau) et certains indirecte dotations (gonfler). Modifier ce code afin que toutes les affectations sont dans le onCreateViewHolder. Par exemple:

    Dans onCreateViewHolder ne les allocations comme suit:

    //Allocate Listener only ONCE per recycled view 
viewHolder.deleteMenu.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
//Get needed data from the view TAG, we will set it later
final int itemPosition = (Integer)view.getTag();
//Do work only when needed - when user clicked the button
final PopupMenu popup = new PopupMenu(getContext(), viewHolder.deleteMenu);
MenuInflater inflater = popup.getMenuInflater();
inflater.inflate(R.menu.delete_menu, popup.getMenu());
popup.setOnMenuItemClickListener(new PopupMenu.OnMenuItemClickListener() {
@Override
public boolean onMenuItemClick(MenuItem item) {
//Do logic using itemPosition etc
return true;
}
});
popup.show();
}
});

    Dans onBindViewHolder lier les données pertinentes comme:

    viewHolder.deleteMenu.setTag(holder.getAdapterPosition());

    2) Réutilisation LayoutInflater, au lieu d'obtenir un nouveau à chaque fois.

    Dans le code suivant:

    public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
View rowView = LayoutInflater.from(parent.getContext()).inflate(viewType == TYPE_PRODOTTO ? R.layout.item_cart : R.layout.item_cart_sconto, parent, false);
return new ViewHolder(rowView);
}

    Vous êtes l'obtention d'un nouvel LayoutInflater à chaque fois. C'est un gaspillage. Mieux d'en obtenir un dans le constructeur de l'Adaptateur et de l'enregistrer en tant que membre.

    3) Minimiser le travail répétitif dans onBindViewHolder mise en œuvre

    Par exemple, dans le code suivant:

    viewHolder.cartoneQuantity.setVisibility(position % 2 == 1 ? View.GONE : View.VISIBLE); //Controllo da togliere in futuro
viewHolder.cartoneValue.setVisibility(position % 2 == 1 ? View.GONE : View.VISIBLE); //Controllo da togliere in futuro

    Vous êtes le calcul de la logique elle-même deux fois. Mieux calculer une seule fois, et de réutiliser le résultat:

    int cartoneVisibility = position % 2 == 1 ? View.GONE : View.VISIBLE;
viewHolder.cartoneQuantity.setVisibility(cartoneVisibility); //Controllo da togliere in futuro
viewHolder.cartoneValue.setVisibility(cartoneVisibility); //Controllo da togliere in futuro

    4)Spinner Adaptateur devrait également recycler les points de vue

    Dans CartSpinnerAdapter.getView() vous êtes également l'allocation de mémoire. Il arrive (à chaque fois * élément de liste * comte) - C'est beaucoup d'allocations. Veuillez utiliser le convertView à la place. Jetez un oeil sur ce tutoriel dzone.com/articles/android-listview-optimizations

    Bonjour, merci pour la réponse. Je suis en train de vivre le gal problème depuis que j'ai ajouté à la carte logique pour le compteur et même avec ces améliorations, le gal problème ne se résoudra pas. Donc ma question est, comment puis-je utiliser un spinner de l'adaptateur à l'intérieur d'un recyclerview adaptateur?
    Veuillez voir mon edit de la section (1) de la solution.
    Aussi, appliquer des suggestions à la CartSpinnerAdapter. Là, vous n'êtes pas à l'aide de la carte correctement, puisque vous n'avez pas recycler les points de vue. Utiliser le convertView.
    Donc, comme je l'ai dit, dans CartSpinnerAdapter. getView vous allouez de la mémoire. Il arrive à chaque fois * élément de liste * nombre de - C'est beaucoup d'allocations. Où, en fait, vous avez besoin d'0 allocations. Veuillez utiliser le convertView à la place. Jetez un oeil sur ce tutoriel dzone.com/articles/android-listview-optimizations
    dans ce cas, la "Liaison" est le setCount fonction. Dans d'autres cas, nous pourrions avoir d'autres liaisons. En tout cas, la chose la plus importante à retenir est qu'il ne faut pas allouer de la mémoire dans les fonctions qui sont appelées rapidement lors de la disposition de l'INTERFACE utilisateur. Trouver les allocations, les faire sortir, et vous êtes de l'or 🙂

    OriginalL'auteur Eyal Biran