1

Я не могу разобраться в чем причина тормозов при прокрутке моего списка. Прокрутка вызывается методом mRecyclerView.smoothScrollToPosition(position); При прокрутке список дергается (время отрисовки фрейма >16ms)

AlphaAdapter.java

public class AlphaAdapter extends RecyclerView.Adapter<AlphaAdapter.ViewHolder> {
    private static final String TAG = "AlphaAdapter";
    private Context mContext;
    private List<PrimaryWeapon> mGunList;
    private boolean isAnimate;

    private final int VIEW_TYPE_NULL        = 0;
    private final int VIEW_TYPE_ITEM        = 1;


    public AlphaAdapter(Context mContext, List<PrimaryWeapon> mGunList) {
        this.mContext = mContext;
        this.mGunList = mGunList;
    }

    public AlphaAdapter(Context mContext, List<PrimaryWeapon> mGunList, boolean a) {
        this.mContext = mContext;
        this.mGunList = mGunList;
        this.isAnimate = a;
    }

    @Override
    public int getItemViewType(int position) {
        try {
            if (mGunList.get(position).getNameWeapon().equals(""))
                return VIEW_TYPE_NULL;
            else return VIEW_TYPE_ITEM;
        } catch (Exception e) {
            e.printStackTrace();
            Log.e("AlphaAdapter", String.valueOf(position));
            return VIEW_TYPE_NULL;
        }
    }

    @Override
    public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View view = null;
        if (viewType == VIEW_TYPE_ITEM)
            view = LayoutInflater.from(mContext).inflate(R.layout.item_game, parent, false);
        else
            view = LayoutInflater.from(mContext).inflate(R.layout.item_null, parent, false);
        return new ViewHolder(view);
    }

    @Override
    public void onViewDetachedFromWindow(ViewHolder holder) {
        super.onViewDetachedFromWindow(holder);
        holder.itemView.clearAnimation();
    }

    @Override
    public boolean onFailedToRecycleView(ViewHolder holder) {
        return true;
    }

    @TargetApi(Build.VERSION_CODES.JELLY_BEAN)
    @Override
    public void onBindViewHolder(final ViewHolder holder, int position) {
        try {
            long startTime = System.currentTimeMillis();
            if (!getWeapon(position).getNameWeapon().equals("")) {
                if (isAnimate) {
                    if (mLastAnimPosition < position) {
                        mLastAnimPosition = position;
                        Animation a = AnimationUtils.loadAnimation(mContext, R.anim.item_game_anim);
                        holder.itemView.startAnimation(a);
                        a.setAnimationListener(new Animation.AnimationListener() {
                            @Override
                            public void onAnimationStart(Animation animation) {

                            }

                            @Override
                            public void onAnimationEnd(Animation animation) {
                            }

                            @Override
                            public void onAnimationRepeat(Animation animation) {

                            }
                        });
                    }
                }

               Picasso.with(mContext)
                        .load("file:///android_asset/" + getWeapon(position).getImagePath())
                        .placeholder(R.drawable.loading)
                        .error(R.drawable.load_fail)
                        .into(holder.image);
                holder.main.setText(getWeapon(position).getNameWeapon());
                holder.desc.setText(getWeapon(position).getNameSkin());

            }
            Log.i(TAG, String.valueOf(System.currentTimeMillis() - startTime) + "ms onBind (" + String.valueOf(position) + ");");
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    @Override
    public long getItemId(int position) {
        return position;
    }

    private PrimaryWeapon getWeapon(int position) {
        return mGunList.get(position);
    }

    @Override
    public int getItemCount() {
        return mGunList.size();
    }

    class ViewHolder extends RecyclerView.ViewHolder {

        TextView main, desc;
        ImageView image;

        ViewHolder(View itemView) {
            super(itemView);
            main = (TextView) itemView.findViewById(R.id.item_textMain);
            desc = (TextView) itemView.findViewById(R.id.item_textDescription);
            image = (ImageView) itemView.findViewById(R.id.item_image);
        }
    }

}

Layout элемента, который участвует в адаптере состоит из LinearLayout, ImageView и двух TextView.

Сначала думал, что проблема в загрузке изображений, пробовал загружать их в AsyncTask, загружать bitmap через Picasso и без, но после вовсе убрал их из разметки, и список все равно лагает.

Сделал дамп ОЗУ после скрола, показывает, что в памяти существует 71 экзепляр ViewHolder, после добавления

@Override
    public boolean onFailedToRecycleView(ViewHolder holder) {
        return true;
    }

это значение уменьшилось до 15-16, хотя норма для RecyclerView - 5 экземпляров, а при прокрутке он должен их переопределять, но не создавать новые. Screenshot 1

5
  • попробуйте закомментить код с анимацией, мне кажется в этом направлении копать надо 29 янв 2017 в 17:46
  • Анимация включается при истинном значении флажка, в данной ситуации этот флажок эквивалентен false.
    – Nikita G.
    29 янв 2017 в 17:49
  • У вас R.layout.item_null имеет содержательную ценность, или это заглушка нулевой высоты? Если второе, то попробуйте почистить ваш список и убрать оттуда getNameWeapon().equals("") заранее.
    – tse
    29 янв 2017 в 18:45
  • Это заглушка, хорошо, попробую. Мне буквально 5 минут назад другой пользователь (stackoverflow.com/a/41924445/5860004) подсказал, что не нужно передавать контекст в конструкторе, и это дало прирост производительности, но без картинок. С картинками всё снова дергается и лагает.
    – Nikita G.
    29 янв 2017 в 18:46
  • 1
    посмотрите, думаю Вам будет интересно. events.yandex.ru/lib/talks/3456 29 янв 2017 в 20:03

1 ответ 1

0

Решение проблемы заключалось в отмене предыдущего запроса к Picasso, подробнее по ссылке.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.