RecycerView为每十分之一的iconbutton设置状态，而不仅仅是按下的iconbutton

Question

我的RecycerView包含图像按钮，这些按钮是星形按钮。因此，当用户单击imagebutton时，它必须变成黄色。当用户再次单击此图像按钮时，它必须变成灰色。我将按钮的位置和状态保存在HashMap中。状态可以是-1或1。当它是1时，按钮变成黄色；当它是-1时，按钮变成灰色。

我做的第一件事是为onBindViewHolder方法中的每个位置设置一个状态，就像为hashMap中的每个位置设置一个-1，这意味着按钮没有被选中并且是灰色的，所以位置是键，状态（-1或1）是hashMap中的值。

 public void onBindViewHolder(final MyViewHolder holder, final int position){
      if(!hashMapStarButtons.containsKey(position)){
         hashMapStarButtons.put(position), -1);
      }

因此，当按下图像按钮时，在

onBindViewHolder（最终的MyViewHolder持有者，最终的int位置）

方法我获取位置，我检查状态并更改它

holder.starButton.setOnClickListener(new View.OnClickListener() {
        public void onClick(View view) {
           int currentStat = hashMapStarButtons.get(position);   
           currentStat = currentStat * (-1);

           if(currentStat==1){
                holder.starButton.setImageDrawable(ContextCompat.getDrawable(view.getContext(),android.R.drawable.btn_star_big_on));
           }else{
                holder.starButton.setImageDrawable(ContextCompat.getDrawable(view.getContext(),android.R.drawable.btn_star_big_off));
           }

这里的问题是，当我按下位置2的按钮时，该按钮必须变成黄色，因为状态变为1，recycerview也会将位置12、22、32的按钮变成黄色。当我按下位置13上的按钮时，位置3、23、33上的按钮的颜色也会改变。真的很奇怪。当我检查HAshMap中的所有值时，只有被按下的按钮的状态发生了变化，而其他按钮的状态则没有变化。

这是我的持有者类

class MyViewHolder extends   RecyclerView.ViewHolder {

    View view;

    // final
    private ImageButton starButton;
    private TextView movie;


    public MyViewHolder(View viewItem){
        super(viewItem);

        starButton = (ImageButton)viewItem.findViewById(R.id.starButton);
        movie = (TextView)viewItem.findViewById(R.id.movie);

    }


    public void setMovie(String movie){
        movie.setText(movie);
    }

}

这是适配器类

public class MyAdapter extends RecyclerView.Adapter<MyAdapter.MyViewHolder>{

private List<String> list;
private HashMap<Integer, Integer> hashMapStarButtons;

public MyAdapter(List<Movie> list){
    this.list = list;
    hashMapStarButtons = new HashMap<Integer, Integer>();

}

@Override
public MyViewHolder onCreateViewHolder(ViewGroup parent, int viewType){
    // Hole das Layout für die Row
    return new MyViewHolder(LayoutInflater.from(parent.getContext()).inflate(R.layout.movie_row, parent, false));
}

// onBindViewHolder is called for every single item in the RecyclerView
@Override
public void onBindViewHolder(final MyViewHolder holder, final int position){
    Movie movie = list.get(position);
    holder.setMovie(movie.getMovie());

    if(!hashMapStarButtons.containsKey(position)){
        hashMapStarButtons.put(position, -1);
    }

    holder.starButton.setOnClickListener(new View.OnClickListener() {
        public void onClick(View view) {
            int currentStat = hashMapStarButtons.get(position);
            currentStat = currentStat * (-1);

            if(currentStat==1){
                holder.starButton.setImageDrawable(ContextCompat.getDrawable(view.getContext(),android.R.drawable.btn_star_big_on));
            }else{
                holder.starButton.setImageDrawable(ContextCompat.getDrawable(view.getContext(),android.R.drawable.btn_star_big_off));
            }



        }
    });

}

}

下面是iam从firebase加载数据的活动

public class MyActivity  extends AppCompatActivity {

private FirebaseAuth mAuth;
private DatabaseReference mStatusDB;

// everything for RecyclerView
private RecyclerView recyclerView;
private List<String> list;
private MyAdapter adapter;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_mymovies);

    mStatusDB = FirebaseDatabase.getInstance().getReference().child("movies");

    LinearLayoutManager linearLayoutManager = new LinearLayoutManager(this);
    linearLayoutManager.setOrientation(LinearLayoutManager.VERTICAL);
    recyclerView = (RecyclerView)findViewById(R.id.movRecylerView);
    recyclerView.setHasFixedSize(true);
    recyclerView.setLayoutManager(linearLayoutManager);
    linearLayoutManager.setReverseLayout(true);
    linearLayoutManager.setStackFromEnd(true);

    list = new ArrayList<>();
    getList();

    adapter = new MyAdapter(list);
    recyclerView.setAdapter(adapter);
}

private void getList(){
    mStatusDB.addChildEventListener(new ChildEventListener() {
        @Override
        public void onChildAdded(DataSnapshot dataSnapshot, String s) {

            Movie movie = new Movie();

            for(DataSnapshot snapshot:dataSnapshot.getChildren()){

                String key = snapshot.getKey();
                String mov = snapshot.getValue().toString();
                movie.setMovie(value);
            }

            list.add(movie);
            adapter.notifyDataSetChanged();
        }

        @Override
        public void onChildChanged(DataSnapshot dataSnapshot, String s) {

        }

        @Override
        public void onChildRemoved(DataSnapshot dataSnapshot) {

        }

        @Override
        public void onChildMoved(DataSnapshot dataSnapshot, String s) {

        }

        @Override
        public void onCancelled(DatabaseError databaseError) {

        }
    });
}

}

在这里，您只能看到10行的问题：

请帮帮我问题出在哪里？它不在状态下的hashmap中。在图片中，第10行的状态将是-1，这意味着按钮应该是灰色的。

Answer 1

问题是，您没有检查onbindviewholder（final MyViewHolder holder,final int position)内部，这是位于position位置的项的状态。查看代码，您只在OnClickListener（）内部检查它。

在您的代码中发生的情况是，项目1的视图被回收并用于放置项目10，但视图中的星形颜色为黄色，在您的OnBindViewHolder中，您没有将其颜色重置为白色。

这是我的建议：

@Override
public void onBindViewHolder(final MyViewHolder holder, final int position){
    Movie movie = list.get(position);
    holder.setMovie(movie.getMovie());

    if(!hashMapStarButtons.containsKey(position)){
        hashMapStarButtons.put(position, -1);
    }
    if(hashMapStarButtons.get(position)==-1){
        holder.starButton.setImageDrawable(ContextCompat.getDrawable(view.getContext(),android.R.drawable.btn_star_big_off));
    } else {
         holder.starButton.setImageDrawable(ContextCompat.getDrawable(view.getContext(),android.R.drawable.btn_star_big_on));
    }
    ...
}