使用Java实现Android按钮点击变色及ArrayList操作的问题排查

• 所有按钮共用同一个全局flag变量，点击任意按钮都会累加这个值，根本没法单独记录每个按钮的选中状态。比如点一次button1后flag变成2，再点button3时flag累加成3，直接导致button3的状态判断错位，多按钮点几次状态就全乱了。
• 选中逻辑写反了：你代码里flag%2==0分支把按钮设为绿色（选中态），但执行的是从列表移除数字的操作；else分支把按钮设为默认黄色（未选中态），反而执行添加数字的操作，效果完全和需求反过来。
• 每个按钮单独写重复的点击监听代码，后续加新按钮维护成本很高。

不用给每个按钮单独维护状态，直接复用同一个点击监听器，通过按钮自身存储的标识判断对应数字、切换状态即可。
首先初始化基础变量：

// 按钮默认背景色，替换成你项目里的原始颜色值即可
private final int btnDefaultColor = bg_yellow;
private ArrayList<Integer> SeatArrayList = new ArrayList<>();

写通用的按钮点击逻辑，所有座位按钮都复用这一个监听器：

View.OnClickListener seatBtnClick = new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        // 提前给每个按钮绑定对应的座位数字，存在tag里
        int seatNo = (int) v.getTag();
        // 判断当前按钮是不是已经是选中的绿色
        boolean isSelected = v.getSolidColor() == Color.GREEN;
        
        if (isSelected) {
            // 已选中→切为未选中：恢复默认色、从列表移除对应数字
            v.setBackgroundColor(btnDefaultColor);
            SeatArrayList.remove(Integer.valueOf(seatNo));
        } else {
            // 未选中→切为选中：设为绿色、往列表加对应数字
            v.setBackgroundColor(Color.GREEN);
            SeatArrayList.add(seatNo);
        }
    }
};

给每个按钮绑定对应数字和监听器就可以了：

button1.setTag(1);
button1.setOnClickListener(seatBtnClick);

button3.setTag(3);
button3.setOnClickListener(seatBtnClick);

button4.setTag(4);
button4.setOnClickListener(seatBtnClick);

更规范的写法建议用selector背景实现，不用手动硬改背景色：给按钮写一个状态选择器drawable，选中态对应绿色、默认态对应原始黄色，点击时直接调用v.setSelected(!v.isSelected())切换状态即可，不会覆盖按钮原有的圆角、描边等样式，代码也更简洁。

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