RecyclerView选座/取消选座时图片显示异常问题

嘿，我帮你找到了座位图片错位的问题根源，咱们一步步来修复它！

核心问题原因 #

你遇到的座位点击后图片错位的情况，本质是**SeatAdapter里的空白座位判断逻辑和loadBustListSeats中生成座位数据的逻辑完全不一致**，导致RecyclerView的position和seatDetailsList的索引没法正确对应。简单说就是：你点的是第20个座位，但代码实际修改了第1个座位的状态。

具体来看：

• 在生成座位数据的loadBustListSeats方法里，你用isBlankSeat(i)判断哪些是空白座位；
• 但在SeatAdapter的getItemViewType里，你写了一套完全不同的判断规则，这就造成了ViewHolder绑定的时候，position对应的座位数据根本不是用户实际点击的那个。

修复步骤 #

1. 统一空白座位的判断逻辑

把SeatAdapter里的getItemViewType改成和数据生成逻辑一致的判断，直接复用Activity里的isBlankSeat方法，确保position和数据索引完全匹配：

@Override
public int getItemViewType(int position) {
    // 直接复用Activity的判断逻辑，保证和数据生成一致
    return isBlankSeat(position) ? 1 : 0;
}

2. 修正getItemCount的返回值

别再硬编码返回60了，直接返回seatDetailsList的实际长度，避免后续修改数据量时出现不一致：

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

3. 简化并优化座位数据生成逻辑（可选但推荐）

你当前的isBlankSeat方法逻辑有点绕，可以简化成更易懂的写法：

public boolean isBlankSeat(int i) {
    // 最后一排（索引56-59，共4个座位）不显示空白，其余行的第3个位置（索引2）设为空白
    return i < 56 && i % 5 == 2;
}

同时，座位编号的生成可以更清晰，避免特殊值判断混淆：

public List<SeatDetails> loadBustListSeats() {
    seatDetailsList = new ArrayList<SeatDetails>();
    int seatSerialNumber = 1;
    for (int i = 0; i < 60; i++) {
        if (isBlankSeat(i)) {
            seatDetailsList.add(new SeatDetails("", true, false));
        } else {
            String seatName;
            // 前两个座位固定为A、B，后续按数字递增
            if (i == 0) {
                seatName = "A";
            } else if (i == 1) {
                seatName = "B";
            } else {
                seatName = String.valueOf(seatSerialNumber);
                seatSerialNumber++;
            }
            seatDetailsList.add(new SeatDetails(seatName, false, false));
        }
    }
    return seatDetailsList;
}

4. 移除重复的图片设置操作

在点击事件里，你已经调用了seatSelected/seatDeselected，这两个方法里已经触发了notifyDataSetChanged()，会自动重新绑定视图更新图片，所以不需要手动再设置ImageView的Bitmap了：

holder.itemView.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        SeatDetails seatDetails = seatDetailsList.get(position);
        if (seatDetails.isSelected){
            seatDeselected(position);
            // 删掉这句：((ViewHolderOriginal) holder).imageView.setImageBitmap(seatIcon);
        } else {
            seatSelected(position);
            // 删掉这句：((ViewHolderOriginal) holder).imageView.setImageBitmap(seatSelect);
        }
    }
});

额外的小提醒 #

还有两个容易触发空指针的地方要注意：

• 初始化seatNo列表：在onCreate里加上seatNo = new ArrayList<>();，不然第一次选座会崩溃；
• 给amt和charge设置初始值：比如amt = 100;（根据你的实际票价调整），避免费用计算时出现异常。

内容的提问来源于stack exchange，提问作者pranali dilip gaikwad