RecyclerView按钮滚动异常：点击顶部按钮后底部按钮自动触发，如何维护选中状态？

问题描述 #

在RecyclerView中添加按钮后，点击顶部按钮，滚动视图时底部的其他按钮会自动被触发；点击视图上的按钮，滚动时底部按钮会自动处于按下状态。请问如何确保仅单个按钮被按下，并维护已按下按钮的状态？

用户提供的代码 #

Adapter代码 #

public class SingleAdapter extends RecyclerView.Adapter<SingleAdapter.SingleViewHolder> {
    private Context context;
    private ArrayList<Employee> employees;

    // if checkedPosition = -1, there is no default selection
    // if checkedPosition = 0, 1st item is selected by default
    private int checkedPosition = 0;

    public SingleAdapter(Context context, ArrayList<Employee> employees) {
        this.context = context;
        this.employees = employees;
    }

    public void setEmployees(ArrayList<Employee> employees) {
        this.employees = new ArrayList<>();
        this.employees = employees;
        notifyDataSetChanged();
    }

    @NonNull
    @Override
    public SingleViewHolder onCreateViewHolder(@NonNull ViewGroup viewGroup, int i) {
        View view = LayoutInflater.from(context).inflate(R.layout.item_employee, viewGroup, false);
        return new SingleViewHolder(view);
    }

    @Override
    public void onBindViewHolder(@NonNull SingleViewHolder singleViewHolder, int position) {
        singleViewHolder.bind(employees.get(position));
    }

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

    class SingleViewHolder extends RecyclerView.ViewHolder {

        private TextView textView;
        private ImageView imageView;
        private Button yesbutton;
        private Button nobutton;

        SingleViewHolder(@NonNull View itemView) {
            super(itemView);
            textView = itemView.findViewById(R.id.textView);
            imageView = itemView.findViewById(R.id.imageView);
            yesbutton = itemView.findViewById(R.id.yesbutton);
            nobutton = itemView.findViewById(R.id.nobutton);
        }

        void bind(final Employee employee) {
            if (checkedPosition == -1) {
                imageView.setVisibility(View.GONE);
                yesbutton.setOnClickListener(null);
                nobutton.setOnClickListener(null);
            } else {
                if (checkedPosition == getAdapterPosition()) {
                    imageView.setVisibility(View.VISIBLE);
                } else {
                    imageView.setVisibility(View.GONE);
                    yesbutton.setOnClickListener(null);
                    nobutton.setOnClickListener(null);
                }
            }
            textView.setText(employee.getName());

            itemView.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    imageView.setVisibility(View.VISIBLE);
                    if (checkedPosition != getAdapterPosition()) {
                        notifyItemChanged(checkedPosition);
                        checkedPosition = getAdapterPosition();
                    }
                }
            });

            nobutton.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    employee.setYesButtonColor(Color.RED);
                    employee.setNoButtonColor(Color.GREEN);

                    yesbutton.setBackgroundColor(employee.getYesButtonColor());
                    nobutton.setBackgroundColor(employee.getNoButtonColor());
                }
            });

            yesbutton.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    employee.setYesButtonColor(Color.GREEN);
                    employee.setNoButtonColor(Color.RED);

                    yesbutton.setBackgroundColor(employee.getYesButtonColor());
                    nobutton.setBackgroundColor(employee.getNoButtonColor());

                    if (checkedPosition != getAdapterPosition()) {
                        if(employee.getYesButtonColor() != null) {
                            yesbutton.setBackgroundColor(employee.getYesButtonColor());
                        }
                        if(employee.getNoButtonColor() != null) {
                            nobutton.setBackgroundColor(employee.getNoButtonColor());
                        }
                    }
                }
            });
        }
    }

    public Employee getSelected() {
        if (checkedPosition != -1) {
            return employees.get(checkedPosition);
        }
        return null;
    }
}

布局XML代码 #

<?xml version="1.0" encoding="utf-8"?>
<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="match_parent"
    android:orientation="vertical"
    tools:context=".SingleSelectionActivity">

    <Button
        android:id="@+id/btnGetSelected"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_margin="8dp"
        android:layout_gravity="center"
        android:text="Get Selected" />

    <androidx.recyclerview.widget.RecyclerView
        android:id="@+id/recyclerView"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

    </androidx.recyclerview.widget.RecyclerView>

</LinearLayout>

问题原因及解决方法 #

问题根源 #

这个问题是RecyclerView的视图复用机制导致的：当列表滚动时，屏幕外的Item View会被回收，用来显示新进入屏幕的Item。你的代码没有在bind方法中根据Employee的状态正确还原按钮UI，仅在点击时修改当前View的临时状态，导致复用的View带着旧状态显示出来。另外还存在几个逻辑漏洞：

• 非选中项的按钮被设置setOnClickListener(null)，导致无法响应点击；
• yesbutton点击事件里的冗余代码没有实际作用；
• 没有统一从数据源读取状态来更新UI。

修复步骤 #

1. 优化Employee类的状态存储

用枚举代替颜色值，更清晰地标记按钮选择状态：

public class Employee {
    private String name;
    public enum ButtonState {NONE, YES, NO}
    private ButtonState buttonState = ButtonState.NONE;

    // 构造器、getter、setter
    public String getName() { return name; }
    public void setName(String name) { this.name = name; }
    public ButtonState getButtonState() { return buttonState; }
    public void setButtonState(ButtonState buttonState) { this.buttonState = buttonState; }
}

2. 重构Adapter的bind方法

核心原则：所有UI状态都从数据源读取，而非依赖View临时状态，同时维护单选逻辑：

class SingleViewHolder extends RecyclerView.ViewHolder {

    private TextView textView;
    private ImageView imageView;
    private Button yesbutton;
    private Button nobutton;

    SingleViewHolder(@NonNull View itemView) {
        super(itemView);
        textView = itemView.findViewById(R.id.textView);
        imageView = itemView.findViewById(R.id.imageView);
        yesbutton = itemView.findViewById(R.id.yesbutton);
        nobutton = itemView.findViewById(R.id.nobutton);
    }

    void bind(final Employee employee) {
        textView.setText(employee.getName());
        
        // 1. 根据Employee状态还原按钮UI
        updateButtonUI(employee);
        
        // 2. Item选中标记逻辑
        imageView.setVisibility(checkedPosition == getAdapterPosition() ? View.VISIBLE : View.GONE);

        // 3. Item点击事件（仅切换选中标记）
        itemView.setOnClickListener(v -> {
            if (checkedPosition != getAdapterPosition()) {
                notifyItemChanged(checkedPosition);
                checkedPosition = getAdapterPosition();
                imageView.setVisibility(View.VISIBLE);
            }
        });

        // 4. Yes按钮点击逻辑
        yesbutton.setOnClickListener(v -> {
            // 重置之前选中项的按钮状态
            resetPreviousSelectedButtonState();
            // 更新当前项状态
            employee.setButtonState(Employee.ButtonState.YES);
            checkedPosition = getAdapterPosition();
            updateButtonUI(employee);
            notifyItemChanged(getAdapterPosition());
        });

        // 5. No按钮点击逻辑
        nobutton.setOnClickListener(v -> {
            resetPreviousSelectedButtonState();
            employee.setButtonState(Employee.ButtonState.NO);
            checkedPosition = getAdapterPosition();
            updateButtonUI(employee);
            notifyItemChanged(getAdapterPosition());
        });
    }

    // 重置上一个选中项的按钮状态
    private void resetPreviousSelectedButtonState() {
        if (checkedPosition != -1) {
            Employee oldEmployee = employees.get(checkedPosition);
            oldEmployee.setButtonState(Employee.ButtonState.NONE);
            notifyItemChanged(checkedPosition);
        }
    }

    // 统一更新按钮UI的方法
    private void updateButtonUI(Employee employee) {
        switch (employee.getButtonState()) {
            case YES:
                yesbutton.setBackgroundColor(Color.GREEN);
                nobutton.setBackgroundColor(Color.RED);
                break;
            case NO:
                yesbutton.setBackgroundColor(Color.RED);
                nobutton.setBackgroundColor(Color.GREEN);
                break;
            case NONE:
                // 设置默认按钮颜色
                yesbutton.setBackgroundColor(ContextCompat.getColor(context, android.R.color.darker_gray));
                nobutton.setBackgroundColor(ContextCompat.getColor(context, android.R.color.darker_gray));
                break;
        }
    }
}

3. 调整Adapter其他逻辑

• 移除原bind方法中对非选中项设置setOnClickListener(null)的代码，保证所有按钮可点击；
• 若不需要默认选中项，将checkedPosition初始值设为-1；
• 简化setEmployees方法：

public void setEmployees(ArrayList<Employee> employees) {
    this.employees = employees;
    checkedPosition = -1; // 重置选中状态
    notifyDataSetChanged();
}

关键要点总结 #

• 数据驱动UI：所有View状态都从数据源读取，彻底解决RecyclerView复用导致的状态混乱；
• 明确单选逻辑：点击新按钮时，重置之前选中项的状态并通知列表更新；
• 统一UI入口：将按钮UI更新封装为单独方法，避免重复代码，保证逻辑一致性。

内容的提问来源于stack exchange，提问作者Obakeng Balatseng