Android Recyclerview插入过多数据就会导致item重复，乱序

Question

item数量超过6条就会出现item重复，乱序

代码如下

fragment布局

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:orientation="vertical"
    tools:context="invinciblejoe.com.lightingbuy.main.LightingFragment">

    <android.support.v7.widget.RecyclerView
        android:id="@+id/commodity_list"
        android:layout_width="match_parent"
        android:layout_height="wrap_content" />
</LinearLayout>

recyclerview item布局

<?xml version="1.0" encoding="utf-8"?>
<android.support.v7.widget.CardView
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/invinciblejoe.com.lightingbuy"
    android:layout_width="match_parent"
    android:layout_height="150dp"
    android:layout_marginTop="10dp"
    android:layout_marginBottom="10dp"
    android:layout_marginLeft="10dp"
    android:layout_marginRight="10dp"
    app:cardBackgroundColor="@color/orange"
    app:cardCornerRadius="20dp">

    <LinearLayout
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:orientation="vertical">
        
        <TextView
            android:id="@+id/pic"
            android:layout_width="match_parent"
            android:layout_height="120dp"
            android:gravity="center"
            android:textSize="50sp"
            />

        <TextView
            android:clickable="true"
            android:id="@+id/name"
            android:layout_width="match_parent"
            android:layout_height="60dp"
            android:background="@color/orange"
            android:textColor="@android:color/white" />

    </LinearLayout>

</android.support.v7.widget.CardView>

recycerview adapter

public class LightingRVAdapter extends RecyclerView.Adapter {

    private List<Commodity> mlist;
    private Context mContext;
    private LightingViewHolder viewHolder;

    public LightingRVAdapter(Context mContext, List<Commodity> mlist) {
        this.mContext = mContext;
        this.mlist = mlist;
    }

    @Override
    public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View v = LayoutInflater.from(parent.getContext()).inflate(R.layout.cardview_lighting, parent, false);
        viewHolder = new LightingViewHolder(v);
        return viewHolder;
    }

    @Override
    public void onBindViewHolder(RecyclerView.ViewHolder holder, int position) {

        Commodity c = mlist.get(position);
        viewHolder.mImageView.setText(c.getName());
        viewHolder.mTextView.setText(String.valueOf(c.getPrice_discont()));
    }

    @Override
    public int getItemCount() {
        return mlist == null ? 0 : mlist.size();
    }

    private class LightingViewHolder extends RecyclerView.ViewHolder
        implements View.OnClickListener
    {
        public TextView mTextView;

        public TextView mImageView;

        public LightingViewHolder(View v )
        {
            super(v);
            mTextView = (TextView) v.findViewById(R.id.name);
            mImageView = (TextView) v.findViewById(R.id.pic);
            mImageView.setOnClickListener(this);
        }

        @Override
        public void onClick(View view) {
            switch (view.getId()) {
                case R.id.pic :
                    mTextView.setText("OnChlic");
                    break;
            }
        }
    }

}

Answer 1

はい、混乱するのは奇妙なことです。ビュー ホルダーの使用法が間違っています...
OnCreateViewHolder は新しいインスタンスを直接作成し、新しい項目が必要なときに呼び出されます。
このように書くと、アダプター全体で同じホルダーが使用されます。表示がキャッシュされるため、2番目の項目が混乱することはありません。
OnBind は、作成したホルダー インスタンスを提供し、そこでデータを操作します。
同時に、上記のメソッドの戻り値の型はカスタム ホルダーである必要があります。

Answer 2

繰り返しの異常とはどういう意味ですか? 位置 7 で、表示されているデータは以前に表示されたものです。
onBindViewHolder() メソッドでデバッグして、関連する変数の値を確認することをお勧めします。

コードを見ると問題ないようです。あなたが言及した「挿入」など、他のコードはありますか

Answer 3

この行が鍵です
これが正しい書き方です

リーリー

修正後の実装インターフェースはこんな感じ

リーリー リーリー

次に、onBindViewHolder でデータを直接操作します

袁文海、訂正ありがとうございます、ありがとうございます