12

更新 #1

hasStableIds(true) を追加し、Picasso をバージョン 2.5.2 に更新しました。問題は解決しません。

再生：

RecyclerView と GridLayoutManager (spanCount = 3)。リスト アイテムは、内部に ImageView を含む CardViews です。

すべてのアイテムが画面に収まらない場合、1 つのアイテムで notifyItemChanged を呼び出すと、onBindViewHolder() が複数回呼び出されます。1 つの呼び出しは、画面に表示されていない項目の notifyItemChanged 他のものからの位置です。

問題：

notifyItemChanged に渡された position のアイテムに、画面上にないアイテムに属する画像が読み込まれることがあります (ビューホルダーのリサイクルが原因である可能性が最も高いですが、アイテムが所定の位置に残っている場合、渡されたビューホルダーは同じだろう）。

file/uri が null の場合でも load() を呼び出すことに関する他の問題に関する Jake のコメントを見つけました。画像はすべての 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);
    }
}
4

3 に答える 3

6

RecyclerView私は、奇妙な点とそれに付属する新しいアダプターを回避するために、認めたくないほど多くの時間を費やしました。正しい更新とnotifyDataSetChanges、他のすべての兄弟が奇妙な動作を引き起こさなかったことを確認するという点で、最終的に私にとってうまくいった唯一のことは次のとおりです。

私のアダプターでは、設定しました

setHasStableIds(true);

コンストラクターで。次に、このメソッドをオーバーライドしました。

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

そして、すべてのアイテムが一意の ID を返すようにしました。

これをどのように達成するかはあなた次第です。私の場合、データは Web サービスから UUID の形式で提供され、これを使用して UUID の一部を long に変換することでごまかしました。

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

明らかに、これはあまり安全なアプローチではありませんが、1000 個未満のアイテムを含む私のリストではうまくいく可能性があります。これまでのところ、私はそれで問題に遭遇したことはありません。

私がお勧めするのは、このアプローチを試して、それがうまくいくかどうかを確認することです. 配列があるので、一意の番号を取得するのは簡単です。おそらく、実際のアイテムの位置 (に渡された位置ではなくgetItemId()) を返すか、レコードごとに一意の long を作成して渡してみてください。

于 2015-03-27T23:27:11.397 に答える
0

Drawable でメソッドを呼び出してみmutate()ましたか? たとえば、ここを参照してください。

于 2015-03-30T14:40:16.847 に答える