RecyclerView с GridLayoutManager и Picasso показывает неправильное изображение

Обновление # 1

Добавлено hasStableIds (true) и обновлено Picasso до версии 2.5.2. Это не решает проблему.

Размножение:

RecyclerView с GridLayoutManager (spanCount = 3). Элементами списка являются CardViews с ImageView внутри.

Когда все элементы не соответствуют экрану, вызывающему notifyItemChanged на одном элементе, вызывает более одного вызова onBindViewHolder(). Один вызов - это позиция из notifyItemChanged для элементов, невидимых на экране.

Вопрос:

Иногда элемент в позиции, переданный в notifyItemChanged, загружается изображением, принадлежащим элементу, который не находится на экране (скорее всего, из-за повторного использования владельца изображения), хотя я бы предположил, что если элемент остается на месте, пройденный зритель будет таким же).

Я нашел комментарий Джейка к другой проблеме здесь о вызове load(), даже если файл /uri имеет значение NULL. Изображение загружается на каждый объект onBindViewHolder.

Простой пример приложения:

git clone https://github.com/gswierczynski/recycler-view-grid-layout-with-picasso.git

Нажмите на элемент, который вызывает notifyItemChanged с параметром, равным позиции этого элемента.

Код:

public class MainActivity extends ActionBarActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        if (savedInstanceState == null) {
            getSupportFragmentManager().beginTransaction()
                    .add(R.id.container, new PlaceholderFragment())
                    .commit();
        }
    }

    public static class PlaceholderFragment extends Fragment {

        public PlaceholderFragment() {
        }

        @Override
        public View onCreateView(LayoutInflater inflater, ViewGroup container,
                                 Bundle savedInstanceState) {
            View rootView = inflater.inflate(R.layout.fragment_main, container, false);

            RecyclerView rv = (RecyclerView) rootView.findViewById(R.id.rv);

            rv.setLayoutManager(new GridLayoutManager(getActivity(), 3));
            rv.setItemAnimator(new DefaultItemAnimator());
            rv.setAdapter(new ImageAdapter());

            return rootView;
        }
    }

    private static class ImageAdapter extends RecyclerView.Adapter<ImageViewHolder> implements ClickableViewHolder.OnClickListener {

        public static final String TAG = "ImageAdapter";
        List<Integer> resourceIds = Arrays.asList(
                R.drawable.a0,
                R.drawable.a1,
                R.drawable.a2,
                R.drawable.a3,
                R.drawable.a4,
                R.drawable.a5,
                R.drawable.a6,
                R.drawable.a7,
                R.drawable.a8,
                R.drawable.a9,
                R.drawable.a10,
                R.drawable.a11,
                R.drawable.a12,
                R.drawable.a13,
                R.drawable.a14,
                R.drawable.a15,
                R.drawable.a16,
                R.drawable.a17,
                R.drawable.a18,
                R.drawable.a19,
                R.drawable.a20);

        @Override
        public ImageViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
            View v = LayoutInflater.from(parent.getContext()).inflate(R.layout.list_item, parent, false);
            return new ImageViewHolder(v, this);
        }

        @Override
        public void onBindViewHolder(ImageViewHolder holder, int position) {
            Log.d(TAG, "onBindViewHolder position: " + position + " | holder obj:" + holder.toString());
            Picasso.with(holder.iv.getContext())
                    .load(resourceIds.get(position))
                    .fit()
                    .centerInside()
                    .into(holder.iv);
        }

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

        @Override
        public void onClick(View view, int position) {
            Log.d(TAG, "onClick position: " + position);
            notifyItemChanged(position);
        }

        @Override
        public boolean onLongClick(View view, int position) {
            return false;
        }
    }

    private static class ImageViewHolder extends ClickableViewHolder {

        public ImageView iv;

        public ImageViewHolder(View itemView, OnClickListener onClickListener) {
            super(itemView, onClickListener);
            iv = (ImageView) itemView.findViewById(R.id.iv);
        }
    }
}

public class ClickableViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener, View.OnLongClickListener {
    OnClickListener onClickListener;


    public ClickableViewHolder(View itemView, OnClickListener onClickListener) {
        super(itemView);
        this.onClickListener = onClickListener;
        itemView.setOnClickListener(this);
        itemView.setOnLongClickListener(this);
    }

    @Override
    public void onClick(View view) {
        onClickListener.onClick(view, getPosition());
    }

    @Override
    public boolean onLongClick(View view) {
        return onClickListener.onLongClick(view, getPosition());
    }

    public static interface OnClickListener {
        void onClick(View view, int position);
        boolean onLongClick(View view, int position);
    }
}

ОТВЕТЫ

Ответ 1

Я потратил больше времени, чем я хотел бы признаться, чтобы обойти странности с RecyclerView и новым адаптером, который поставляется вместе с ним. Единственное, что, наконец, работало для меня с точки зрения правильных обновлений и убедило, что notifyDataSetChanges и все другие его братья и сестры не вызывали нечетное поведение:

На моем адаптере я установил

setHasStableIds(true);

В конструкторе. Затем я перепробовал этот метод:

@Override
public long getItemId(int position) {
    // return a unique id here
}

И убедитесь, что все мои объекты возвратили уникальный идентификатор.

Как вы достигаете этого, зависит от вас. Для меня данные были предоставлены с моей веб-службы в форме UUID, и я обманул, превратив части UUID в длинный, используя это:

SomeContent content = _data.get(position);
Long code = Math.abs(content.getContentId().getLeastSignificantBits());

Очевидно, что это не очень безопасный подход, но, скорее всего, он работает для моих списков, которые будут содержать < 1000 наименований. До сих пор я не сталкивался с этой проблемой.

Я рекомендую попробовать этот подход и посмотреть, работает ли он на вас. Поскольку у вас есть массив, получение уникального номера для вас должно быть простым. Возможно, попробуйте вернуть позицию фактического элемента (, а не позицию, которая передается в getItemId()), или создать уникальную длину для каждой из ваших записей и передать ее.

Ответ 2

Пробовал ли вы вызов метода mutate() в Drawable? См. здесь, например.

Ответ 3

здесь работает решение, но имеет графические сбои при вызове notifyDataSetChanged()

holder.iv.post(new Runnable() {
            @Override
            public void run() {
                     Picasso.with(holder.iv.getContext())
                              .load(resourceIds.get(position))
                              .resize(holder.iv.getWidth(), 0)
                              .into(holder.iv);
            });

он работает, потому что на этом этапе изображение имеет ширину, к сожалению, когда мне нужно обновить все флажки в узле просмотра (например, выбрать все действие), и я вызываю notifyDataSetChanged(), и эффект очень уродлив

все еще ищет лучшее решение

изменить это решение работает для меня:

    holder.iv.getViewTreeObserver().addOnGlobalLayoutListener(new ViewTreeObserver.OnGlobalLayoutListener() {
        @Override
        public void onGlobalLayout() {
            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.JELLY_BEAN)
                holder.iv.getViewTreeObserver().removeOnGlobalLayoutListener(this);
            else
                holder.iv.getViewTreeObserver().removeGlobalOnLayoutListener(this);

                Picasso.with(holder.iv.getContext())
                         .load(resourceIds.get(position))
                         .resize(holder.iv.getMeasuredWidth(), 0)
                         .into(holder.iv);
        }
    });